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: "Ingo Molnar" <mingo@elte.hu>,
	"ath9k-devel@lists.ath9k.org" <ath9k-devel@venema.h4ckr.net>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [ath9k-devel] ath9k: massive unexplained latency in 2.6.27 (rc5, rc6, probably others)
Date: Thu, 18 Sep 2008 14:49:45 -0700	[thread overview]
Message-ID: <f488382f0809181449x7343aa7endff066cb5836a73c@mail.gmail.com> (raw)
In-Reply-To: <43e72e890809181344q416b5944w3332ee5a33db048c@mail.gmail.com>

On Thu, Sep 18, 2008 at 1:44 PM, Luis R. Rodriguez
<lrodriguez@atheros.com> wrote:
> On Thu, Sep 18, 2008 at 12:00 PM, Steven Noonan <steven@uplinklabs.net> wrote:
>> On Thu, Sep 18, 2008 at 11:42 AM, Luis R. Rodriguez
>> <lrodriguez@atheros.com> wrote:
>>> On Thu, Sep 18, 2008 at 11:34 AM, Luis R. Rodriguez
>>>> irqpoll is a monster of evil and that should make your system crawl to
>>>> its knees. I would advise instead we work with you fixing the the
>>>> missed interrupts issue upon rmmod.
>>>
>>> Also, please provide the output of
>>>
>>> cat /proc/interrupts
>>
>> Note that the problem necessitating use of irqpoll in the first place
>> seems to only happen under certain conditions. I am unsure what these
>> conditions are. Before 'ath9k: connectivity is lost after Group
>> rekeying is done',
>
> You mean this patch:
>
> [PATCH] ath9k: connectivity is lost after Group rekeying is done
> http://marc.info/?l=linux-wireless&m=122163541519736&w=2
>
> So let me get this straight -- you applied this new patch, and haven't
> tried disabling irqpoll now?

I hadn't at the time of that writing, no. I saw it as a fix for future
ignored IRQs, and hadn't noticed any difference with it on or off. So
it seemed like since there was no consequence having it enabled, why
not leave it enabled all the time?

Now that I'm aware it's the spawn of satan, I'm trying with it off. So
far, so good. But I haven't had need to reload ath9k so frequently,
and even when I do, I can't reproduce the specific conditions which
caused the problem in the first place.

>> I had used rmmod/modprobe as my solution to the
>> issue, which triggered the IRQ issue.
>
> Understood, but I also have used this before with ath9k and I got
> exactly the same results you did -- I just refused to use it again and
> just try to fix the issues present.
>
> ath9k issues tons of interrupts, not sure why irqpoll option would
> cause latency so bad as the interrupts *are* handled. Not sure
> *exactly* how irqpoll works but its description mentions using it
> forces each interrupt handler on the IRQ line to check the interrupt
> is for it. You have to keep in mind that not only are ath9k interrupts
> then being sent to the devices on its line but it would seem that all
> other devices on each line would suffer from the interrupts of the
> other guys. Why ath9k would be the *only* culprit of causing latency
> when using irqpoll if the irq line it son is clean? Beats me.

I'm guessing there's at least one interrupt that wasn't accounted for somehow.

>
>> alcarin steven # cat /proc/interrupts
>>           CPU0       CPU1
>> 0x0:      63227          0   IO-APIC-edge      hpet
>> 0x8:          1          0   IO-APIC-edge      rtc0
>> 0x9:      13080          0   IO-APIC-fasteoi   acpi
>> 0xe:       8195          0   IO-APIC-edge      ide0
>> 0xf:          0          0   IO-APIC-edge      ide1
>> 0x10:         36          0   IO-APIC-fasteoi   uhci_hcd:usb5
>> 0x11:      10645          0   IO-APIC-fasteoi   ath
>
> In this case your 11n Atheros device is on a clean line.
>
>> 0x12:         42          0   IO-APIC-fasteoi   uhci_hcd:usb4
>> 0x17:        919          0   IO-APIC-fasteoi   ehci_hcd:usb1, uhci_hcd:usb2
>
> But it was this interrupt line which had an interrupt not handled.

No, it's in hex. 0x17 = 23, 0x11 = 17. IRQ 17 is the one that pooped
in my case, which is my wireless chipset.

>> 0x13:      32885          0   IO-APIC-fasteoi   uhci_hcd:usb3,
>> ata_piix, ohci1394
>> 0x200100:          1          0   PCI-MSI-edge      eth0
>> 0x16:        223          0   IO-APIC-fasteoi   HDA Intel
>> NMI:          0          0   Non-maskable interrupts
>> LOC:      78087      95718   Local timer interrupts
>> RES:      11576      16384   Rescheduling interrupts
>> CAL:       6862       8889   Function call interrupts
>> TLB:         54         41   TLB shootdowns
>> TRM:          0          0   Thermal event interrupts
>> THR:          0          0   Threshold APIC interrupts
>> SPU:          0          0   Spurious interrupts
>> ERR:          0
>
> Can you try to reproduce the irq not handled again?

If I do, I'll need to know what precisely to do about it. What debug
info should I collect before rebooting?

>
>>>
>>> and also please do not cross post to all these lists, just use
>>> linux-wireless or ath9k.
>>>
>>
>> Sorry, but in the past I've posted to linux-wireless, ath9k-devel, and
>> all the maintainers of ath9k and didn't get a single response (except
>> a 'me too' from a fellow ath9k user). I didn't just want to hear
>> crickets this time.
>
> Patches speak more than words, but yeah sorry, we should have
> addressed this there. I've personally have just been busy with
> tackling aggregation.
>

Which is far more important, I agree. It's annoying to get speeds
<802.11b on my pre-802.11n capable chipset and network. ;)

- Steven

  parent reply	other threads:[~2008-09-18 21:50 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 [this message]
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
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=f488382f0809181449x7343aa7endff066cb5836a73c@mail.gmail.com \
    --to=steven@uplinklabs.net \
    --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®