Re: [RFC EDAC/GHES] edac: lock module owner to avoid error report conflicts

Borislav Petkov
2012-11-01T11:05:19Z

+ Tony.

On Wed, Oct 31, 2012 at 11:58:15AM -0200, Mauro Carvalho Chehab wrote:
> There's a know bug that happens when apei/ghes is loaded together
> with an EDAC module: the same error is reported several times,
> as ghes calls mcelog, with, in tune, calls edac.

This is exactly why I think APEI is crap. So it is a completely useless
additional layer between the MCA code and the rest.

The #MC handler runs, logs the error, and then a split happens which
runs in parallel:

* we do mce_log which carries the error to EDAC
* we enter APEI, do some mumbo jumbo and then do mce_log AGAIN! Wtf?

So, in order to sort this out properly, let's take a step back first:
what do we actually want to do?

* the error coming from APEI still needs to get decoded by EDAC? If yes,
then WTF we need APEI for anyway?

* the error coming from APEI is already decoded, so no need for EDAC? I
highly doubt that.

* add a filter to the MCE code so that certain types of errors are not
reported by it but by APEI so that the double reporting doesn't happen?

Right about now, I'm open for hints as to why we need that APEI crap at
all. And I don't want to hear that "clear interface so that OS coders
don't need to know the hardware" bullshit argument from the sick world
of windoze.

Thanks.

--
Regards/Gruss,
Boris.