From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH] libxc bitmap utils and vcpu-affinity Date: Tue, 30 Mar 2010 17:27:54 +0100 Message-ID: References: <940bcfd21003300905y78754e2cu101055e429a303a1@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <940bcfd21003300905y78754e2cu101055e429a303a1@mail.gmail.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Dulloor Cc: "xen-devel@lists.xensource.com" , Jan Beulich List-Id: xen-devel@lists.xenproject.org On 30/03/2010 17:05, "Dulloor" 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