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
next prev 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®