Re: [PATCH v2 0/6] CPER: Add Memory Error Section 2 support

[Pierre Gondois]

On 10/20/26 19:10, Jonathan Cameron wrote:
> On Mon, 19 Oct 2026 22:38:39 +0200
> Pierre Gondois <pierre.gondois@xxxxxxx> wrote:
>
> > The support is mostly duplicated from Memory Error Section handling.
> > Other than differences coming from the content of  Memory Error
> > Section 1 and 2, differences in the support are:
> >
> > - v2 do not rely on a compact "struct cper_mem_err_compact" struct.
> > This structure was used to support older versions of the Memory
> > Error Section (1). This is not needed for the new structure.
> >
> > - The compact version is also used by the tracing framework.
> > To avoid relying and propagating a non-standard record format,
> > through trace event (cf. extlog_mem event) the new trace event
> > prints each field individually.
> >
> > The last patch of the serie was only build-tested due to the
> > absence of relevant hardware. apei_mce_report_mem_error() and
> > arch_apei_report_mem_error() implementations were also removed since
> > last version.
> >
> > It will also require a proper handling from the rasdaemon. Applying
> > this last patch without rasdaemon support will silent the reports of
> > Memory Error Section 2 error, cf. ras_userspace_consumers() check.
> Hi Pierre,
>
> I'd slow down a bit.  We are mid merge window and convention is normally
> one version a week or so anyway.  Better to gather up feedback from
> a number of folk and spin only one revision to cover it all.
Hello Jonathan,

Ok sure, as the v1 was reviewed quickly I thought there was a chance
of having the serie accepted quickly aswell. I ll wait a bit more.

Thanks for the review,

Pierre