Re: [Xen-devel] [PATCH] xen: sched: get rid of cpupool_scheduler_cpumask()

[George Dunlap <george.dunlap@xxxxxxxxxx>]

On 24/09/15 04:40, Dario Faggioli wrote:
> and of (almost every) direct use of cpupool_online_cpumask().
> 
> In fact, what we really want for the most of the times,
> is the set of valid pCPUs of the cpupool a certain domain
> is part of. Furthermore, in case it's called with a NULL
> pool as argument, cpupool_scheduler_cpumask() does more
> harm than good, by returning the bitmask of free pCPUs!
> 
> This commit, therefore:
>  * gets rid of cpupool_scheduler_cpumask(), in favour of
>    cpupool_domain_cpumask(), which makes it more evident
>    what we are after, and accommodates some sanity checking;
>  * replaces some of the calls to cpupool_online_cpumask()
>    with calls to the new functions too.
> 
> Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> Acked-by: Juergen Gross <jgross@xxxxxxxx>
> Acked-by: Joshua Whitehead <josh.whitehead@xxxxxxxxxxxxxxx>
> Reviewed-by: Meng Xu <mengxu@xxxxxxxxxxxxx>

Right, sorry about not getting back to the new version.  Looks good:

Reviewed-by: George Dunlap <george.dunlap@xxxxxxxxxx>



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel