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

[Justin He]

> -----Original Message-----
> From: Borislav Petkov <bp@xxxxxxxxx>
> Sent: Wednesday, August 10, 2022 4:28 PM
> To: Justin He <Justin.He@xxxxxxx>
> Cc: Kani, Toshi <toshi.kani@xxxxxxx>; Rafael J. Wysocki <rafael@xxxxxxxxxx>;
> Len Brown <lenb@xxxxxxxxxx>; James Morse <James.Morse@xxxxxxx>;
> Tony Luck <tony.luck@xxxxxxxxx>; Mauro Carvalho Chehab
> <mchehab@xxxxxxxxxx>; Robert Richter <rric@xxxxxxxxxx>; Shuai Xue
> <xueshuai@xxxxxxxxxxxxxxxxx>; Jarkko Sakkinen <jarkko@xxxxxxxxxx>; ACPI
> Devel Maling List <linux-acpi@xxxxxxxxxxxxxxx>; Linux Kernel Mailing List
> <linux-kernel@xxxxxxxxxxxxxxx>; open list:EDAC-CORE
> <linux-edac@xxxxxxxxxxxxxxx>
> 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)