Re: [Xen-devel] [PATCH] x86: add an extra check when validating a huge pv L2 entry

[Keir Fraser <keir.fraser@xxxxxxxxxxxxx>]

On 09/09/2009 17:15, "Michael J Coss" <mjcoss@xxxxxxxxxxxxxxxxxx> wrote:

>> Oh, yes - I got confused (again) by the inconsistencies in what return
>> values mean for the various functions.
>> 
>> Jan
>>   
> I don't think that that's correct.  The else clause is for the
> mfn_valid(m) failure case, and it seems appropriate to set the rc to
> some non-zero value.

You're confused - look at the surrounding context of the patch. Anyway, I
ultimately checked in soemthing even clearer, as c/s 20189.

 K.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel