[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <BN6PR12MB166720AEED7FE086308BC4ABF8390@BN6PR12MB1667.namprd12.prod.outlook.com>
Date: Thu, 9 Jan 2020 20:39:57 +0000
From: "Ghannam, Yazen" <Yazen.Ghannam@....com>
To: Borislav Petkov <bp@...en8.de>
CC: Jan H. Schönherr <jschoenh@...zon.de>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-edac@...r.kernel.org" <linux-edac@...r.kernel.org>,
Tony Luck <tony.luck@...el.com>,
Thomas Gleixner <tglx@...utronix.de>,
Ingo Molnar <mingo@...hat.com>,
"H. Peter Anvin" <hpa@...or.com>, "x86@...nel.org" <x86@...nel.org>
Subject: RE: [PATCH v2 6/6] x86/mce: Dynamically register default MCE handler
> -----Original Message-----
> From: Borislav Petkov <bp@...en8.de>
> Sent: Wednesday, January 8, 2020 5:04 AM
> To: Ghannam, Yazen <Yazen.Ghannam@....com>
> Cc: Jan H. Schönherr <jschoenh@...zon.de>; linux-kernel@...r.kernel.org; linux-edac@...r.kernel.org; Tony Luck
> <tony.luck@...el.com>; Thomas Gleixner <tglx@...utronix.de>; Ingo Molnar <mingo@...hat.com>; H. Peter Anvin <hpa@...or.com>;
> x86@...nel.org
> Subject: Re: [PATCH v2 6/6] x86/mce: Dynamically register default MCE handler
>
> On Wed, Jan 08, 2020 at 04:24:33AM +0000, Ghannam, Yazen wrote:
> > I've encountered an issue where EDAC didn't load (either due to a
> > bug or missing hardware enablement) and the MCE got swallowed by the
> > mcelog notifier. The mcelog utility wasn't in use, so there was no
> > record of the MCE.
>
> Can you reproduce this using the injector?
>
Yes, I was just able to do it. Here's what I did.
1) Unload EDAC decoder modules.
2) Inject a corrected MCE using mce-inject.
3) Observe "Machine check events logged" message and no other MCE info.
4) Manually load mcelog and find MCE info.
It seems to me that the issue is the mcelog notifier counts toward the number
of notifiers, so the default notifier doesn't print anything.
Of course, this can be avoided if the EDAC modules are loaded, or if mcelog is
disabled in the kernel config, or if rasdaemon is used, etc.
Thanks,
Yazen
Powered by blists - more mailing lists