public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: James Prestwood <prestwoj@gmail.com>
To: Paul Menzel <pmenzel@molgen.mpg.de>
Cc: Kalle Valo <kvalo@kernel.org>,
	Baochen Qiang <quic_bqiang@quicinc.com>,
	linux-wireless@vger.kernel.org, ath10k@lists.infradead.org,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: failed to remove key (0, ce:ce:1e:27:bb:e0) from hardware (-110) (ETIMEDOUT)
Date: Fri, 11 Oct 2024 09:36:29 -0700	[thread overview]
Message-ID: <7ccfcd67-986b-40a3-8311-73b5335f980c@gmail.com> (raw)
In-Reply-To: <8eda1bd2-88fc-4262-aec0-6f88a73797ef@molgen.mpg.de>

Hi Paul,

On 10/11/24 5:48 AM, Paul Menzel wrote:
> Dear James,
>
>
> Am 04.09.24 um 16:09 schrieb James Prestwood:
>
>> On 9/4/24 3:45 AM, Paul Menzel wrote:
>
>>> Linux 6.11-rc6+ logged the warning below when resuming from ACPI S3 
>>> (or unloading and loading the `ath10k_core`/`ath10k_pci` modules) 
>>> having been connected to an AVM network:
>>>
>>>     wlp58s0: failed to remove key (0, ce:ce:1e:27:bb:e0) from 
>>> hardware (-110)
>>>
>>> Error code 110 is the value for ETIMEDOUT. I saw James patch [1], 
>>> and applied it, and the error is still there (as expected).
>>
>> My patch won't actually fix the timeout, I just lowered the time that 
>> ath10k would wait before it continued which fixed some 
>> incompatibility on the AP side of things. Based on your logs though, 
>> it appears you already got disconnected before the failure to remove 
>> the key...
>
> I am still applying your patch manually on the current master branch, 
> and it looks like it hasn’t still been reviewed. Maybe it’s due to the 
> RFC tag. Do you plan on moving it ahead?

There was another thread, which I cant seem to find where Kalle had 
mentioned the RFC patch. IIRC Kalle had said the proper fix is in the 
firmware, and would rather not take this patch upstream if we can get 
the firmware addressed. And I would have to agree this is the best 
approach if this is an option. This RFC patch was merely a workaround 
for some odd behavior with Cisco APs that one of our customers was 
using, where the APs would actually reject roams if they took too long, 
hence why reducing the timeout "fixed" the problem.

Last I checked your situation was different, your device got 
disconnected _then_ failed to remove the key. If this is the case my RFC 
patch isn't going to really change much, apart from fail in 1 second vs 
3 seconds.

Thanks,

James

>
>
> Kind regards,
>
> Paul
>
>
>>> [1]: 
>>> https://lore.kernel.org/all/20240814164507.996303-1-prestwoj@gmail.com/

  reply	other threads:[~2024-10-11 16:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-04 10:45 failed to remove key (0, ce:ce:1e:27:bb:e0) from hardware (-110) (ETIMEDOUT) Paul Menzel
2024-09-04 10:49 ` Baochen Qiang
2024-09-04 14:09 ` James Prestwood
2024-10-11 12:48   ` Paul Menzel
2024-10-11 16:36     ` James Prestwood [this message]
2024-09-04 14:48 ` Kalle Valo
2024-09-04 21:42   ` Paul Menzel
2024-09-05 13:23     ` Kalle Valo

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=7ccfcd67-986b-40a3-8311-73b5335f980c@gmail.com \
    --to=prestwoj@gmail.com \
    --cc=ath10k@lists.infradead.org \
    --cc=kvalo@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=pmenzel@molgen.mpg.de \
    --cc=quic_bqiang@quicinc.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