Re: [Xen-devel][PATCH] libxc bitmap utils and vcpu-affinity

[Keir Fraser <keir.fraser@xxxxxxxxxxxxx>]

On 30/03/2010 17:05, "Dulloor" <dulloor@xxxxxxxxx> wrote:

>> It actually bloats the libraries by a net 650 LOC
>> (747 added, 87 deleted according to diffstat).
> In the patch, we have used the library only for vcpu get/set affinity.
> There are clearly other opportunities (right now and in future) to use
> most of the functions provided by the library, which will offset this.
> Also, this provides a cleaner/standard way of using the cpumap
> structure in libxc.

Clearly it's not a simplicity win right now as it net adds a lot of code.
I'd rather see this as part of a patch series that actually uses it more
substantially. And even then I'd bet that half of this patch could be
removed as unused. If NUMA changes end up manipulating cpumaps in, say, a
dozen places then I could see this approach being useful, instead of
pointless abstraction (how it appears currently).

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel