Re: [PATCH] xen: sched: adjustments to some performance counters

[Juergen Gross <jgross@suse.com>]

On 09/25/2015 11:53 AM, Dario Faggioli wrote:
> More specifically:
>
> 1) rename vcpu_destroy to vcpu_remove
>
> It seems this have had to be done as part of 7e6b926a
> ("cpupools: Make interface more consistent"), which
> renamed the function but not the counter.
>
> In fact, because of cpupools, vcpus are not only removed
> from a scheduler when they are destroyed, but also when
> domains move between pools.
>
> Make the related statistics counter reflect that more
> properly.
>
> 2) rename vcpu_init to vcpu_alloc
>
> As it lives in *_alloc_vdata.
>
> 3) add vcpu_insert
>
> matching vcpu_remove, and useful to quickly check
> whether the number of insertions and removal matches,
> or in general investigare their relationship.
>
> Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
> ---
> Cc: George Dunlap <george.dunlap@eu.citrix.com>
> Cc: Juergen Gross <jgross@suse.com>
> Cc: Meng Xu <mengxu@cis.upenn.edu>
> Cc: Jan Beulich <jbeulich@suse.com>
> ---
> This would be v2 of "xen: sched: rename vcpu_destroy perf counter to
> vcpu_remove", but both the title and the content changed, so I'm treating it as
> a fresh submission (and not applying the Rev-by Juergen gave to it).

You can have it again:

Reviewed-by: Juergen Gross <jgross@suse.com>


Juergen