>>> 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".

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to