Re: [RFC PATCH 5/5] GHES: Make NMI handler have a single reader

Borislav Petkov
2015-04-23T08:59:49Z

On Thu, Apr 23, 2015 at 10:39:58AM +0200, Jiri Kosina wrote:
> Three weeks have passed, therefore I find this an appropriate time for a
> friendly ping :)
>
> Rafael? Naoya? Huang?
>
> This fixes a contention spinlock problem in NMI observed on a real HW, so
> it would be really nice to have it fixed.

I think we should apply this.

Here's why: nothing in the ghes_notify_nmi() handler does CPU-specific
accesses. It iterates over the list of ghes sources which do NMI
notification but those sources are the *same* regardless of which core
does the access as their addresses are per-source, i.e. in that

struct acpi_generic_address error_status_address;

thing.

And it is a safe bet to say that all that error information is
serialized in the firmware for the error source to consume.

So I'm going to route this through the RAS tree unless Rafael wants to
take it.

Ok?

Tony, objections?

--
Regards/Gruss,
Boris.

ECO tip #101: Trim your mails when you reply.
--