linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Emmanuel Grumbach <egrumbach@gmail.com>
To: Andy Lutomirski <luto@amacapital.net>
Cc: Linux Wireless List <linux-wireless@vger.kernel.org>
Subject: Re: Help debugging iwldvm / ath10k stalls
Date: Mon, 26 May 2014 11:39:28 +0300	[thread overview]
Message-ID: <CANUX_P2XpeDOmNOX=p4yMUe5ZAZv-UQ97M0ayFcFSsTrEDS3CA@mail.gmail.com> (raw)
In-Reply-To: <CANUX_P2sH1yzx5aeLbHXBGHAsXmkZ70ioEhH1Ad_xNrCEJbJRg@mail.gmail.com>

On Mon, May 26, 2014 at 11:29 AM, Emmanuel Grumbach <egrumbach@gmail.com> wrote:
>>>>>
>>>>> Every now and then, my connection drops out.  From wireshark on the
>>>>> station, it looks like traffic from the AP to the STA is coming
>>>>> through but traffic from the STA to the AP is not.  I could be wrong
>>>>> here, though.  Nothing is logged in either machine's kernel log.
>>>>> After 5 seconds to a couple minutes, everything starts working again.
>>>>>
>>>>> A couple of Apple devices using the same radio on the same AP seem to work fine.
>>>>>
>>>>> Any advice for debugging this?  I don't know where to start.
>>>>
>>>> I caught one of the stalls with iwlwifi debugging on.  I've attached
>>>> the dmesg.  The stall ended ten seconds or so before the end of this
>>>> trace.  Is this an indication that something's wrong with iwlwifi?
>>>>
>>>> [151258.096091] iwlwifi 0000:03:00.0: U iwlagn_good_plcp_health plcp health thre
>>>> shold 50 delta 96 msecs 103
>>>> [151258.096095] iwlwifi 0000:03:00.0: U iwl_force_rf_reset perform radio reset.
>>>>
>>> Can you please disable powersave?
>>> It seems that it should be disabled by default, but you enabled it?
>>> I might be confused though... Looking again at the code...
>>
>> This?
>>
>> $ cat /sys/module/iwlwifi/parameters/power_save
>> N
>>
>> I have pcie_aspm.policy=powersave, but I don't think that should
>> matter.  Or am I missing something?
>
> That should be ok.

Does this help?

diff --git a/drivers/net/wireless/iwlwifi/dvm/power.c
b/drivers/net/wireless/iwlwifi/dvm/power.c
index f2c1439..e5a6b70 100644
--- a/drivers/net/wireless/iwlwifi/dvm/power.c
+++ b/drivers/net/wireless/iwlwifi/dvm/power.c
@@ -288,6 +288,9 @@ static void iwl_power_build_cmd(struct iwl_priv *priv,
        bool enabled = priv->hw->conf.flags & IEEE80211_CONF_PS;
        int dtimper;

+       iwl_power_sleep_cam_cmd(priv, cmd);
+       return;
+
        dtimper = priv->hw->conf.ps_dtim_period ?: 1;

        if (priv->wowlan)

  reply	other threads:[~2014-05-26  8:39 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-25 17:23 Help debugging iwldvm / ath10k stalls Andy Lutomirski
2014-05-26  5:06 ` Andy Lutomirski
2014-05-26  5:39   ` Emmanuel Grumbach
2014-05-26  6:11     ` Andy Lutomirski
2014-05-26  8:29       ` Emmanuel Grumbach
2014-05-26  8:39         ` Emmanuel Grumbach [this message]
2014-05-26 17:28           ` Andy Lutomirski
2014-05-26 18:00             ` Emmanuel Grumbach
2014-05-26 18:08               ` Andy Lutomirski
2014-05-26 18:13                 ` Emmanuel Grumbach
2014-05-26 18:47                   ` Andy Lutomirski
2014-05-27 17:09                     ` Andy Lutomirski
2014-05-27 18:01                       ` Andy Lutomirski
2014-05-27 20:30                         ` Emmanuel Grumbach
2014-05-28  3:07                           ` Andy Lutomirski
2014-05-28 12:09                             ` Emmanuel Grumbach
2014-06-02 16:54                               ` Andy Lutomirski
2014-06-02 18:11                                 ` Andy Lutomirski
2014-06-02 18:40                                   ` Emmanuel Grumbach
2014-06-02 18:45                                     ` Andy Lutomirski
2014-06-02 18:53                                       ` Emmanuel Grumbach
2014-06-02 18:56                                         ` Andy Lutomirski
2014-06-02 19:32                                           ` Emmanuel Grumbach
2014-06-02 19:57                                             ` Andy Lutomirski
2014-06-02 20:15                                               ` Emmanuel Grumbach

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='CANUX_P2XpeDOmNOX=p4yMUe5ZAZv-UQ97M0ayFcFSsTrEDS3CA@mail.gmail.com' \
    --to=egrumbach@gmail.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=luto@amacapital.net \
    /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).