linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Jussi Kivilinna" <jussi.kivilinna@mbnet.fi>
To: "Maxim Levitsky" <maximlevitsky@gmail.com>
Cc: ath5k-devel@lists.ath5k.org, linux-wireless@vger.kernel.org,
	"Luis R. Rodriguez" <mcgrof@bombadil.infradead.org>,
	"Bob Copeland" <bcopeland@gmail.com>
Subject: Re: [PATCH] ath5k: disable ASPM
Date: Fri, 18 Jun 2010 11:20:26 +0300	[thread overview]
Message-ID: <20100618112026.17623g6uhdjk8hts@naisho.dyndns.info> (raw)
In-Reply-To: <1276806785.20754.8.camel@maxim-laptop>

Quoting "Maxim Levitsky" <maximlevitsky@gmail.com>:

> On Fri, 2010-05-28 at 13:09 +0300, Jussi Kivilinna wrote:
>> Atheros card on Acer Aspire One (AOA150, Atheros Communications Inc. AR5001
>> Wireless Network Adapter [168c:001c] (rev 01)) doesn't work well with ASPM
>> enabled. With ASPM ath5k will eventually stall on heavy traffic with often
>> 'unsupported jumbo' warnings appearing. Disabling ASPM L0s/L1 in ath5k fixes
>> these problems.
>>
>> Reproduced with pcie_aspm=force and by using 'nc < /dev/zero > /dev/null' at
>> both ends (usually stalls within seconds).
>
> This fixes the same nasty problem on my AR2425.
>
> My AR2425 will stall if it transmits for about 1~2 minutes.
>
> It sends storm of RXORN interrupts although it only transmits.
>
> I see that now lspci calls it AR5001.
>
> Jussi Kivilinna, million thanks to you. I would never think of going
> this direction.
>
> Luis, so I was right after all, wasn't I?
> It is a hardware bug that is worked around in windows driver by
> disabling PCIE ASPM L0S.

I noticed this L0s disabling in windows driver too. I cannot test this  
anymore, since I don't have ath5k hw installed anymore (I switched to  
b43).

(ok, I might switch back ath5k to work on this, but opening AAO is  
pain.. on the other hand, I'm just user in this case and pretty  
unwilling to work with dual-license)

I did test device with L0s+L1 enabled (aspm=force), on this setup  
device fails within seconds. I tested patch with disabling L1 but not  
L0s, still fails but after longer time. I did _not_ test with L0s off  
but L1 enabled. So maybe it would be worth to test this patch with  
just disabling L0s.

AAO150 seems to enable L0s from BIOS, so this happens without  
aspm=force or CONFIG_.._ASPM at all.

  -Jussi


  reply	other threads:[~2010-06-18  8:20 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-28 10:09 [PATCH] ath5k: disable ASPM Jussi Kivilinna
2010-05-28 16:19 ` [ath5k-devel] " Pavel Roskin
2010-05-28 18:25   ` Jussi Kivilinna
2010-05-28 20:27     ` Pavel Roskin
2010-05-31  1:06       ` Bruno Randolf
2010-06-01 20:43         ` Luis R. Rodriguez
2010-05-28 17:40 ` Luis R. Rodriguez
2010-05-28 18:20   ` Jussi Kivilinna
2010-06-17 20:33 ` Maxim Levitsky
2010-06-18  8:20   ` Jussi Kivilinna [this message]
2010-06-18  9:09     ` [ath5k-devel] " RHS Linux User
2010-06-18 10:15     ` Maxim Levitsky
2010-06-18 10:49       ` Jussi Kivilinna
2010-06-18 11:05         ` Maxim Levitsky
2010-06-18 13:59           ` Bob Copeland
2010-06-18 14:11             ` Maxim Levitsky
2010-06-19  7:49               ` [PATCH v2] " Maxim Levitsky
2010-06-19 12:38                 ` Bob Copeland
2010-06-19 13:02                   ` Maxim Levitsky
2010-06-19 15:32                     ` [PATCH v3] " Maxim Levitsky
2010-07-26 20:13                       ` [ath5k-devel] " Luis R. Rodriguez
2010-07-26 20:49                         ` Maxim Levitsky
2010-07-26 21:06                           ` Luis R. Rodriguez
2010-07-26 21:14                             ` Matthew Garrett
2010-07-26 22:20                               ` Luis R. Rodriguez
2010-07-26 22:24                                 ` Matthew Garrett
2010-07-26 22:29                                   ` Luis R. Rodriguez
2010-07-26 21:17                             ` Maxim Levitsky
2010-07-26 21:25                               ` Matthew Garrett
2010-07-26 22:15                                 ` Luis R. Rodriguez
2010-07-26 22:21                                   ` Matthew Garrett
2010-07-26 22:26                                     ` Luis R. Rodriguez
2010-07-26 22:29                                       ` Matthew Garrett
2010-07-26 22:31                                         ` Luis R. Rodriguez
2010-07-26 22:33                                           ` Matthew Garrett
2010-07-26 22:43                                             ` Luis R. Rodriguez
2010-07-26 22:50                                               ` Matthew Garrett
2010-07-27  9:35                                                 ` Maxim Levitsky
2010-07-27 15:57                                                   ` Luis R. Rodriguez
2010-07-28 23:48                                                     ` Maxim Levitsky
2010-07-29  0:06                                                       ` Luis R. Rodriguez
2010-07-26 22:13                               ` Luis R. Rodriguez
2010-07-26 22:56                         ` Luis R. Rodriguez
2010-06-20  8:13                 ` [ath5k-devel] [PATCH v2] " Luis R. Rodriguez
2010-06-20 11:18                   ` Maxim Levitsky
2010-06-20 18:04                     ` Maxim Levitsky
2010-06-21  5:53                     ` Luis R. Rodriguez
2010-06-21 20:01                       ` Jussi Kivilinna
2010-06-21 20:16                       ` Maxim Levitsky
2010-06-21 20:33                         ` Jussi Kivilinna
2010-06-21 20:39                           ` Luis R. Rodriguez
2010-06-22 16:31                             ` Matthew Garrett
2010-06-22 16:48                               ` Luis R. Rodriguez
2010-06-22 16:52                                 ` Matthew Garrett
2010-06-22 17:17                                   ` Luis R. Rodriguez
2010-06-22 17:25                                     ` Matthew Garrett
2010-06-22 17:40                                       ` Luis R. Rodriguez
2010-06-22 17:50                                         ` Matthew Garrett
2010-06-22 18:28                                           ` Luis R. Rodriguez
2010-06-22 18:44                                             ` Matthew Garrett
2010-06-22 19:13                                               ` Luis R. Rodriguez
2010-06-22 19:31                                               ` Johannes Stezenbach
2010-06-22 19:37                                                 ` Luis R. Rodriguez
2010-06-22 19:38                                                   ` Luis R. Rodriguez
2010-06-23 14:39                                                     ` Johannes Stezenbach
2010-06-23 16:28                                                       ` Luis R. Rodriguez
2010-06-23 19:07                                                         ` Johannes Stezenbach
2010-06-23 19:23                                                   ` Johannes Stezenbach
2010-06-21 20:37                         ` Luis R. Rodriguez
2010-06-21 23:55                           ` Maxim Levitsky
2010-07-26 16:34 ` [PATCH] " Maxim Levitsky
2010-07-26 18:37   ` John W. Linville
2010-07-26 18:41   ` [ath5k-devel] " Luis R. Rodriguez

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=20100618112026.17623g6uhdjk8hts@naisho.dyndns.info \
    --to=jussi.kivilinna@mbnet.fi \
    --cc=ath5k-devel@lists.ath5k.org \
    --cc=bcopeland@gmail.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=maximlevitsky@gmail.com \
    --cc=mcgrof@bombadil.infradead.org \
    /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;
as well as URLs for NNTP newsgroup(s).