[Xen-changelog] [xen master] sched: always ask the scheduler to re-place the vcpu when the affinity changes

[patchbot@xxxxxxx]

commit 0f3b02d07e12b76c568ce82d7ec6bf80fdf3870f
Author:     George Dunlap <george.dunlap@xxxxxxxxxxxxx>
AuthorDate: Fri Mar 8 09:43:40 2013 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Fri Mar 8 09:43:40 2013 +0100

    sched: always ask the scheduler to re-place the vcpu when the affinity 
changes
    
    It's probably a good idea to re-evaluate placement whenever the
    affinity changes.
    
    This additionally has the benefit of removing scheduler-specific
    exceptions introduced in git c/s e6a6fd63.
    
    The conditionals surrounding vcpu_migrate() are left pending a re-work
    of the logic to avoid the common case calling vcpu_migrate() twice (once
    here, and once in context_saved().
    
    Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
---
 xen/common/schedule.c |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/xen/common/schedule.c b/xen/common/schedule.c
index 5f677c3..83fae4c 100644
--- a/xen/common/schedule.c
+++ b/xen/common/schedule.c
@@ -614,9 +614,10 @@ int vcpu_set_affinity(struct vcpu *v, const cpumask_t 
*affinity)
     vcpu_schedule_lock_irq(v);
 
     cpumask_copy(v->cpu_affinity, affinity);
-    if ( VCPU2OP(v)->sched_id == XEN_SCHEDULER_SEDF ||
-         !cpumask_test_cpu(v->processor, v->cpu_affinity) )
-        set_bit(_VPF_migrating, &v->pause_flags);
+
+    /* Always ask the scheduler to re-evaluate placement
+     * when changing the affinity */
+    set_bit(_VPF_migrating, &v->pause_flags);
 
     vcpu_schedule_unlock_irq(v);
 
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog