Re: [PATCH 00/12] ras: share firmware-first estatus handling

[Borislav Petkov <bp@alien8.de>]

On Thu, Jan 15, 2026 at 12:17:17PM +0000, Ahmed Tiba wrote:
> In drivers/acpi/apei/ghes.c, this subset of CPER handling
> is already implemented using a number of helpers mostly named
> estatus_* rather than cper_*.

Which helpers are those?

$ git grep -Ei "\Westatus_" drivers/acpi/apei/ghes.c

doesn't give me a single one. Maybe I can't grep.

> The naming therefore originates from the existing GHES implementation, not
> from a new abstraction.

And even if, calling it a "estatus core" and making it a separate thing is
just confusing and unnecessary.

> What I did was lift that existing estatus_* logic so it can be reused by a
> non-ACPI provider, rather than duplicating the CPER handling
> in a parallel DT path.

Nothing wrong with that as long as you simply export common helpers and we
agree on how they're going to be shared. We do stuff like that all the time.

> Because the DT-based path still needs a clean way to call into the shared
> logic without dragging in ACPI plumbing.

Yes, carve them out in a separate compilation unit and select it in your DT
glue and that's it. No need for a Kconfig item and calling it a "core" bla.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette