perfcounters: fix "perf counters kills oprofile" bug, v2
authorIngo Molnar <mingo@elte.hu>
Thu, 5 Feb 2009 14:23:08 +0000 (15:23 +0100)
committerIngo Molnar <mingo@elte.hu>
Thu, 5 Feb 2009 14:24:14 +0000 (15:24 +0100)
commit82aa9a1829199233f9bdaf26e2ee271114f4701e
tree638cccf1b1708bdce1cc45d54408c0907f051128
parent5b75af0a02fcf3b8899f38ff6f22164c5d8e2fdd
perfcounters: fix "perf counters kills oprofile" bug, v2

Impact: fix kernel crash

Both oprofile and perfcounters register an NMI die handler, but only one
can handle the NMI.  Conveniently, oprofile unregisters it's notifier
when not actively in use, so setting it's notifier priority higher than
perfcounter's allows oprofile to borrow the NMI for the duration of it's
run.  Tested/works both as module and built-in.

While testing, I found that if kerneltop was generating NMIs at very
high frequency, the kernel may panic when oprofile registered it's
handler.  This turned out to be because oprofile registers it's handler
before reset_value has been allocated, so if an NMI comes in while it's
still setting up, kabOom.  Rather than try more invasive changes, I
followed the lead of other places in op_model_ppro.c, and simply
returned in that highly unlikely event.  (debug warnings attached)

Signed-off-by: Mike Galbraith <efault@gmx.de>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/oprofile/op_model_ppro.c