All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sathishkumar Muruganandam <murugana@codeaurora.org>
To: linux-wireless@vger.kernel.org
Cc: Johannes Berg <johannes.berg@intel.com>
Subject: Multi-client EAPOL key timeout when not having RTNL lock protection
Date: Thu, 02 Sep 2021 10:57:04 +0530	[thread overview]
Message-ID: <6cae54aee83a19dc13e458e4d20e4f28@codeaurora.org> (raw)

Hi,

In the multi-client (64 or higher clients on single radio) test scenario 
of AP mode using hostapd, we are facing EAPOL key timeout for random 
clients.

wlan1: STA 00:41:c0:a8:03:10 WPA: received EAPOL-Key msg 4/4 in invalid 
state (7) – dropped

This is happening due to delay in transmission of association response 
frames for retried association request frames from the client and one of 
the association requests is received when EAPOL key exchange is in 
process. NL80211_CMD_NEW_STATION is received on hostapd when already 
EAPOL M3 is transmitted and waiting for EAPOL M4. But since hostapd 
received NL80211_CMD_NEW_STATION, it resets the handshake process from 
M1 again by the time client sends M4 to see the above error.

This delay is seen only after the commit, a05829a7222e ("cfg80211: avoid 
holding the RTNL when calling the driver") and not seen before/without 
this commit. We could see delay in processing of nl80211_get_key, 
nl80211_set_key, nl80211_new_key, nl80211_del_key and nl80211_tx_mgmt 
commands.

The delay and EAPOL key timeout is not seen when NL80211_FLAG_NEED_RTNL 
is set back to internal_flags of nl80211_get_key, nl80211_set_key, 
nl80211_new_key, nl80211_del_key and nl80211_tx_mgmt messages alone.

Please share your comments on this issue requiring RTNL lock for key and 
mgmt nl80211 commands.

With regards,
Sathishkumar

             reply	other threads:[~2021-09-02  5:27 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-02  5:27 Sathishkumar Muruganandam [this message]
2021-09-02 14:15 ` Multi-client EAPOL key timeout when not having RTNL lock protection Johannes Berg

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=6cae54aee83a19dc13e458e4d20e4f28@codeaurora.org \
    --to=murugana@codeaurora.org \
    --cc=johannes.berg@intel.com \
    --cc=linux-wireless@vger.kernel.org \
    /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.