On 05/27/16 06:27, Jan Beulich wrote:
> >>> On 27.05.16 at 14:13, <haozhong.zh...@intel.com> wrote:
> > --- a/tools/tests/mce-test/tools/xen-mceinj.c
> > +++ b/tools/tests/mce-test/tools/xen-mceinj.c
> > @@ -317,7 +317,9 @@ static int inject_mci_addr(xc_interface *xc_handle,
> > domid_t domid)
> > {
> > return add_msr_bank_intpose(xc_handle, cpu_nr,
> > - MC_MSRINJ_F_INTERPOSE | MC_MSRINJ_F_GPADDR,
> > + MC_MSRINJ_F_INTERPOSE |
> > + (domid > DOMID_FIRST_RESERVED ?
>
> Simply by the name of it I would say this ought to be >=. However,
> DOMID_FIRST_RESERVED == DOMID_SELF, so the > seems right
> here (on the hypervisor side it should be changed though). But for
> clarity I would suggest either adding a respective comment or
> indeed using
>
> domid >= DOMID_FIRST_RESERVED && domid != DOMID_SELF
>
> (which I would hope the compiler can fold).
>
I'll change on both sides in the next version.
Thanks,
Haozhong
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
