• Ingo Molnar's avatar
    softlockup watchdog fixes and cleanups · 00aceb50
    Ingo Molnar authored
    This is a merge of commits a5f2ce3c and
    43581a10 in mainline to fix a warning in
    the 2.6.23.3 kernel release.
    
    softlockup watchdog: style cleanups
    
    kernel/softirq.c grew a few style uncleanlinesses in the past few
    months, clean that up. No functional changes:
    
    text    data     bss     dec     hex filename
    1126      76       4    1206     4b6 softlockup.o.before
    1129      76       4    1209     4b9 softlockup.o.after
    
    ( the 3 bytes .text increase is due to the "<1>" appended to one of
    the printk messages. )
    Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
    
    
    softlockup: improve debug output
    
    Improve the debuggability of kernel lockups by enhancing the debug
    output of the softlockup detector: print the task that causes the lockup
    and try to print a more intelligent backtrace.
    
    The old format was:
    
    BUG: soft lockup detected on CPU#1!
    [<c0105e4a>] show_trace_log_lvl+0x19/0x2e
    [<c0105f43>] show_trace+0x12/0x14
    [<c0105f59>] dump_stack+0x14/0x16
    [<c015f6bc>] softlockup_tick+0xbe/0xd0
    [<c013457d>] run_local_timers+0x12/0x14
    [<c01346b8>] update_process_times+0x3e/0x63
    [<c0145fb8>] tick_sched_timer+0x7c/0xc0
    [<c0140a75>] hrtimer_interrupt+0x135/0x1ba
    [<c011bde7>] smp_apic_timer_interrupt+0x6e/0x80
    [<c0105aa3>] apic_timer_interrupt+0x33/0x38
    [<c0104f8a>] syscall_call+0x7/0xb
    =======================
    
    The new format is:
    
    BUG: soft lockup detected on CPU#1! [prctl:2363]
    
    Pid: 2363, comm:                prctl
    EIP: 0060:[<c013915f>] CPU: 1
    EIP is at sys_prctl+0x24/0x18c
    EFLAGS: 00000213    Not tainted  (2.6.22-cfs-v20 #26)
    EAX: 00000001 EBX: 000003e7 ECX: 00000001 EDX: f6df0000
    ESI: 000003e7 EDI: 000003e7 EBP: f6df0fb0 DS: 007b ES: 007b FS: 00d8
    CR0: 8005003b CR2: 4d8c3340 CR3: 3731d000 CR4: 000006d0
    [<c0105e4a>] show_trace_log_lvl+0x19/0x2e
    [<c0105f43>] show_trace+0x12/0x14
    [<c01040be>] show_regs+0x1ab/0x1b3
    [<c015f807>] softlockup_tick+0xef/0x108
    [<c013457d>] run_local_timers+0x12/0x14
    [<c01346b8>] update_process_times+0x3e/0x63
    [<c0145fcc>] tick_sched_timer+0x7c/0xc0
    [<c0140a89>] hrtimer_interrupt+0x135/0x1ba
    [<c011bde7>] smp_apic_timer_interrupt+0x6e/0x80
    [<c0105aa3>] apic_timer_interrupt+0x33/0x38
    [<c0104f8a>] syscall_call+0x7/0xb
    =======================
    
    Note that in the old format we only knew that some system call locked
    up, we didnt know _which_. With the new format we know that it's at a
    specific place in sys_prctl(). [which was where i created an artificial
    kernel lockup to test the new format.]
    
    This is also useful if the lockup happens in user-space - the user-space
    EIP (and other registers) will be printed too. (such a lockup would
    either suggest that the task was running at SCHED_FIFO:99 and looping
    for more than 10 seconds, or that the softlockup detector has a
    false-positive.)
    
    The task name is printed too first, just in case we dont manage to print
    a useful backtrace.
    
    [satyam@infradead.org: fix warning]
    Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
    Signed-off-by: default avatarSatyam Sharma <satyam@infradead.org>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
    Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
    
    00aceb50
softlockup.c 5.02 KB