x86/mce: Fix regression. All error records should report via /dev/mcelog
authorTony Luck <tony.luck@intel.com>
Mon, 9 Feb 2015 17:36:53 +0000 (09:36 -0800)
committerTony Luck <tony.luck@intel.com>
Mon, 9 Feb 2015 17:36:53 +0000 (09:36 -0800)
commita2413d8b2982fb6823cdb66704938a9a345d0a7d
tree0f10c9846f0082b144af631b4f64009052bfb7dd
parentbfa76d49576599a4b9f9b7a71f23d73d6dcff735
x86/mce: Fix regression. All error records should report via /dev/mcelog

I'm getting complaints from validation teams that have updated their
Linux kernels from ancient versions to current. They don't see the
error logs they expect. I tell the to unload any EDAC drivers[1], and
things start working again.  The problem is that we short-circuit
the logging process if any function on the decoder chain claims to
have dealt with the problem:

ret = atomic_notifier_call_chain(&x86_mce_decoder_chain, 0, m);
if (ret == NOTIFY_STOP)
return;

The logic we used when we added this code was that we did not want
to confuse users with double reports of the same error.

But it turns out users are not confused - they are upset that they
don't see a log where their tools used to find a log.

I could also get into a long description of how the consumer of this
log does more than just decode model specific details of the error.
It keeps counts, tracks thresholds, takes actions and runs scripts
that can alert administrators to problems.

[1] We've recently compounded the problem because the acpi_extlog
driver also registers for this notifier and also returns NOTIFY_STOP.

Signed-off-by: Tony Luck <tony.luck@intel.com>
arch/x86/kernel/cpu/mcheck/mce.c