mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Andrew Morton <akpm@osdl.org>
Cc: Dave Jones <davej@redhat.com>, Adrian Bunk <bunk@stusta.de>,
	linux-kernel@vger.kernel.org,
	Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [patch] debugging feature: SysRq-Q to print timers
Date: Tue, 19 Dec 2006 13:01:50 +0100	[thread overview]
Message-ID: <20061219120150.GA27819@elte.hu> (raw)
In-Reply-To: <20061218160019.988171f5.akpm@osdl.org>


* Andrew Morton <akpm@osdl.org> wrote:

> > /proc/timer-list is useful for profiling applications doing 
> > excessive wakeups. With the move towards being tickless, this is 
> > more important than ever, and giving users the right tools to find 
> > these problems themselves is important.
> > 
> 
> oic.  Nobody ever tells me squat.  <updates changelog>
> 
> Your explanation doesn't explain why we need this info in a sysrq 
> triggerable form.
> 
> And what about /proc/timer-stat?

/proc/timer_stats does timer profiling. You start it via:

	echo 1 > /proc/timer_stats

and then the profile info gathers into /proc/timer_stats. Useful way to 
look at it is:

	sort -n /proc/timer_stats

for example:

Timer Stats Version: v0.1
   1,     0 swapper          page_writeback_init (wb_timer_fn)
   1,  1898 modprobe         neigh_table_init_no_netlink (neigh_periodic_timer)
   1,     1 init             schedule_timeout (process_timeout)
   1,     1 swapper          neigh_table_init_no_netlink (neigh_periodic_timer)
   1,  2700 hald-addon-stor  do_nanosleep (hrtimer_wakeup)
   1,     6 softirq-timer/0  __netdev_watchdog_up (dev_watchdog)
   2,     1 swapper          queue_delayed_work_on (delayed_work_timer_fn)
   2,     1 swapper          queue_delayed_work_on (delayed_work_timer_fn)
   2,   480 IRQ 218          e1000_intr_msi (e1000_watchdog)
   3,  2652 yum-updatesd     schedule_timeout (process_timeout)
   4,  2472 pcscd            do_nanosleep (hrtimer_wakeup)
   4,  7824 sshd             sk_reset_timer (tcp_write_timer)
  13,   428 insmod           usb_hcd_poll_rh_status (rh_timer_func)
  13,   428 insmod           usb_hcd_poll_rh_status (rh_timer_func)
  13,   437 insmod           usb_hcd_submit_urb (rh_timer_func)
  19,    21 softirq-net-rx/  sk_reset_timer (tcp_delack_timer)
 164,  1868 kondemand/0      queue_delayed_work_on (delayed_work_timer_fn)
 164,  1869 kondemand/1      queue_delayed_work_on (delayed_work_timer_fn)
 282,     0 swapper          hrtimer_stop_sched_tick (hrtimer_sched_tick)
 321,     0 swapper          hrtimer_stop_sched_tick (hrtimer_sched_tick)
 335,     0 swapper          hrtimer_restart_sched_tick (hrtimer_sched_tick)
 408,     0 swapper          hrtimer_restart_sched_tick (hrtimer_sched_tick)
1755 total events, 585.534 events/sec

this shows us that the kondemand kernel threads are causing 90% of the 
timeout events on this system.

/proc/timer_list shows all currently pending timers, and all the state 
of the hardware timers. That is a bit different from timer_stat but 
still very useful: it gives us a snapshot into the current state of the 
(hr)timer subsystem. I needed it to debug a couple of high-res timers 
subsystem bugs. The SysRq trigger was useful for things like timer 
related boot hangs. (It's also useful to catch excessive waiters during 
bootup.)

	Ingo

  reply	other threads:[~2006-12-19 12:04 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-15  6:59 2.6.20-rc1-mm1 Andrew Morton
2006-12-15  6:59 ` BUG: NMI Watchdog detected LOCKUP (was: 2.6.20-rc1-mm1) Tilman Schmidt
2006-12-19 18:52   ` Thomas Gleixner
2006-12-19 19:56   ` [patch] hrtimers: add state tracking, fix Ingo Molnar
2006-12-20  1:38     ` Tilman Schmidt
2006-12-20 20:05     ` Tilman Schmidt
2006-12-15 14:46 ` OOPS: deref 0x14 at pdc_port_start+0x82 [Was: 2.6.20-rc1-mm1] Jiri Slaby
2006-12-15 19:24   ` Andrew Morton
2006-12-15 22:49     ` Jiri Slaby
2006-12-15 20:39 ` 2.6.20-rc1-mm1 Damien Wyart
2006-12-15 21:01   ` 2.6.20-rc1-mm1 Andrew Morton
2006-12-17 11:07     ` 2.6.20-rc1-mm1 Damien Wyart
2006-12-18  8:03       ` 2.6.20-rc1-mm1 Laurent Riffard
2006-12-18 18:35         ` 2.6.20-rc1-mm1 Damien Wyart
2006-12-19 23:29           ` 2.6.20-rc1-mm1 Luben Tuikov
2006-12-18  7:44     ` 2.6.20-rc1-mm1 Jens Axboe
2006-12-15 23:26 ` WARNING (1) at .../arch/i386/mm/highmem.c:49 [Was: 2.6.20-rc1-mm1] Jiri Slaby
2006-12-16  0:16   ` Andrew Morton
2006-12-16  0:04 ` 2.6.20-rc1-mm1: unused sysrq_timer_list_show() Adrian Bunk
2006-12-16  7:56   ` [patch] debugging feature: SysRq-Q to print timers Ingo Molnar
2006-12-18 23:31     ` Andrew Morton
2006-12-18 23:45       ` Dave Jones
2006-12-19  0:00         ` Andrew Morton
2006-12-19 12:01           ` Ingo Molnar [this message]
2006-12-16 12:37 ` (Cross) compiling fails on first try (was Re: 2.6.20-rc1-mm1) Jan Dittmer
2006-12-16 13:56 ` [-mm patch] drivers/ide/pci/tc86c001.c: make a function static Adrian Bunk
2006-12-16 14:18   ` Alan
2006-12-17 18:08   ` Sergei Shtylyov
2006-12-17 20:52   ` Sergei Shtylyov
2006-12-16 13:56 ` [-mm patch] make uio_irq_handler() static Adrian Bunk
2006-12-20  6:09   ` Greg KH
2006-12-16 13:56 ` [-mm patch] drivers/video/{s3fb,svgalib}.c: possible cleanups Adrian Bunk
2006-12-16 17:39   ` Randy Dunlap
2006-12-16 18:16   ` Ondrej Zajicek
2006-12-16 13:57 ` [-mm patch] mm/vmscan.c: make a function static Adrian Bunk
2006-12-16 19:30 ` [-mm patch] noinitramfs cleanup Frederik Deweerdt
2006-12-18 13:38 ` [-mm patch] kill pxa2xx Kconfig warning Frederik Deweerdt
2006-12-18 20:06 ` 2.6.20-rc1-mm1 Bartlomiej Zolnierkiewicz
2006-12-19  0:29 ` 2.6.20-rc1-mm1 Randy Dunlap
2006-12-19  0:42   ` 2.6.20-rc1-mm1 Andrew Morton
2006-12-21 18:35 ` [-mm patch] ptrace: make {put,get}reg work again for gs and fs Frederik Deweerdt
2006-12-21 19:22   ` Jeremy Fitzhardinge
2006-12-21 20:53     ` Frederik Deweerdt
2006-12-21 21:59     ` Frederik Deweerdt
2006-12-22  2:00       ` Jeremy Fitzhardinge
     [not found]         ` <20061221181108.6cede9ba.akpm@osdl.org>
2006-12-22  6:06           ` Frederik Deweerdt
     [not found]             ` <20061221225414.de09c7df.akpm@osdl.org>
2006-12-22  7:00               ` Jeremy Fitzhardinge
2006-12-22  8:05               ` Frederik Deweerdt
2006-12-22  6:52           ` Jeremy Fitzhardinge
2006-12-22  6:55       ` Jeremy Fitzhardinge

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20061219120150.GA27819@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@osdl.org \
    --cc=bunk@stusta.de \
    --cc=davej@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome