]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit
ARM: OMAP2+: powerdomain: use raw_smp_processor_id() for trace
authorTero Kristo <t-kristo@ti.com>
Fri, 9 Mar 2018 09:50:20 +0000 (11:50 +0200)
committerStefan Bader <stefan.bader@canonical.com>
Mon, 1 Oct 2018 12:56:09 +0000 (14:56 +0200)
commit11effa9fb7fca717ac0444b75936d5dbdb98622a
treec59820f29f0cb1040d342acd38d74807b95ab296
parentdaa4667dfec697ec25f2b26520e099c46467a50c
ARM: OMAP2+: powerdomain: use raw_smp_processor_id() for trace

BugLink: http://bugs.launchpad.net/bugs/1794889
[ Upstream commit 33e9572483031a79ad0a4468064675144d9269ec ]

smp_processor_id() checks preemption if CONFIG_DEBUG_PREEMPT is enabled,
causing a warning dump during boot:

[    5.042377] BUG: using smp_processor_id() in preemptible [00000000] code: swapper/0/1
[    5.050281] caller is pwrdm_set_next_pwrst+0x48/0x88
[    5.055330] CPU: 1 PID: 1 Comm: swapper/0 Not tainted 4.14.24-g57341df0b4 #1

Use the raw_smp_processor_id() for the trace instead, this value does
not need to be perfectly correct. The alternative of disabling preempt
is too heavy weight operation to be applied in PM hot path for just
tracing purposes.

Signed-off-by: Tero Kristo <t-kristo@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
arch/arm/mach-omap2/powerdomain.c