Re: [PATCH 0/9] Some (not only) cpupool related fixes and improvements

[Ian Campbell <ian.campbell@citrix.com>]

On Fri, 2015-03-06 at 18:20 +0100, Dario Faggioli wrote:
> Hi everyone,
> 
> The main goal of this series is making it possible to specify ranges of pCPUs
> when manipulating (creating, adding/removing pCPUs) cpupools. Something like
> this:
> 
>  xl cpupool-cpu-remove Pool-node0 6-10
> 
> while, right now, only single pCPU IDs can be specified, which means that, to
> achieve the above, we need to do this:
> 
>  xl cpupool-cpu-remove Pool-node0 6
>  xl cpupool-cpu-remove Pool-node0 7
>  xl cpupool-cpu-remove Pool-node0 8
>  xl cpupool-cpu-remove Pool-node0 9
>  xl cpupool-cpu-remove Pool-node0 10
> 
> This is done in patches 5/9, 6/9 and 7/9.
> 
> The series also add a new parameter to `xl list', '-c' or '--cpupool', which
> shows in which cpupool the domain(s) lives. Such information was not easy to
> find in any other way. One can use `xl sched-<schedule_of_cpupool_x>' and have,
> as a side effect, something similar, but that's rather impractical.
> 
> Sample output for the new `xl list' can be found here:
> 
>  http://pastebin.com/qSagPimL
> 
> This happens in patch 3/9 and 4/9.
> 
> While there, the series includes some other, somewhat related, fixes of various
> kind.
> 
> This is available as a git branch here:
> 
>  git://xenbits.xen.org/people/dariof/xen.git rel/cpupools/allow-ranges-v1
>  http://xenbits.xen.org/gitweb/?p=people/dariof/xen.git;a=shortlog;h=refs/heads/rel/cpupools/allow-ranges-v1
> 
> 
> Thanks and Regards,
> Dario
> 
> ---
> Dario Faggioli (9):
>   1/9  docs: RTDS is a valid alternative as a scheduler for a cpupool
>   2/9  docs: fix `xl list' manpage entry

Acked + applied these two.

>   3/9  xl: turn some int local variable into bool

I'd have applied this with Wei's ack but he latter noticed that some 0/1
hadn't been converted so I thought I'd give you the chance to update if
you wanted.

For the rest I think there are outstanding comments to be addressed.