Re: [PATCH 3/3] ghes_edac: add platform check to enable ghes_edac

From: Kani, Toshimitsu
Date: Mon Jul 24 2017 - 13:47:31 EST


On Mon, 2017-07-24 at 18:37 +0200, Borislav Petkov wrote:
> On Mon, Jul 24, 2017 at 03:56:27PM +0000, Kani, Toshimitsu wrote:
> > Yes, Mauro has already pointed this out.ÂÂAs I replied to him, we
> > do have a separate series of platforms that do not have built-in
> > RAS, and
>
> So this whitelist entry
>
> +static struct acpi_oemlist oemlist[] = {
> +ÂÂÂÂÂÂÂ{"HPEÂÂÂ", "ServerÂÂ", 0, ACPI_SIG_FADT, all_versions},
> +ÂÂÂÂÂÂÂ{ } /* End */
> +};
>
> looks like it'll match every HP server platform not only the ones
> with built-in RAS.

I assumed our platforms w/o build-in RAS do not implement GHES, but I
will check for sure. Also, all our previous/current platforms have
"HP".

Thanks,
-Toshi