Re: [PATCH v4] cxl/events: Use a common struct for DRAM and General Media events

From: Alison Schofield
Date: Wed May 22 2024 - 11:08:22 EST


On Tue, May 21, 2024 at 04:05:08PM +0200, Fabio M. De Francesco wrote:
> cxl_event_common was an unfortunate naming choice and caused confusion with
> the existing Common Event Record. Furthermore, its fields didn't map all
> the common information between DRAM and General Media Events.
>
> Remove cxl_event_common and introduce cxl_event_media_hdr to record common
> information between DRAM and General Media events.
>
> cxl_event_media_hdr, which is embedded in both cxl_event_gen_media and
> cxl_event_dram, leverages the commonalities between the two events to
> simplify their respective handling.
>
> Suggested-by: Dan Williams <dan.j.williams@xxxxxxxxx>
> Signed-off-by: Fabio M. De Francesco <fabio.m.de.francesco@xxxxxxxxxxxxxxx>

Reviewed-by: Alison Schofield <alison.schofield@xxxxxxxxx>

> ---
>