[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH 9/9] xl: use libxl_cpupoolinfo_list_free() in main_cpupoolnumasplit
instead manually free 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> --- 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 14b5a3d..ec2b6a9 100644 --- a/tools/libxl/xl_cmdimpl.c +++ b/tools/libxl/xl_cmdimpl.c @@ -7477,9 +7477,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; @@ -7493,8 +7492,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 @@ -7503,7 +7501,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 |