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

Re: [Xen-devel] [PATCH v8 10/13] libxl/xl: deprecate the build_info->cpumap field



On ven, 2014-06-13 at 14:38 +0100, Ian Campbell wrote:
> On Fri, 2014-06-13 at 14:34 +0100, Wei Liu wrote:
> > > @@ -261,8 +261,6 @@ int libxl__build_pre(libxl__gc *gc, uint32_t domid,
> > >              return rc;
> > >      }
> > >      libxl_domain_set_nodeaffinity(ctx, domid, &info->nodemap);
> > > -    libxl_set_vcpuaffinity_all(ctx, domid, info->max_vcpus,
> > > -                               &info->cpumap, NULL);
> > >  
> > 
> > So you actually remove cpumap handling logic in libxl as well? IMHO this
> > is not right.
> 
> Correct.
> 
> Old apps which are using the deprecated field must continue to work,
> which means handling the deprecated field internally.
> 
Right. Fixing it for v9 (which I'm waiting a bit more to post, to try
gathering some more comments, if any).

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)

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

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

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