On Wed, Jun 19, 2013 at 07:05:16PM +0000, Luck, Tony wrote: > > Why, fill out struct mce and do mce_log(mce) does not suffice? > > There is (or should be) Ha! > a lot more interesting stuff in the CPER than just the address. Stuff > that we don't have fields for in the existing mcelog structure. We > also need to treat filtered records from modern APEI implementations a > bit differently from the old stuff. Great, the CPER record is described in the UEFI spec. Those BIOS people are all like a mafia. Ok, seriously: so the situation should still be fine, FF reported errors get the CPER format while the rest, the "old" MCE format. cper.c is doing printk so I'm guessing it would need to get its own tracepoint and carry that to userspace. Concerning the RAS daemon, Robert and I are making good progress so once we have the persistent events in perf, we can read that tracepoint in userspace and do whatever we want with the error info. > The original user of this code was Westmere-EX, which used it as a > workaround for a missing address in MCi_ADDR for corrected errors. > So in that scenario we had every error being reported and mcelog(8) > deamon doing the threshold analysis to decide when to take action. > > In this new modern world - Naveen wants to have the BIOS decide the > threshold, so we'd like Linux to take some action as soon as it sees > just one CPER. Why would Linux have to intervene if it is doing FF - wasn't the deal behind Firmware First for the firmware to get the error first and handle accordingly? -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. -- -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html