[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4DD17EAF.4040305@gmail.com>
Date: Mon, 16 May 2011 23:44:47 +0400
From: Cyrill Gorcunov <gorcunov@...il.com>
To: Huang Ying <ying.huang@...el.com>
CC: huang ying <huang.ying.caritas@...il.com>,
Ingo Molnar <mingo@...e.hu>, Don Zickus <dzickus@...hat.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Andi Kleen <andi@...stfloor.org>,
Robert Richter <robert.richter@....com>,
Andi Kleen <ak@...ux.intel.com>
Subject: Re: [RFC] x86, NMI, Treat unknown NMI as hardware error
On 05/16/2011 05:09 AM, Huang Ying wrote:
...
>>
>> I'm personally fine even if it's enabled by default, only worried to have
>> an option to disable hwerr from boot line.
>
> The white list mechanism is not sufficient? Spurious unknown NMI can
> occur on white list machines? People don't want to protect their data?
>
I suppose no, it's not sufficient considering how many cpu errata already
out in general. And I see no guarantee that unknown NMIs never triggers on
white list machines and I know that you know that as well ;)
>>> And, I am not a big fan of notifiers, that makes code hard to be
>>> understood. If you have concerns about the size of traps.c, we can
>>> move all NMI logic to a new file.
>>
>> Ying, the concern is rather related to the code scheme in general. Since
>> we have notifiers I think the better way to be consistent here and use
>> hwerr notifier too. But it's IMHO ;)
>
> As for go notifiers or not. IMHO, a rule can be:
>
> - If it is something like a driver, than it should go notifier
> - If it is architectural/PC defacto standard, it can sit outside of notifier.
>
> I think that seeing unknown NMI as hardware error should be part of PC
> defacto standard. Do you think so?
Ying, movin the handler into notifier is my IMHO, this would release nmi handler
from details since with time more and more "standarts" would appear. If Don, Ingo
and x86-team is fine with your approach -- of course I'm pretty fine too ;)
>
> Best Regards,
> Huang Ying
/me Just found Don has some more concerns
--
Cyrill
--
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/
Powered by blists - more mailing lists