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

Re: [Xen-devel] [PATCH 04 of 11] libxl: expand the libxl_{cpu, node}map API a bit



On Thu, 2012-05-31 at 15:13 +0100, Ian Jackson wrote:
> Dario Faggioli writes ("[PATCH 04 of 11] libxl: expand the 
> libxl_{cpu,node}map API a bit"):
> > +void libxl_cpumap_copy(/*XXX libxl_ctx *ctx,*/ libxl_cpumap *dst,
> 
> XXX ?
> 
Yep, I left it on purpose but I also wanted to mention that in the
changelog, which I apparently missed. Sorry.

However, now that we are here, the whole point is, does that function
need a libxl_ctx argument? It doesn't use it, but I was concerned about
consistency with the other libxl_cpumap_* calls, and maybe future
needs...

Thanks and Regards,
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

 


Rackspace

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