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

Re: [Xen-devel] [PATCH v2 1/7] xen: vNUMA support for guests.

On gio, 2013-11-14 at 11:48 +0000, David Vrabel wrote:
> On 14/11/13 03:26, Elena Ufimtseva wrote:
> > +/* 
> > + * Memory ranges can be used to define
> > + * vNUMA memory node boundaries by the 
> > + * linked list. As of now, only one range
> > + * per domain is suported.
> > + */
> > +
> > +struct vmemrange {
> > +    uint64_t start, end;
> > +    struct vmemrange *next;
> I think this probably wants to be an index into the vmemrange array in
> struct vnuma_topology_info.  It certainly cannot be a bare pointer like
> this.
I think Elena is aiming at a liked list, and so there really is no such
thing as a vmemrange array. This is (probably) because right now only
one memory range is supported, and thus it sounds a bit too much to have
an array there already (e.g., how big?), but at the same time she wanted
to be sure to leave room for future extensions (namely, multiple memory
ranges in each node).

So, if we can ask, with that in mind, what do you think the best ABI
would be?

Thanks and Regards,

<<This happens because I choose it to happen!>> (Raistlin Majere)
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

Xen-devel mailing list



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