[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 Thu, Nov 14, 2013 at 7:11 AM, Dario Faggioli <dario.faggioli@xxxxxxxxxx> wrote: > 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; >> Thank you for reviewing. Will take into account the comments above. >> 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). Right, that was my idea of a linked list of memory ranges. > > So, if we can ask, with that in mind, what do you think the best ABI > would be? > > Thanks and Regards, > Dario > > -- > <<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) > -- Elena _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |