On Wed, 24 Jun, at 07:14:43PM, Luck, Tony wrote: > > Another option would be to have 2 structs, the first one > > "struct cper_sec_mem_err" holds the structure as defined by UEFI > > 2.1, the 2nd one "struct cper_sec_mem_err_24_ext" holds the 4 > > elements added in UEFI 2.3.1. > > Reading some more of the UEFI 2.5 spec ... I see we are in for a world of pain > here. > > 2.5 adds some small tweaks to the memory structure (adding a couple of extra > bits to the "row" entry that can be grabbed from the formerly reserved byte > at offset 73). But then there is a whole new GUID for a "Memory Error Section 2" > which has doubled the width of the device, row, column, rank, and bit_pos fields > together with adding two new fields for chip_id and status. This will be painful > because we hardwired the old sizes into extlog_mem_event in <ras/ras_event.h> Can't you just create a new memory event type? Granted, you'll need changes to the ras daemon, but since there's a new GUID involved I think that's forgivable. -- Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
