[Xen-changelog] [xen master] complete conversion set_bit() -> __cpumask_set_cpu() by 4aaca0e9cd

[patchbot@xxxxxxx]

commit 5dbdf33c57e3c95125b92f86d847ed8432e28f1c
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Fri Feb 27 16:09:27 2015 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Fri Feb 27 16:09:27 2015 +0100

    complete conversion set_bit() -> __cpumask_set_cpu() by 4aaca0e9cd
    
    While converting to __cpumask_set_cpu() was correct, the first argument
    passed should have been corrected to be "cpu" instead of "nr" at once.
    The wrong construct results in problems on systems with relatively few
    CPUs.
    
    Reported-by: Sander Eikelenboom <linux@xxxxxxxxxxxxxx>
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 xen/common/softirq.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/xen/common/softirq.c b/xen/common/softirq.c
index bb79aac..11e3345 100644
--- a/xen/common/softirq.c
+++ b/xen/common/softirq.c
@@ -106,7 +106,7 @@ void cpu_raise_softirq(unsigned int cpu, unsigned int nr)
     if ( !per_cpu(batching, this_cpu) || in_irq() )
         smp_send_event_check_cpu(cpu);
     else
-        __cpumask_set_cpu(nr, &per_cpu(batch_mask, this_cpu));
+        __cpumask_set_cpu(cpu, &per_cpu(batch_mask, this_cpu));
 }
 
 void cpu_raise_softirq_batch_begin(void)
--
generated by git-patchbot for /home/xen/git/xen.git#master

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