public inbox for linux-wireless@vger.kernel.org
 help / color / mirror / Atom feed
From: "Alexey Klimov" <alexey.klimov@linaro.org>
To: "Loic Poulain" <loic.poulain@oss.qualcomm.com>
Cc: <jjohnson@kernel.org>,
	"Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
	<ath10k@lists.infradead.org>, <linux-wireless@vger.kernel.org>,
	<jeff.johnson@oss.qualcomm.com>, <linux-arm-msm@vger.kernel.org>
Subject: Re: [question, bug] regularly disconnecting wifi on RB1 and RB2 boards, ath10
Date: Wed, 30 Jul 2025 09:45:54 +0100	[thread overview]
Message-ID: <DBP9B4ZMXOMK.1QLITVTW393YI@linaro.org> (raw)
In-Reply-To: <DBKDAXJ0V9U0.10YYX5H5HAJBT@linaro.org>

On Thu Jul 24, 2025 at 3:50 PM BST, Alexey Klimov wrote:
> Hi Loic,
>
> On Wed Jul 23, 2025 at 11:42 AM BST, Loic Poulain wrote:
>> Hi Alexey,
>>

[..]

>> Could you check if this change helps:
>>
>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c
>> b/drivers/net/wireless/ath/ath10k/mac.c
>> index c61b95a928da..4fa7dd62aeac 100644
>> --- a/drivers/net/wireless/ath/ath10k/mac.c
>> +++ b/drivers/net/wireless/ath/ath10k/mac.c
>> @@ -288,8 +288,10 @@ static int ath10k_send_key(struct ath10k_vif *arvif,
>>                 key->flags |= IEEE80211_KEY_FLAG_GENERATE_IV;
>>
>>         if (cmd == DISABLE_KEY) {
>> -               arg.key_cipher = ar->wmi_key_cipher[WMI_CIPHER_NONE];
>> -               arg.key_data = NULL;
>> +               /*  Not all hardware supports key deletion operations. so we
>> +                *  replace the key with a junk value to invalidate it.
>> +                */
>> +               memset(arg.key_data, 0, arg.key_len);
>>         }

So far looks good.

I didn't see any kind of GROUP_KEY_HANDSHAKE_TIMEOUT messages and long wifi
outages leaving the RB1, for instance, overnight.

I do observe some packet loss while pinging the board for a while --
around 0.1..0.6% packet loss but that might be because of my wifi network
and absence of external antenna and it is still much much better than
default behaviour.

Could you please add me to C/c when you're going to send it over?

Thank you for looking into this,
Alexey


      reply	other threads:[~2025-07-30  8:45 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Zgp0ym-MGzX2eSZdlkVYbgvjkJ0CzKItjaC5pafzQnj1AOZnVAqvCIZfYoK7nwDhUgOA0U8eNolNtaWXbExOAQ==@protonmail.internalid>
2025-06-26 14:48 ` [question, bug] regularly disconnecting wifi on RB1 and RB2 boards, ath10 Alexey Klimov
2025-06-26 15:02   ` Jeff Johnson
2025-06-26 15:10     ` Alexey Klimov
2025-06-26 23:09   ` Bryan O'Donoghue
2025-07-22 15:53     ` Loic Poulain
2025-07-23 10:42       ` Loic Poulain
2025-07-24 14:50         ` Alexey Klimov
2025-07-30  8:45           ` Alexey Klimov [this message]

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=DBP9B4ZMXOMK.1QLITVTW393YI@linaro.org \
    --to=alexey.klimov@linaro.org \
    --cc=ath10k@lists.infradead.org \
    --cc=bryan.odonoghue@linaro.org \
    --cc=jeff.johnson@oss.qualcomm.com \
    --cc=jjohnson@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=loic.poulain@oss.qualcomm.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