Re: [Xen-devel] [PATCH RFC v2 5/7] libxl/vNUMA: VM config parsing functions

[Elena Ufimtseva <ufimtseva@xxxxxxxxx>]

On Thu, Sep 19, 2013 at 10:29 AM, George Dunlap
<George.Dunlap@xxxxxxxxxxxxx> wrote:
> On Fri, Sep 13, 2013 at 9:50 AM, Elena Ufimtseva <ufimtseva@xxxxxxxxx> wrote:
>> vNUMA VM config parsing functions.
>> Parses VM vNUMA related config, verifies and
>> sets default values for errorneous parameters.
>> If sum of all vnodes memory sizes does not match
>> memory, it will be adjusted accordingly;
>
> Also:
>
> * I think if there's a mismatch in the config file we should error
> out, not allow one bit of the config to override another part.
> * It's probably worth making a default for vnumamem as well, as there
> is for vcpu_to_vnode.
>
>  -George

The other simplified way is to specify only two values. One for the
distance between same node and another between not the same nodes.
like this:

vdistance = [10, 20]

and this will expand to

10 20 20 20
20 10 20 20
20 20 10 20
20 20 20 10

for 4 nodes.
There is a loss in flexibility of course and other parsing can remain
the same or as George proposed, using lists.

Elena






-- 
Elena

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