From: Dario Faggioli <dario.faggioli@citrix.com>
Date: Wed, 11 Sep 2013 11:06:22 +0000 (+0200)
Subject: xl: fix `xl cpupool-list' behavior in case no pool name is provided
X-Git-Tag: 4.4.0-rc1~632^2~17
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=3998afdbf99959582dcd9f9f4df5a6fe7ce4ded8;p=xen.git

xl: fix `xl cpupool-list' behavior in case no pool name is provided

since it errors out, asking for at least one argument, and does
not display any useful output, which is wrong (we want the list
and the info about all the existing cpupools).

IOW, the output is as follows:

 ~# xl cpupool-list -c
 'xl cpupool-list' requires at least 1 argument.
 ...

While it should be as follows:

 ~# xl cpupool-list -c
 Name               CPU list
 Pool-0             0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15

Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
Acked-by: Juergen Gross <juergen.gross@ts.fujitsu.com>
---

diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
index 884f050644..35b3e290ff 100644
--- a/tools/libxl/xl_cmdimpl.c
+++ b/tools/libxl/xl_cmdimpl.c
@@ -6570,7 +6570,7 @@ int main_cpupoollist(int argc, char **argv)
     char *name;
     int ret = 0;
 
-    SWITCH_FOREACH_OPT(opt, "hc", opts, "cpupool-list", 1) {
+    SWITCH_FOREACH_OPT(opt, "hc", opts, "cpupool-list", 0) {
     case 'c':
         opt_cpus = 1;
         break;