On 23.06.11 13:34:56, Paul Gortmaker wrote:
> From: Robert Richter <[email protected]>
> 
>                    -------------------
>     This is a commit scheduled for the next v2.6.34 longterm release.
>     If you see a problem with using this for longterm, please comment.
>                    -------------------
> 
> commit c7c25802b39c443b3745cfa973dc49a97a3491f8 upstream.
> 
> Disable preemption in init_ibs(). The function only checks the
> ibs capabilities and sets up pci devices (if necessary). It runs
> only on one cpu but operates with the local APIC and some MSRs,
> thus it is better to disable preemption.
> 
> [    7.034377] BUG: using smp_processor_id() in preemptible [00000000] code: 
> modprobe/483
> [    7.034385] caller is setup_APIC_eilvt+0x155/0x180
> [    7.034389] Pid: 483, comm: modprobe Not tainted 2.6.37-rc1-20101110+ #1
> [    7.034392] Call Trace:
> [    7.034400]  [<ffffffff812a2b72>] debug_smp_processor_id+0xd2/0xf0
> [    7.034404]  [<ffffffff8101e985>] setup_APIC_eilvt+0x155/0x180
> [ ... ]
> 
> Addresses https://bugzilla.kernel.org/show_bug.cgi?id=22812
> 
> Reported-by: <[email protected]>
> Signed-off-by: Robert Richter <[email protected]>
> Cc: [email protected] <[email protected]>
> Cc: Peter Zijlstra <[email protected]>
> Cc: Frederic Weisbecker <[email protected]>
> Cc: Rafael J. Wysocki <[email protected]>
> Cc: Dan Carpenter <[email protected]>
> Cc: Andrew Morton <[email protected]>
> LKML-Reference: <[email protected]>
> [ small cleanups ]
> Signed-off-by: Ingo Molnar <[email protected]>
> Signed-off-by: Paul Gortmaker <[email protected]>
> ---
>  arch/x86/oprofile/op_model_amd.c |   24 ++++++++++++++++--------
>  1 files changed, 16 insertions(+), 8 deletions(-)

This patch introduces a sleeping-while-atomic bug. There is another
upstream fix on top, see:

commit 3d2606f42984613d324ad3047cf503bcddc3880a
Author: Robert Richter <[email protected]>
Date:   Fri May 20 09:46:54 2011 +0200

    oprofile, x86: Enable preemption during pci device setup in IBS init

This fix should be applied too (or instead).

-Robert

-- 
Advanced Micro Devices, Inc.
Operating System Research Center

_______________________________________________
stable mailing list
[email protected]
http://linux.kernel.org/mailman/listinfo/stable

Reply via email to