[Xen-changelog] [xen stable-4.4] Revert "cpupools: update domU's node-affinity on the cpupool_unassign_cpu() path"

[patchbot@xxxxxxx]

commit 77da6c2de65de3b00e5181cd26a3869744555ea4
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Thu Mar 26 08:41:03 2015 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Thu Mar 26 08:41:03 2015 +0100

    Revert "cpupools: update domU's node-affinity on the cpupool_unassign_cpu() 
path"
    
    This reverts commit 93be8285a79c6cbbf66c8681fec1d1bfb71d84cc.
    
    At the point this patch calls domain_update_node_affinity(), the vcpu
    hard affinities have not yet been updated; so calling it at this point
    can in some circumstances trigger an ASSERT().
    
    domain_update_node_affinity() is already called in
    cpu_disable_scheduler(), so adding it to cpupool_unassign_cpu() is
    redundant.  Simply reverting the patch is sufficient.
    
    Acked-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
---
 xen/common/cpupool.c |    6 ------
 1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/xen/common/cpupool.c b/xen/common/cpupool.c
index 53a1394..2c283b9 100644
--- a/xen/common/cpupool.c
+++ b/xen/common/cpupool.c
@@ -379,12 +379,6 @@ int cpupool_unassign_cpu(struct cpupool *c, unsigned int 
cpu)
     atomic_inc(&c->refcnt);
     cpupool_cpu_moving = c;
     cpumask_clear_cpu(cpu, c->cpu_valid);
-
-    rcu_read_lock(&domlist_read_lock);
-    for_each_domain_in_cpupool(d, c)
-        domain_update_node_affinity(d);
-    rcu_read_unlock(&domlist_read_lock);
-
     spin_unlock(&cpupool_lock);
 
     work_cpu = smp_processor_id();
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.4

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