[Xen-changelog] [xen master] xl: fix `xl cpupool-list' behavior in case no pool name is provided

[patchbot@xxxxxxx]

commit 3998afdbf99959582dcd9f9f4df5a6fe7ce4ded8
Author:     Dario Faggioli <dario.faggioli@xxxxxxxxxx>
AuthorDate: Wed Sep 11 13:06:22 2013 +0200
Commit:     Ian Campbell <ian.campbell@xxxxxxxxxx>
CommitDate: Fri Sep 13 12:50:39 2013 +0100

    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@xxxxxxxxxx>
    Acked-by: Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>
---
 tools/libxl/xl_cmdimpl.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
index 884f050..35b3e29 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;
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog