linux-kernel - Re: [PATCH 2/3] x86, MCE: Drop default decoding notifier

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4DA5B1B1.5090905@redhat.com>
Date:	Wed, 13 Apr 2011 10:22:41 -0400
From:	Prarit Bhargava <prarit@...hat.com>
To:	Borislav Petkov <bp@...64.org>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Russ Anderson <rja@....com>,
	"Luck, Tony" <tony.luck@...el.com>,
	"dzickus@...hat.com" <dzickus@...hat.com>,
	"mstowe@...hat.com" <mstowe@...hat.com>,
	"dnelson@...hat.com" <dnelson@...hat.com>,
	"rja@...ricas.sgi.com" <rja@...ricas.sgi.com>
Subject: Re: [PATCH 2/3] x86, MCE: Drop default decoding notifier



On 04/13/2011 10:18 AM, Borislav Petkov wrote:
> On Wed, Apr 13, 2011 at 10:01:22AM -0400, Prarit Bhargava wrote:
>   
>> On 04/13/2011 09:36 AM, Borislav Petkov wrote:
>>     
>>> From: Borislav Petkov <borislav.petkov@....com>
>>>
>>> The default notifier doesn't make a lot of sense to call in the
>>> correctable errors case. Drop it and emit the mcelog decoding hint only
>>> in the uncorrectable errors case and when no notifier is registered.
>>>
>>> Signed-off-by: Borislav Petkov <borislav.petkov@....com>
>>> ---
>>>  arch/x86/include/asm/mce.h       |    4 ++--
>>>  arch/x86/kernel/cpu/mcheck/mce.c |   26 +++++++++-----------------
>>>  2 files changed, 11 insertions(+), 19 deletions(-)
>>>
>>>       
>>   
>>     
>>> +extern atomic_t mce_decoders;
>>> +
>>>       
>> Boris,
>>
>> I don't think we need to do this.  I think we can use the existing notifier chain tools to do this check for us ... *untested and uncompiled* patch below.
>>
>>   	 * Print out human-readable details about the MCE error,
>>     
>>>  	 * (if the CPU has an implementation for that)
>>>  	 */
>>> -	atomic_notifier_call_chain(&x86_mce_decoder_chain, 0, m);
>>> +	if (!atomic_read(&mce_decoders)) {
>>> +		pr_emerg(HW_ERR "No human readable MCE decoding support on this CPU type.\n");
>>> +		pr_emerg(HW_ERR "Run the above through 'mcelog --ascii' to decode.\n");
>>>       
>> I thought we didn't want these lines at all for CE errors?
>>     
> This is the UC only case - we call print_mce() before we panic.
>   

.. right ... but we would still see nonsensical messages before the
panic, which will confuse the end user.  IMO, dump the messages unless
it's UC.  And we shouldn't print out the human readable message at all.

I'm installing F14 on  a system that is known to generate CEs @
boot-time.  I'll test patches+latest upstream to here to see what happens...

P.

>
>   
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/