All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Lamparter <chunkeey@googlemail.com>
To: ath9k-devel@lists.ath9k.org
Subject: [ath9k-devel] Auth Packet TX Delay
Date: Sun, 24 Mar 2013 23:58:16 +0100	[thread overview]
Message-ID: <201303242358.17133.chunkeey@googlemail.com> (raw)
In-Reply-To: <CAMgrJ3buZxZermRmpUW2r3gYtUmF=3UoEo7NW9VYTkAwS7gdtQ@mail.gmail.com>

(Added Marco and me in the CC - please keep it)

On Sunday, March 24, 2013 11:40:27 PM Robert Shade wrote:
> > Hm, so it's doing some fast channel changes?
> 
> Yes, the fastcc does seem to work, it's just that sometimes the chip
> can get in a bad state when it's not cold reset.
> > Just disable fast channel change entirely and re-test? And why not
> > just force a cold reset always? Why bother checking for the queue to
> > be stopped?
> 
> Just disabling fastcc was not enough, the cold resets are what seemed
> to have made the difference.  I was actually thinking about
> re-enabling fastcc and testing again.
> 
> The TXE/RXE checking path is from felix's "ath9k_hw: improve reset
> reliability after errors" patch.  I've just got the exception in there
> for 9160, since I don't have other hardware to test with.  What do the
> hardware engineers say about warm vs. cold reset?  I did notice that
> your latest ar9300 HAL has a note stating that "Warm reset is
> optimistic."

Marco reported in "carl9170: monitor mode hangs due to channel changes"
<http://www.spinics.net/lists/linux-wireless/msg105100.html> that
carl9170 would stop receiving frames after some time.
I looked a bit closer look and it seems that the fast channel change
for AR9170 also seems to be the culprit in this case.

(It seems that the AGC_CONTROL register is suddenly (re-)set by
something to "0x20" (default is 0x0004dd20) and then everything
stops.)

Regards,
	Christian

WARNING: multiple messages have this Message-ID (diff)
From: Christian Lamparter <chunkeey@googlemail.com>
To: Robert Shade <robert.shade@gmail.com>
Cc: Adrian Chadd <adrian@freebsd.org>,
	linux-wireless@vger.kernel.org, ath9k-devel@lists.ath9k.org,
	Marco Fonseca <marco@tampabay.rr.com>
Subject: Re: Auth Packet TX Delay
Date: Sun, 24 Mar 2013 23:58:16 +0100	[thread overview]
Message-ID: <201303242358.17133.chunkeey@googlemail.com> (raw)
In-Reply-To: <CAMgrJ3buZxZermRmpUW2r3gYtUmF=3UoEo7NW9VYTkAwS7gdtQ@mail.gmail.com>

(Added Marco and me in the CC - please keep it)

On Sunday, March 24, 2013 11:40:27 PM Robert Shade wrote:
> > Hm, so it's doing some fast channel changes?
> 
> Yes, the fastcc does seem to work, it's just that sometimes the chip
> can get in a bad state when it's not cold reset.
> > Just disable fast channel change entirely and re-test? And why not
> > just force a cold reset always? Why bother checking for the queue to
> > be stopped?
> 
> Just disabling fastcc was not enough, the cold resets are what seemed
> to have made the difference.  I was actually thinking about
> re-enabling fastcc and testing again.
> 
> The TXE/RXE checking path is from felix's "ath9k_hw: improve reset
> reliability after errors" patch.  I've just got the exception in there
> for 9160, since I don't have other hardware to test with.  What do the
> hardware engineers say about warm vs. cold reset?  I did notice that
> your latest ar9300 HAL has a note stating that "Warm reset is
> optimistic."

Marco reported in "carl9170: monitor mode hangs due to channel changes"
<http://www.spinics.net/lists/linux-wireless/msg105100.html> that
carl9170 would stop receiving frames after some time.
I looked a bit closer look and it seems that the fast channel change
for AR9170 also seems to be the culprit in this case.

(It seems that the AGC_CONTROL register is suddenly (re-)set by
something to "0x20" (default is 0x0004dd20) and then everything
stops.)

Regards,
	Christian

  reply	other threads:[~2013-03-24 22:58 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-20 18:45 [ath9k-devel] Auth Packet TX Delay Robert Shade
2013-01-15 22:06 ` Robert Shade
2013-01-15 22:06   ` Robert Shade
2013-01-16 23:09   ` [ath9k-devel] " Adrian Chadd
2013-01-16 23:09     ` Adrian Chadd
2013-01-17  1:00     ` [ath9k-devel] " Robert Shade
2013-01-17  1:00       ` Robert Shade
2013-01-17  3:12       ` [ath9k-devel] " Adrian Chadd
2013-01-17  3:12         ` Adrian Chadd
2013-02-06  2:51         ` [ath9k-devel] " Robert Shade
2013-02-06  2:51           ` Robert Shade
2013-02-06  3:08           ` [ath9k-devel] " Adrian Chadd
2013-02-06  3:08             ` Adrian Chadd
2013-02-06 12:53             ` [ath9k-devel] " Robert Shade
2013-02-06 12:53               ` Robert Shade
2013-02-06 22:58               ` [ath9k-devel] " Robert Shade
2013-02-06 22:58                 ` Robert Shade
2013-02-07  5:06                 ` [ath9k-devel] " Adrian Chadd
2013-02-07  5:06                   ` Adrian Chadd
2013-02-07 16:25                   ` [ath9k-devel] " David Littell
2013-02-07 16:25                     ` David Littell
2013-02-07 19:43                   ` Robert Shade
2013-02-07 19:43                     ` Robert Shade
2013-02-07 21:40                     ` [ath9k-devel] " Adrian Chadd
2013-02-07 21:40                       ` Adrian Chadd
2013-02-08 18:48                       ` [ath9k-devel] " Robert Shade
2013-02-08 18:48                         ` Robert Shade
2013-02-09  7:39                         ` [ath9k-devel] " Adrian Chadd
2013-02-09  7:39                           ` Adrian Chadd
2013-03-24 18:55                           ` [ath9k-devel] " Robert Shade
2013-03-24 18:55                             ` Robert Shade
2013-03-24 21:52                             ` [ath9k-devel] " Adrian Chadd
2013-03-24 21:52                               ` Adrian Chadd
2013-03-24 22:40                               ` [ath9k-devel] " Robert Shade
2013-03-24 22:40                                 ` Robert Shade
2013-03-24 22:58                                 ` Christian Lamparter [this message]
2013-03-24 22:58                                   ` Christian Lamparter
2013-03-25  0:03                                   ` [ath9k-devel] " Adrian Chadd
2013-03-25  0:03                                     ` Adrian Chadd
2013-03-25  2:23                                     ` [ath9k-devel] " Adrian Chadd
2013-03-25  2:23                                       ` Adrian Chadd
2013-03-25 16:12                                       ` [ath9k-devel] " Christian Lamparter
2013-03-25 16:12                                         ` Christian Lamparter
2013-03-25 16:45                                         ` [ath9k-devel] " Adrian Chadd
2013-03-25 16:45                                           ` Adrian Chadd
2013-03-26 12:21                                           ` [ath9k-devel] " Robert Shade
2013-03-26 12:21                                             ` Robert Shade
2013-03-26 13:23                                             ` [ath9k-devel] " Robert Shade
2013-03-26 13:23                                               ` Robert Shade
2013-03-26 16:28                                             ` [ath9k-devel] " Adrian Chadd
2013-03-26 16:28                                               ` Adrian Chadd
2013-03-26 16:29                                               ` [ath9k-devel] " Robert Shade
2013-03-26 16:29                                                 ` Robert Shade
2013-03-26 17:13                                                 ` [ath9k-devel] " Ben Greear
2013-03-26 17:13                                                   ` Ben Greear
2013-03-26 17:16                                                   ` Adrian Chadd
2013-03-26 17:16                                                     ` Adrian Chadd
2013-03-26 17:27                                                     ` Ben Greear
2013-03-26 17:27                                                       ` Ben Greear
2013-03-26 17:33                                                       ` Adrian Chadd
2013-03-26 17:33                                                         ` Adrian Chadd
2013-03-26 17:45                                                         ` Ben Greear
2013-03-26 17:45                                                           ` Ben Greear
2013-03-27  0:55                                                   ` Robert Shade
2013-03-27  0:55                                                     ` Robert Shade
2013-03-27  3:11                                                     ` Robert Shade
2013-03-27  3:11                                                       ` Robert Shade
2013-03-27 16:33                                                       ` Adrian Chadd
2013-03-27 16:33                                                         ` Adrian Chadd
2013-03-26 17:14                                                 ` Adrian Chadd
2013-03-26 17:14                                                   ` Adrian Chadd
2013-03-26 14:13                                           ` [ath9k-devel] " Marco Fonseca
2013-03-26 14:13                                             ` Marco Fonseca
  -- strict thread matches above, loose matches on Subject: below --
2013-01-17 18:48 [ath9k-devel] " David Littell

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=201303242358.17133.chunkeey@googlemail.com \
    --to=chunkeey@googlemail.com \
    --cc=ath9k-devel@lists.ath9k.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.