Messages in this thread | | | From | "Liu, Jinsong" <> | Subject | RE: [Xen-devel] [PATCH 1/3] Add mcelog support for xen platform | Date | Mon, 23 Apr 2012 15:43:47 +0000 |
| |
Luck, Tony wrote: >> Because, if you'd hooked into it, just imagine one fine day, when we >> remove mcelog support, what screaming the xen people will be doing >> when mcelog doesn't work anymore. > > Agreed. Even before we get to deleting mcelog, "struct mce" can > change (new fields could be added) ... and you don't want to have > your hypervisor to have to know which version of Linux it is talking > to. >
Hmm, adding new fields block neither native nor xen -- xen just ignore new added fields, that would be no problem for xen: /* Fields are zero when not available */ struct mce { ... }
and, hypervisor only record error into its mc_info cookie (in its own format), it doesn't care what's dom0's version and how dom0 translate it -- that's dom0's business. The role of hypervisor is virtual 'platform', just like h/w platform don't care what linux version runs over it.
Thanks, Jinsong-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |