RE: 回复: [PATCH] ACPI: APEI: move edac_init ahead of ghes platform drv register

Justin He
2022-08-10T08:53:36Z

> -----Original Message-----
> From: Borislav Petkov <bp@alien8.de>
> Sent: Wednesday, August 10, 2022 4:28 PM
> To: Justin He <Justin.He@arm.com>
> Cc: Kani, Toshi <toshi.kani@hpe.com>; Rafael J. Wysocki <rafael@kernel.org>;
> Len Brown <lenb@kernel.org>; James Morse <James.Morse@arm.com>;
> Tony Luck <tony.luck@intel.com>; Mauro Carvalho Chehab
> <mchehab@kernel.org>; Robert Richter <rric@kernel.org>; Shuai Xue
> <xueshuai@linux.alibaba.com>; Jarkko Sakkinen <jarkko@kernel.org>; ACPI
> Devel Maling List <linux-acpi@vger.kernel.org>; Linux Kernel Mailing List
> <linux-kernel@vger.kernel.org>; open list:EDAC-CORE
> <linux-edac@vger.kernel.org>
> Subject: Re: 回复: [PATCH] ACPI: APEI: move edac_init ahead of ghes
> platform drv register
>
> On Wed, Aug 10, 2022 at 06:00:49AM +0000, Justin He wrote:
> > Is this strange or any other side effects?
>
> This is no different than what we do now on x86 MCA.
>
> If the logging machinery is modules, they consume error records only when
> loaded.
Ok, got it, thanks a lot
From my local testing, after the provided patch from you + notifier register.
It works after I resolved a few other issues.

When I verify different building kconfig(X86/Arm64, ghes_edac[M]/[*]), and booting, I will send out
the new version.

>
> Btw, pls get rid of this if you want to be posting on public mailing lists. You can
> ask your other ARM colleagues how they solved it. :)
>
Sorry, missed one configuration in the mailbox ☹

--
Cheers,
Justin (Jia He)