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

Re: [Xen-devel] [PATCH 10/28] libxl: only free cpupoolinfo if necessary in libxl_list_cpupool



On Wed, Sep 18, 2013 at 8:06 PM, Dario Faggioli
<dario.faggioli@xxxxxxxxxx> wrote:
> I'm less confident about libxl_cpupoolinfo_list_* than about topology
> and numa info handling via libxl, but I suspect this is something
> similar to what I just said in my reply to 12/28 ("libxl: only free
> cputopology if it was allocated in libxl__get_numa_candidate").
>
> That is to say, this does not look necessary to me, as
> libxl_cpupoolinfo_list_free() seems to cope well with a non-allocated
> prt, and making it like this deviates from the usual libxl exit
> pattern/error handling style.
>
> Thoughts?

I think you're right, again ;) Please ignore this patch, too.

- Matthew

_______________________________________________
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®.