[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH v2 7/7] xl: use libxl_cpupoolinfo_list_free() in main_cpupoolnumasplit
instead of manually freeing the elements of the list, which is exactly the purpose of the said function. Trade also a couple of 'return'-s with 'goto out'-s, which is more in line with libxl usage paradigm. Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Cc: Ian Campbell <ian.campbell@xxxxxxxxxx> Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> Cc: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx> Cc: Wei Liu <wei.liu2@xxxxxxxxxx> Cc: Juergen Gross <JGross@xxxxxxxx> Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx> --- Changes from v1: * grammar fixes in the changelog, as requested during review. --- tools/libxl/xl_cmdimpl.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index 8587cc2..efb6024 100644 --- a/tools/libxl/xl_cmdimpl.c +++ b/tools/libxl/xl_cmdimpl.c @@ -7460,9 +7460,8 @@ int main_cpupoolnumasplit(int argc, char **argv) } poolid = poolinfo[0].poolid; sched = poolinfo[0].sched; - for (p = 0; p < n_pools; p++) { - libxl_cpupoolinfo_dispose(poolinfo + p); - } + libxl_cpupoolinfo_list_free(poolinfo, n_pools); + if (n_pools > 1) { fprintf(stderr, "splitting not possible, already cpupools in use\n"); return 1; @@ -7476,8 +7475,7 @@ int main_cpupoolnumasplit(int argc, char **argv) if (libxl_cpu_bitmap_alloc(ctx, &cpumap, 0)) { fprintf(stderr, "Failed to allocate cpumap\n"); - libxl_cputopology_list_free(topology, n_cpus); - return 1; + goto out; } /* Reset Pool-0 to 1st node: first add cpus, then remove cpus to avoid @@ -7486,7 +7484,7 @@ int main_cpupoolnumasplit(int argc, char **argv) node = topology[0].node; if (libxl_cpupool_cpuadd_node(ctx, 0, node, &n)) { fprintf(stderr, "error on adding cpu to Pool 0\n"); - return 1; + goto out; } snprintf(name, 15, "Pool-node%d", node); _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |