All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arend van Spriel <arend.vanspriel@broadcom.com>
To: Daniel Drake <drake@endlessm.com>
Cc: franky.lin@broadcom.com, hante.meuleman@broadcom.com,
	chi-hsien.lin@cypress.com, wright.feng@cypress.com,
	linux-wireless@vger.kernel.org,
	brcm80211-dev-list.pdl@broadcom.com,
	brcm80211-dev-list@cypress.com,
	Linux Upstreaming Team <linux@endlessm.com>
Subject: Re: Make brcmfmac repeat authentication requests
Date: Thu, 15 Feb 2018 13:46:31 +0100	[thread overview]
Message-ID: <5A858127.1030904@broadcom.com> (raw)
In-Reply-To: <CAD8Lp46ta_mQriB1eOJxnTmgoy5n5T9_AyNpVKuL2C4uuLrf3g@mail.gmail.com>

On 2/14/2018 2:03 PM, Daniel Drake wrote:
> Hi,
>
> Thanks for the fast response.
>
> On Tue, Feb 13, 2018 at 12:50 PM, Arend van Spriel
> <arend.vanspriel@broadcom.com> wrote:
>> I tried to find info about that access point equipment, but not getting any
>> hits apart from a olivetti laser printer, but I doubt it is that. Can you
>> provide more details.
>
> The device itself is basically unbranded (just says "4G LTE"). It's an
> access point and mifi bridge (so insert a sim card and it shares your
> mobile data connection on the LAN). It comes as part of a solar home
> solutions package.
>
> MF928 is listed as the product name behind the battery. In the web UI
> it says it is from the EV910 product family, hardware version
> LR521_V1.0. I can't find info online about it.

Probably it is from ZTE although I could not find the MF928 specifically.

>> User-space (wpa_supplicant) would retry the connect attempt so I guess you
>> are saying that the timing between the two auth requests is important?
>
> Yes, the error goes up to userspace which then retries. However around
> 15 seconds pass before the authentication request is sent again, and
> also as part of the retry it redoes the probe requests etc. Windows
> does the same but there is only a 3 second delay. I haven't checked if
> this device needs the authentication request resent in less than 3
> seconds, or if the problem is that it needs to be sent twice in
> consecutive frames (i.e. without another probe request in the middle).
>
>> Is firmware not repeating at all or is the time between the two auth
>> requests too long?
>
> Firmware is not repeating at all.
>
>> Checking firmware there is a 300ms timeout and it does a retry if the limit
>> is not reached. However, that limit is initialized to zero :-p
>>
>> Could you try the patch below?
>
> Thanks for looking into the firmware! Unfortunately the change does
> not appear to make any difference. As before, the auth request is
> ACKed by the AP but then the conversation halts until userspace steps
> in on timeout a few seconds later.

Ok. Could you create a log with driver debugging enabled, ie. build 
driver CONFIG_BRCMDBG=y and load with module param 'debug=0x1416'. The 
problem is probably when the firmware is configured.

Regards,
Arend

> Daniel
>
>
>> Regards,
>> Arend
>>
>> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
>> b/drivers
>> index 19686ef..af1ab00 100644
>> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
>> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
>> @@ -384,6 +384,9 @@ int brcmf_c_preinit_dcmds(struct brcmf_if *ifp)
>>                  goto done;
>>          }
>>
>> +       /* allow join retry by firmware */
>> +       (void)brcmf_fil_iovar_int_set(ifp, "assoc_retry_max", 1);
>> +
>>          /* Enable tx beamforming, errors can be ignored (not supported) */
>>          (void)brcmf_fil_iovar_int_set(ifp, "txbf", 1);
>>

  reply	other threads:[~2018-02-15 12:46 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-12 14:04 Make brcmfmac repeat authentication requests Daniel Drake
2018-02-13  9:50 ` Arend van Spriel
2018-02-14 13:03   ` Daniel Drake
2018-02-15 12:46     ` Arend van Spriel [this message]
2018-02-16  7:36       ` Daniel Drake

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=5A858127.1030904@broadcom.com \
    --to=arend.vanspriel@broadcom.com \
    --cc=brcm80211-dev-list.pdl@broadcom.com \
    --cc=brcm80211-dev-list@cypress.com \
    --cc=chi-hsien.lin@cypress.com \
    --cc=drake@endlessm.com \
    --cc=franky.lin@broadcom.com \
    --cc=hante.meuleman@broadcom.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linux@endlessm.com \
    --cc=wright.feng@cypress.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 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.