From: "Steven Noonan" <steven@uplinklabs.net>
To: "Luis R. Rodriguez" <lrodriguez@atheros.com>
Cc: "Luis Rodriguez" <Luis.Rodriguez@atheros.com>,
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>,
"Johannes Berg" <johannes@sipsolutions.net>,
"Ingo Molnar" <mingo@elte.hu>
Subject: Re: [ath9k-devel] ath9k: massive unexplained latency in 2.6.27 (rc5, rc6, probably others)
Date: Mon, 22 Sep 2008 08:11:30 -0700 [thread overview]
Message-ID: <f488382f0809220811y46e9daf3r5db5a7c76df87800@mail.gmail.com> (raw)
In-Reply-To: <20080922080632.GA20446@tesla>
On Mon, Sep 22, 2008 at 1:06 AM, Luis R. Rodriguez
<lrodriguez@atheros.com> wrote:
> Please test the following patch.
>
> diff --git a/drivers/net/wireless/ath9k/core.c b/drivers/net/wireless/ath9k/core.c
> index c262ef2..9a51739 100644
> --- a/drivers/net/wireless/ath9k/core.c
> +++ b/drivers/net/wireless/ath9k/core.c
> @@ -681,10 +681,12 @@ int ath_open(struct ath_softc *sc, struct ath9k_channel *initial_chan)
> * Enable MIB interrupts when there are hardware phy counters.
> * Note we only do this (at the moment) for station mode.
> */
> +#if 0
> if (ath9k_hw_phycounters(ah) &&
> ((sc->sc_ah->ah_opmode == ATH9K_M_STA) ||
> (sc->sc_ah->ah_opmode == ATH9K_M_IBSS)))
> sc->sc_imask |= ATH9K_INT_MIB;
> +#endif
> /*
> * Some hardware processes the TIM IE and fires an
> * interrupt when the TIM bit is set. For hardware
>
It didn't apply to -rc7, but I managed to apply it manually
(apparently you made this for wireless-testing?) and added the #if 0.
After 7 hours uptime with the driver, no apparent interrupt storm.
I'll do more thorough testing later, but I have a 3 hour drive today,
and some packing to do, so it'll need to wait a bit.
- Steven
next prev parent reply other threads:[~2008-09-22 15:11 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
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 [this message]
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=f488382f0809220811y46e9daf3r5db5a7c76df87800@mail.gmail.com \
--to=steven@uplinklabs.net \
--cc=Luis.Rodriguez@atheros.com \
--cc=Senthilkumar.Balasubramanian@atheros.com \
--cc=ath9k-devel@venema.h4ckr.net \
--cc=johannes@sipsolutions.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®