Re: [Xen-devel] [PATCH 2/4] xen: sched: factor code that moves a vcpu to a new pcpu in a function

[George Dunlap <George.Dunlap@xxxxxxxxxxxxx>]

On Fri, Jul 3, 2015 at 4:49 PM, Dario Faggioli
<dario.faggioli@xxxxxxxxxx> wrote:
> No functional change intended.
>
> Signed-off-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>

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

With one nit...

> ---
> Cc: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
> ---
>  xen/common/schedule.c |   48 +++++++++++++++++++++++++++++-------------------
>  1 file changed, 29 insertions(+), 19 deletions(-)
>
> diff --git a/xen/common/schedule.c b/xen/common/schedule.c
> index 26e8430..e83c666 100644
> --- a/xen/common/schedule.c
> +++ b/xen/common/schedule.c
> @@ -451,6 +451,34 @@ void vcpu_unblock(struct vcpu *v)
>      vcpu_wake(v);
>  }
>
> +/*
> + * Do the actual movemet of a vcpu from old to new CPU. Locks for *both*

"movement"  (missing an 'n')

 -G

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