[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v8 1/3] xen/libxc: Allow changing max number of hypervisor cpuid leaves



>>> On 22.04.14 at 14:32, <boris.ostrovsky@xxxxxxxxxx> wrote:
> On 04/22/2014 05:45 AM, Jan Beulich wrote:
>>>>> On 21.04.14 at 23:57, <boris.ostrovsky@xxxxxxxxxx> wrote:
>>> +    if ( (limit < 2) || (limit > XEN_CPUID_MAX_NUM_LEAVES) )
>>> +        limit = XEN_CPUID_MAX_NUM_LEAVES;
>> Almost: If limit < 2, you would be better off using 2 rather than the
>> default (which may be growing).
> 
> My thinking was that for any invalid limit value we should ignore what 
> the user file specified, not try to guess what was meant.

And I think that 0 or 1 aren't really invalid - if a lower limit is wanted,
provided as low as possible one rather than ignoring the request (I
didn't look at what upsets hvmloader - I suppose this could be fixed
rather than going with the slightly awkward lower limit of 2).

Jan


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.