mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Steven Noonan" <steven@uplinklabs.net>
To: "Luis R. Rodriguez" <lrodriguez@atheros.com>
Cc: linux-wireless <linux-wireless@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	"Senthilkumar Balasubramanian" 
	<Senthilkumar.Balasubramanian@atheros.com>,
	"ath9k-devel@lists.ath9k.org" <ath9k-devel@venema.h4ckr.net>,
	"Luis Rodriguez" <Luis.Rodriguez@atheros.com>,
	"Ingo Molnar" <mingo@elte.hu>
Subject: Re: [ath9k-devel] ath9k: massive unexplained latency in 2.6.27 (rc5, rc6, probably others)
Date: Fri, 19 Sep 2008 20:29:39 -0700	[thread overview]
Message-ID: <f488382f0809192029w601e0e72x5c3affc3dbe65118@mail.gmail.com> (raw)
In-Reply-To: <43e72e890809191813n95579f5h83e92b0ba1f95a44@mail.gmail.com>

On Fri, Sep 19, 2008 at 6:13 PM, Luis R. Rodriguez
<lrodriguez@atheros.com> wrote:
> On Fri, Sep 19, 2008 at 5:01 PM, Steven Noonan <steven@uplinklabs.net> wrote:
>> I'll pull wireless-testing and run that kernel for another 8 hours.
>> What's the URL for the repo? Surely you don't mean the
>> wireless-testing on git.kernel.org that hasn't been touched for almost
>> a week?
>
> Yes, that's the guy. This week was the Kernel Summit and the Linux
> Plumbers Conference too so there is understandably some lag in pulls
> as developers are getting drunk.
>
> git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-testing.git
>
>> I kind of wish there was a better debugging facility in ath9k.
>
> Patches are welcomed ;)
>
> Also, we have a few interrupt levels you can use. In this case we are
> debugging interrupts.
>
>> With
>> ATH_DBG_INTERRUPTS, I see several hundred messages per second, which
>> fills log files up like crazy (I switched to ftrace_printk as Ingo
>> suggested, which has alleviated this somewhat). Is there a way to make
>> it less verbose (perhaps so that it prints data at a rate of no more
>> than 2 or 3 messages per second)? Or maybe the rate at which it prints
>> this stuff is indicative something's wrong?
>
> No, interrupts do happen quite often so that is what you get, perhaps
> we can enhance this by not printing on some interrupts but the point
> is to debug interrupts after all. To help us debug in userspace in a
> nicer way we can use sadc and sar. So you can try this:
>
> On a window leave this running:
>
> /usr/lib/sysstat/sadc -I 1 irqlog.sar
>
> And when you are seeing your latency issues, run this:
>
> sar -I 17 -f irqlog.sar | less

Thanks for informing me about this useful utility. I wasn't aware of
its existence.

However, this profiler seems broken. When I check individual or 'ALL'
statistics, they're all 0.00 IRQs/sec, but SUM shows the actual number
of total IRQs per second. I suspect it's only broken because of some
kernel config option not being enabled.

Anyway, there definitely is an interrupt storm here. Let me demonstrate...

This is when the system is operating normally:
[...]
19:14:39          sum   1271.00
19:14:40          sum   1286.14
19:14:41          sum   1373.00
19:14:42          sum   1691.00
19:14:43          sum   1380.00
19:14:44          sum    902.00
19:14:45          sum    772.00
19:14:46          sum   1155.00
19:14:47          sum    804.00
19:14:48          sum    809.00
19:14:49          sum    965.00
19:14:50          sum   1077.23
Average:          sum    615.69


Here's while the latency issue is occurring, which can now safely be
called an interrupt storm:
[...]
20:21:07          sum  18864.08
20:21:08          sum  21152.73
20:21:09          sum  19221.35
20:21:10          sum  20786.14
20:21:11          sum  19891.92
20:21:12          sum  19816.16
20:21:13          sum  19430.69
20:21:14          sum  20432.00
20:21:15          sum  19657.84
20:21:16          sum  19386.00
20:21:17          sum  19386.79
20:21:18          sum  20152.94
20:21:19          sum  19077.88
20:21:20          sum  19852.00
Average:          sum  19958.91


And here's the change as I unload and reload ath9k:

[...]
20:21:38          sum  20138.83
20:21:40          sum  21576.09
20:21:41          sum  17678.38
20:21:42          sum  19834.34
20:21:43          sum  19845.10
20:21:44          sum  19470.00
20:21:45          sum   9273.00  <--- ath9k unloaded/reloaded
20:21:46          sum    187.00
20:21:47          sum   1066.00
20:21:48          sum    610.00
20:21:49          sum    106.93
20:21:50          sum    670.00
20:21:51          sum    742.00
20:21:52          sum    781.00
Average:          sum  19562.56

- Steven

  parent reply	other threads:[~2008-09-20  3:29 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-18 18:23 Steven Noonan
2008-09-18 18:34 ` [ath9k-devel] " Luis R. Rodriguez
2008-09-18 18:42   ` Luis R. Rodriguez
2008-09-18 19:00     ` Steven Noonan
2008-09-18 19:23       ` Steven Noonan
2008-09-18 20:18         ` Justin Mattock
2008-09-18 20:25           ` Steven Noonan
2008-09-18 20:44       ` Luis R. Rodriguez
2008-09-18 21:31         ` Justin Mattock
2008-09-18 21:49         ` Steven Noonan
2008-09-18 22:01           ` Luis R. Rodriguez
2008-09-18 22:08             ` Luis R. Rodriguez
2008-09-18 23:04               ` Steven Noonan
2008-09-18 23:10                 ` Luis R. Rodriguez
2008-09-18 23:16                   ` Johannes Berg
2008-09-19  2:52                   ` Steven Noonan
2008-09-19  3:01                     ` Luis R. Rodriguez
2008-09-19  7:29                       ` Steven Noonan
2008-09-19  8:17                         ` Ingo Molnar
2008-09-19  8:22                           ` Steven Noonan
2008-09-19  8:32                             ` Ingo Molnar
2008-09-27  1:23                               ` Steven Rostedt
2008-09-27 19:20                                 ` Ingo Molnar
2008-09-27 19:28                                   ` Steven Rostedt
2008-09-19  8:45                             ` Steven Noonan
2008-09-19  8:51                               ` Ingo Molnar
2008-09-19  8:22                         ` Steven Noonan
2008-09-19 14:28                         ` Senthil Balasubramanian
2008-09-19 16:42                           ` Steven Noonan
2008-09-19 17:58                             ` Senthil Balasubramanian
2008-09-19 18:23                               ` Luis R. Rodriguez
2008-09-19 23:31                                 ` Steven Noonan
2008-09-19 23:53                                   ` Luis R. Rodriguez
2008-09-20  0:01                                     ` Steven Noonan
2008-09-20  1:13                                       ` Luis R. Rodriguez
2008-09-20  1:15                                         ` Luis R. Rodriguez
2008-09-20  3:29                                         ` Steven Noonan [this message]
2008-09-20  3:40                                           ` Steven Noonan
2008-09-20 20:56                                             ` Luis R. Rodriguez
2008-09-20 15:21                                           ` Steven Noonan
2008-09-20 20:57                                             ` Luis R. Rodriguez
2008-09-20 20:58                                               ` Johannes Berg
2008-09-20 21:03                                                 ` Luis R. Rodriguez
2008-09-20 21:22                                                   ` Steven Noonan
2008-09-20 21:26                                                     ` Luis R. Rodriguez
2008-09-20 21:48                                                       ` Steven Noonan
2008-09-21  1:48                                                         ` Steven Noonan
2008-09-22  6:54                                                           ` Luis R. Rodriguez
2008-09-22  7:14                                                             ` Luis R. Rodriguez
2008-09-22  7:26                                                             ` Steven Noonan
2008-09-22  8:06                                                               ` Luis R. Rodriguez
2008-09-22 15:11                                                                 ` Steven Noonan
2008-09-22 16:01                                                                   ` Luis R. Rodriguez
2008-09-23  7:22                                                                     ` Steven Noonan
2008-09-23  7:55                                                                       ` Steven Noonan
2008-09-23 16:20                                                                         ` Steven Noonan
2008-09-23 19:22                                                                       ` Luis R. Rodriguez
2008-09-20 21:18                                               ` Steven Noonan
2008-09-20 21:24                                                 ` Luis R. Rodriguez
2008-09-18 23:08             ` Steven Noonan
2008-09-18 23:12               ` Luis R. Rodriguez
     [not found] <fa.jQQm4Iwb0lkAWxV+5FAfTMHY4m4@ifi.uio.no>
     [not found] ` <fa.zGA6t2p+jJeMRIaZFOqy1I8En1c@ifi.uio.no>
     [not found]   ` <fa.AGi3s2gpyC3T3KuCJ8cYQ30teak@ifi.uio.no>
     [not found]     ` <fa.59AlSLNBuMx/BjQSvR6whwGydbU@ifi.uio.no>
     [not found]       ` <fa.NwYDTAB59VZjr1quzgR4OU0+8eQ@ifi.uio.no>
     [not found]         ` <fa.tRre/FoeQKwChGwdusnYJgwjRmw@ifi.uio.no>
2008-09-19 14:46           ` Robert Hancock
2008-09-19 16:27             ` Steven Noonan

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=f488382f0809192029w601e0e72x5c3affc3dbe65118@mail.gmail.com \
    --to=steven@uplinklabs.net \
    --cc=Luis.Rodriguez@atheros.com \
    --cc=Senthilkumar.Balasubramanian@atheros.com \
    --cc=ath9k-devel@venema.h4ckr.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=lrodriguez@atheros.com \
    --cc=mingo@elte.hu \
    /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

all inboxes | Powered by JetHome®