This patch removes an unneeded check in vcpu_migrate() for the picked cpu to be in the current cpupool. pick_cpu should only return cpus in the correct cpupool. Signed-off-by: juergen.gross@xxxxxxxxxxxxxx diff -r 3caed2112c65 xen/common/schedule.c --- a/xen/common/schedule.c Tue Mar 15 10:14:27 2011 +0000 +++ b/xen/common/schedule.c Wed Mar 16 07:26:38 2011 +0100 @@ -431,7 +431,7 @@ static void vcpu_migrate(struct vcpu *v) if ( old_lock == per_cpu(schedule_data, old_cpu).schedule_lock ) { /* - * If we selected a CPU on the previosu iteration, check if it + * If we selected a CPU on the previous iteration, check if it * remains suitable for running this vCPU. */ if ( pick_called && @@ -442,8 +442,7 @@ static void vcpu_migrate(struct vcpu *v) /* Select a new CPU. */ new_cpu = SCHED_OP(VCPU2OP(v), pick_cpu, v); - if ( (new_lock == per_cpu(schedule_data, new_cpu).schedule_lock) && - cpu_isset(new_cpu, v->domain->cpupool->cpu_valid) ) + if ( new_lock == per_cpu(schedule_data, new_cpu).schedule_lock ) break; pick_called = 1; }