Sure.
--jyh
>-----Original Message-----
>From: Keir Fraser [mailto:keir.fraser@xxxxxxxxxxxxx]
>Sent: Wednesday, January 06, 2010 5:29 PM
>To: Jiang, Yunhong; andre.przywara@xxxxxxx
>Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
>Subject: Re: A question about changeset 20621:f9392f6eda79 and Discontinuous
>online node
>
>On 06/01/2010 09:07, "Jiang, Yunhong" <yunhong.jiang@xxxxxxxxx> wrote:
>
>>> In the case of the node_to_cpu list returned by xc.physinfo(), it already
>>> accounts for holes in node space by having empty lists of cpus for those
>>> non-existent or empty nodes. Should just be a case of having the xend python
>>> code understand that as necessary?
>>
>> Keir, thanks for pointing out this.The API is sufficient. We only need
>> changes
>> to python side to understand this. One minor thing is, maybe we need change
>> the nr_nodes from number of online node to the maxium of online node id,
>> otherwise, xc.physinfo may have no idea when iterate the node_to_memory
>> information (will it be possible the a node has only memory populated, while
>> no CPU populated?)
>
>Yes, I think actually that would be a bug fix, given how tools currently use
>the nr_nodes field. Feel free to send a patch as part of your fix-up work: I
>don't think even changing the field name is necessary, but we can add a
>comemnt to sysctl.h.
>
> -- Keir
>
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
Home