From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v2 ]libxl: allow to allocate cpumap with specific size Date: Fri, 1 Jun 2012 07:30:40 +0100 Message-ID: <1338532240.14877.40.camel@dagon.hellion.org.uk> References: <1338531971.31901.2.camel@Abyss> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1338531971.31901.2.camel@Abyss> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Dario Faggioli Cc: "Zhang, Yang Z" , "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org On Fri, 2012-06-01 at 07:26 +0100, Dario Faggioli wrote: > Hi all, > > Sorry, my fault I missed the v1 of this but... > > On Fri, 2012-06-01 at 02:41 +0000, Zhang, Yang Z 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". Ian.