mirror of
https://github.com/torvalds/linux.git
synced 2024-12-13 06:32:50 +00:00
398812159e
This fixes the same problem as described in the patch "nohz: fix
printk_needs_cpu() return value on offline cpus" for the arch_needs_cpu()
primitive:
arch_needs_cpu() may return 1 if called on offline cpus. When a cpu gets
offlined it schedules the idle process which, before killing its own cpu,
will call tick_nohz_stop_sched_tick().
That function in turn will call arch_needs_cpu() in order to check if the
local tick can be disabled. On offline cpus this function should naturally
return 0 since regardless if the tick gets disabled or not the cpu will be
dead short after. That is besides the fact that __cpu_disable() should already
have made sure that no interrupts on the offlined cpu will be delivered anyway.
In this case it prevents tick_nohz_stop_sched_tick() to call
select_nohz_load_balancer(). No idea if that really is a problem. However what
made me debug this is that on 2.6.32 the function get_nohz_load_balancer() is
used within __mod_timer() to select a cpu on which a timer gets enqueued.
If arch_needs_cpu() returns 1 then the nohz_load_balancer cpu doesn't get
updated when a cpu gets offlined. It may contain the cpu number of an offline
cpu. In turn timers get enqueued on an offline cpu and not very surprisingly
they never expire and cause system hangs.
This has been observed 2.6.32 kernels. On current kernels __mod_timer() uses
get_nohz_timer_target() which doesn't have that problem. However there might
be other problems because of the too early exit tick_nohz_stop_sched_tick()
in case a cpu goes offline.
This specific bug was indrocuded with
|
||
|---|---|---|
| .. | ||
| vdso32 | ||
| vdso64 | ||
| asm-offsets.c | ||
| audit.c | ||
| audit.h | ||
| base.S | ||
| bitmap.c | ||
| compat_audit.c | ||
| compat_exec_domain.c | ||
| compat_linux.c | ||
| compat_linux.h | ||
| compat_ptrace.h | ||
| compat_signal.c | ||
| compat_wrapper.S | ||
| cpcmd.c | ||
| crash.c | ||
| debug.c | ||
| diag.c | ||
| dis.c | ||
| early.c | ||
| ebcdic.c | ||
| entry64.S | ||
| entry.h | ||
| entry.S | ||
| ftrace.c | ||
| head31.S | ||
| head64.S | ||
| head.S | ||
| init_task.c | ||
| ipl.c | ||
| irq.c | ||
| kprobes.c | ||
| machine_kexec.c | ||
| Makefile | ||
| mcount64.S | ||
| mcount.S | ||
| mem_detect.c | ||
| module.c | ||
| nmi.c | ||
| process.c | ||
| processor.c | ||
| ptrace.c | ||
| reipl64.S | ||
| reipl.S | ||
| relocate_kernel64.S | ||
| relocate_kernel.S | ||
| s390_ext.c | ||
| s390_ksyms.c | ||
| sclp.S | ||
| setup.c | ||
| signal.c | ||
| smp.c | ||
| stacktrace.c | ||
| suspend.c | ||
| switch_cpu64.S | ||
| switch_cpu.S | ||
| swsusp_asm64.S | ||
| sys_s390.c | ||
| syscalls.S | ||
| sysinfo.c | ||
| time.c | ||
| topology.c | ||
| traps.c | ||
| vdso.c | ||
| vmlinux.lds.S | ||
| vtime.c | ||