rcu: Make rcutorture test for hotpluggability before offlining CPUs
authorPaul E. McKenney <paul.mckenney@linaro.org>
Sat, 3 Dec 2011 23:09:28 +0000 (15:09 -0800)
committerPaul E. McKenney <paulmck@linux.vnet.ibm.com>
Sun, 11 Dec 2011 18:32:21 +0000 (10:32 -0800)
The rcutorture test now can automatically exercise CPU hotplug and
collect success statistics, which can be correlated with other rcutorture
activity.  This permits rcutorture to completely exercise RCU regardless
of what sort of userspace and filesystem layout is in use.  Unfortunately,
rcutorture is happy to attempt to offline CPUs that cannot be offlined,
for example, CPU 0 in both the x86 and ARM architectures.  Although this
allows rcutorture testing to proceed normally, it confounds attempts at
error analysis due to the resulting flood of spurious CPU-hotplug errors.

Therefore, this commit uses the new cpu_is_hotpluggable() function to
avoid attempting to offline CPUs that are not hotpluggable, which in
turn avoids spurious CPU-hotplug errors.

Signed-off-by: Paul E. McKenney <paul.mckenney@linaro.org>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
kernel/rcutorture.c

index 1e422ae1506b3a7ee0852f2f70b8abc6a1a59958..186ead9fa34883b930efbf69e672881bcacd99c2 100644 (file)
@@ -1388,7 +1388,7 @@ rcu_torture_onoff(void *arg)
        WARN_ON(maxcpu < 0);
        while (!kthread_should_stop()) {
                cpu = (rcu_random(&rand) >> 4) % (maxcpu + 1);
-               if (cpu_online(cpu)) {
+               if (cpu_online(cpu) && cpu_is_hotpluggable(cpu)) {
                        if (verbose)
                                printk(KERN_ALERT "%s" TORTURE_FLAG
                                       "rcu_torture_onoff task: offlining %d\n",
@@ -1402,7 +1402,7 @@ rcu_torture_onoff(void *arg)
                                               torture_type, cpu);
                                n_offline_successes++;
                        }
-               } else {
+               } else if (cpu_is_hotpluggable(cpu)) {
                        if (verbose)
                                printk(KERN_ALERT "%s" TORTURE_FLAG
                                       "rcu_torture_onoff task: onlining %d\n",