Re: [Xen-devel] [PATCH v2 ]libxl: allow to allocate cpumap with specific size

[Dario Faggioli <raistlin@xxxxxxxx>]

On Fri, 2012-06-01 at 07:30 +0100, Ian Campbell wrote: 
> > > Currently, libxl_cpumap_alloc()allocate the cpumap with size of max_cpus. 
> > > In some place, we may want to allocate specific size of cpumap.
> > >
> > ... Do we? I fail to see this in the code below (you're only replacing
> > libxl_cpumap_alloc calls with new versions _always_ passing 0 as
> > max_cpus, am I right?)
> 
> The change to actually use this is is a followup patch "libxl: allow to
> set more than 31 vcpus".
> 
Yep, I saw that and commented on it too... If we go for this, I'd
suggest at least changing the changelog a bit and mention where/why
we're going to use it.

Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://retis.sssup.it/people/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
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel