[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH 3/4] x86/mce: Translate passed-in GPA to host machine address
On 15/09/15 14:42, Haozhong Zhang 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". Please always always always write useful error messages. "illegal GPFN" is useless without also stating which value is believed to be invalid. Furthermore, per the clarification in c/s e758ed1, "gfn" is the correct term to use here. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |