lkml.org 
[lkml]   [2012]   [Apr]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRE: [Xen-devel] [PATCH 1/3] Add mcelog support for xen platform
    Date
    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/

    \
     
     \ /
      Last update: 2012-04-23 17:47    [W:0.027 / U:37.912 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site