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

[Juergen Gross <juergen.gross@ts.fujitsu.com>]

On 11.09.2013 13:06, Dario Faggioli wrote:
> 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>

> ---
>   tools/libxl/xl_cmdimpl.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> 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;
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
>
>


-- 
Juergen Gross                 Principal Developer Operating Systems
PBG PDG ES&S SWE OS6                   Telephone: +49 (0) 89 3222 2967
Fujitsu Technology Solutions              e-mail: juergen.gross@ts.fujitsu.com
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html