On Tue, Sep 15, 2015 at 07:47:49AM -0600, Jan Beulich wrote:
> >>> On 15.09.15 at 15:42, <haozhong.zh...@intel.com> wrote:
> > On Tue, Sep 15, 2015 at 07:24:34AM -0600, Jan Beulich wrote:
> >> >>> On 15.09.15 at 10:29, <haozhong.zh...@intel.com> wrote:
> >> > + mfn = mfn_x(get_gfn(d, gpfn, &t));
> >> > +
> >> > + if (mfn == INVALID_MFN) {
> >> > + put_domain(d);
> >> > + return x86_mcerr("do_mca inject: illegal MSR value",
> >> > + -EINVAL);
> >>
> >> This message should be better distinguishable from the one further
> >> down (or else it's pretty pointless) - perhaps you mean GPFN instead
> >> of MSR?
> >>
> >
> > Yes, I'll change it to "do_mca_inject: illegal GPFN".
>
> I'd suggest "wrong" or "bad" instead of "illegal".
>
will change to "bad"
> Jan
>
- Haozhong Zhang
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
