Re: [PATCH v5 3/6] cxl/events: Update General Media Event Record to CXL spec rev 3.1

Ira Weiny
2025-01-10T17:55:02Z

shiju.jose@ wrote:
> From: Shiju Jose <shiju.jose@huawei.com>
>
> CXL spec rev 3.1 section 8.2.9.2.1.1 Table 8-45, General Media Event
> Record has updated with following new fields and new types for Memory
> Event Type and Transaction Type fields.
> 1. Advanced Programmable Corrected Memory Error Threshold Event Flags
> 2. Corrected Memory Error Count at Event
> 3. Memory Event Sub-Type
>
> The format of component identifier has changed (CXL spec 3.1 section
> 8.2.9.2.1 Table 8-44).
>
> Update the general media event record and general media trace event for
> the above spec changes. The new fields are inserted in logical places.
>
> Example trace log of cxl_general_media trace event,
>
> cxl_general_media: memdev=mem0 host=0000:0f:00.0 serial=3 log=Fatal : \
> time=156831237413 uuid=fbcd0a77-c260-417f-85a9-088b1621eba6 len=128 \
> flags='0x1' handle=1 related_handle=0 maint_op_class=2 \
> maint_op_sub_class=4 : dpa=30d40 dpa_flags='' \
> descriptor='UNCORRECTABLE_EVENT|THRESHOLD_EVENT|POISON_LIST_OVERFLOW' \
> type='TE State Violation' sub_type='Media Link Command Training Error' \
> transaction_type='Host Inject Poison' channel=3 rank=33 device=5 \
> validity_flags='CHANNEL|RANK|DEVICE|COMPONENT|COMPONENT PLDM FORMAT' \
> comp_id=03 74 c5 08 9a 1a 0b fc d2 7e 2f 31 9b 3c 81 4d \
> comp_id_pldm_valid_flags='PLDM Entity ID | Resource ID' \
> pldm_entity_id=74 c5 08 9a 1a 0b pldm_resource_id=fc d2 7e 2f \
> hpa=ffffffffffffffff region= \
> region_uuid=00000000-0000-0000-0000-000000000000 \
> cme_threshold_ev_flags='Corrected Memory Errors in Multiple Media \
> Components|Exceeded Programmable Threshold' cme_count=120
>
> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> Reviewed-by: Davidlohr Bueso <dave@stgolabs.net>

Reviewed-by: Ira Weiny <ira.weiny@intel.com>

[snip]