nohz: no softirq pending warnings for offline cpus
authorHeiko Carstens <heiko.carstens@de.ibm.com>
Thu, 11 Dec 2008 16:04:11 +0000 (17:04 +0100)
committerIngo Molnar <mingo@elte.hu>
Fri, 12 Dec 2008 06:27:01 +0000 (07:27 +0100)
Impact: remove false positive warning

After a cpu was taken down during cpu hotplug (read: disabled for interrupts)
it still might have pending softirqs. However take_cpu_down makes sure
that the idle task will run next instead of ksoftirqd on the taken down cpu.
The idle task will call tick_nohz_stop_sched_tick which might warn about
pending softirqs just before the cpu kills itself completely.

However the pending softirqs on the dead cpu aren't a problem because they
will be moved to an online cpu during CPU_DEAD handling.

So make sure we warn only for online cpus.

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
kernel/time/tick-sched.c

index 342fc9ccab46d5132aae1c58b1fbce0d02a2feaf..dc17ffcf1919007efecb6bbdfb6dfb2a1de0a392 100644 (file)
@@ -247,7 +247,7 @@ void tick_nohz_stop_sched_tick(int inidle)
        if (need_resched())
                goto end;
 
-       if (unlikely(local_softirq_pending())) {
+       if (unlikely(local_softirq_pending() && cpu_online(cpu))) {
                static int ratelimit;
 
                if (ratelimit < 10) {