Linux wireless drivers development
 help / color / mirror / Atom feed
From: "Greenman, Gregory" <gregory.greenman@intel.com>
To: "kvalo@kernel.org" <kvalo@kernel.org>,
	"mikezackles@gmail.com" <mikezackles@gmail.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"tmvolin@gmail.com" <tmvolin@gmail.com>,
	"dennisba@mail.uni-paderborn.de" <dennisba@mail.uni-paderborn.de>
Subject: Re: [PATCH] iwlwifi: Make missed beacon timeout configurable
Date: Sun, 14 Aug 2022 13:05:52 +0000	[thread overview]
Message-ID: <d9dc0c00d9ea8bb5b9341a7bcd4a75fc369a7d8f.camel@intel.com> (raw)
In-Reply-To: <CAN+PumF3JfmwVh_7BffLza+v7iD58QsxanCvbCrS1ztSt=XjNw@mail.gmail.com>

On Wed, 2022-08-03 at 08:50 -0700, Zachary Michaels wrote:
> > Why not just increase the default beacon timeout value?
> 
> I made it a parameter because I was under the vague impression that a
> larger timeout wouldn't be ideal on a well-behaved network, but this
> isn't my area of expertise. I just thought this would be the most
> conservative change since it would support identical default
> operation.
> 
> Thanks for taking a look.

I see that this issue https://bugzilla.kernel.org/show_bug.cgi?id=203709
has a long history from 2019. But, anyway, what value for
beacon_timeout solves the problem?

May I ask you to perform two experiments that could help us find
the root cause for this issue? The first is to run with a default driver and
to collect a firmware dump (here is a wiki with directions how to do that,
https://wireless.wiki.kernel.org/en/users/drivers/iwlwifi/debugging#firmware_debugging)
and the second is to load the defaut driver with modparam power_scheme=1. This will
completely disable power save and will let us know if the issue is somehow caused by
the NIC going to sleep and missing beacons.

Thanks,
Gregory


  reply	other threads:[~2022-08-14 13:06 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-26  4:50 [PATCH] iwlwifi: Make missed beacon timeout configurable Zachary Michaels
2022-08-02 18:50 ` Dennis Baurichter
2022-08-03  4:32   ` Kalle Valo
2022-08-03 15:50     ` Zachary Michaels
2022-08-14 13:05       ` Greenman, Gregory [this message]
2022-08-14 16:01         ` Zachary Michaels

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=d9dc0c00d9ea8bb5b9341a7bcd4a75fc369a7d8f.camel@intel.com \
    --to=gregory.greenman@intel.com \
    --cc=dennisba@mail.uni-paderborn.de \
    --cc=kvalo@kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=mikezackles@gmail.com \
    --cc=tmvolin@gmail.com \
    /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