RE: [PATCH 1/3] x86/MCE, EDAC/mce_amd: Add support for new MCA_SYND{1,2} registers
From: Luck, Tony
Date: Mon Oct 24 2022 - 18:48:51 EST
We already have:
__u8 cpuvendor; /* Kernel's X86_VENDOR enum */
So the mce structure contains which vendor created it.
> I guess a u8 vendor_info[VENDOR_INFO_SIZE] or so which we can extend
> later if needed.
Extending is hard because we already boxed in the two AMD specific fields
with some generic fields that follow (ppin & microcode).
But we could change the current form to be something like:
union {
struct vendor_info {
__u64 vendor_info[2];
};
struct vendor_amd_info {
__u64 synd; /* MCA_SYND MSR: only valid on SMCA systems */
__u64 ipid; /* MCA_IPID MSR: only valid on SMCA systems */
};
};
to make it clear that these 16 bytes are up for grabs to be re-interpreted based on
the value of "cpuvendor" (and possibly also "cpuid" if a vendor wants different data
logged for different models).
-Tony