* [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH
@ 2024-02-15 14:13 Alexis Lothoré
2024-02-15 14:13 ` [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in " Alexis Lothoré
` (2 more replies)
0 siblings, 3 replies; 15+ messages in thread
From: Alexis Lothoré @ 2024-02-15 14:13 UTC (permalink / raw)
To: Johannes Berg
Cc: Ajay Singh, Kalle Valo, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich,
Alexis Lothoré,
kernel test robot
This small series is the follow-up to discussions started around a sparse
warning in wilc1000 driver ([1]) and implements the solution suggested by
Johannes. It moves a historically needed conversion to be32 in nl80211 (in
NL80211_CMD_EXTERNAL_AUTH, specifically on NL80211_ATTR_AKM_SUITES property
_only_ when it is set to WLAN_AKM_SUITE_SAE) The user scenario affected by
this update is a connect process on a WPA3-protected access point with
authentication offloaded to user-space. Two drivers are affected by the
update: wilc1000 and qtnfmac. wilc1000 case is handled by a small
companion patch which also fixes the sparse warning.
For the quantenna driver, I don't really get how it manipulates AKM suites.
The only thing it currently does on it before calling nl80211 is a
le32_to_cpu. IIUC the raw value (before applying le32_to_cpu) comes from
chip/firmware:
<interrupt>
qtnf_shm_ipc_irq_handler
<some callbacks chains>
qtnf_pcie_control_rx_callback
qtnf_trans_handle_rx_ctl_packet
qtnf_trans_event_enqueue => queue skb to processing queue
qtnf_event_work_handler <= dequeue corresponding skb to process
qtnf_event_process_skb
qtnf_event_parse
qtnf_event_handle_external_auth
cfg80211_external_auth_request => sends NL80211_CMD_EXTERNAL_AUTH
There is no cast to big endian on AKM suite at any point in this chain, but
there are plenty of leXX_to_cpu, so I assume the chip/its firmware sends
its data in little endian. Then, since the be32 conversion is _needed_ with
current wpa_supplicant, I wonder if it works at all in current state, so I
did not modify it. Or has it been tested with another supplicant (iwd ?)
which handles WLAN_AKM_SUITE_SAE differently ? Opinions (and even some
testing) are welcome for this driver, since I do not have the corresponding
hardware.
[1] https://lore.kernel.org/linux-wireless/87a5uatfl1.fsf@kernel.org/
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Ajay Singh <ajay.kathat@microchip.com>
Cc: Kalle Valo <kvalo@kernel.org>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: <linux-wireless@vger.kernel.org>
Cc: <linux-kernel@vger.kernel.org>
Cc: Igor Mitsyanko <imitsyanko@quantenna.com>
Cc: Sergey Matyukevich <geomatsi@gmail.com>
Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
---
Alexis Lothoré (2):
wifi: nl80211: force WLAN_AKM_SUITE_SAE in big endian in NL80211_CMD_EXTERNAL_AUTH
wifi: wilc1000: remove AKM suite be32 conversion for external auth request
drivers/net/wireless/microchip/wilc1000/cfg80211.c | 2 +-
net/wireless/nl80211.c | 19 ++++++++++++++++++-
2 files changed, 19 insertions(+), 2 deletions(-)
---
base-commit: a4c7842e88b0f7d937015e4588ea2a1dec33cf2c
change-id: 20240214-nl80211_fix_akm_suites_endianness-2750a5d7da83
Best regards,
--
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in big endian in NL80211_CMD_EXTERNAL_AUTH
2024-02-15 14:13 [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
@ 2024-02-15 14:13 ` Alexis Lothoré
2024-02-15 16:58 ` Kalle Valo
2024-02-15 14:13 ` [PATCH 2/2] wifi: wilc1000: remove AKM suite be32 conversion for external auth request Alexis Lothoré
2024-02-15 15:50 ` [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
2 siblings, 1 reply; 15+ messages in thread
From: Alexis Lothoré @ 2024-02-15 14:13 UTC (permalink / raw)
To: Johannes Berg
Cc: Ajay Singh, Kalle Valo, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich,
Alexis Lothoré
User-space supplicant (observed at least on wpa_supplicant) historically
parses the NL80211_ATTR_AKM_SUITES from the NL80211_CMD_EXTERNAL_AUTH
message as big endian _only_ when its value is WLAN_AKM_SUITE_SAE, while
processing anything else in host endian. This behavior makes any driver
relying on SAE external auth to switch AKM suite to big endian if it is
WLAN_AKM_SUITE_SAE. A fix bringing compatibility with both endianness
has been brought into wpa_supplicant, however we must keep compatibility
with older versions, while trying to reduce the occurences of this manual
conversion in wireless drivers.
Add the be32 conversion specifically on WLAN_AKM_SUITE_SAE in nl80211 layer
to keep compatibility with older wpa_supplicant versions.
Suggested-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
---
Only a few minor modifications from Johannes' initial suggestion:
- fixed unbalanced parenthesis
- slightly reworded the inlined documentation
---
net/wireless/nl80211.c | 19 ++++++++++++++++++-
1 file changed, 18 insertions(+), 1 deletion(-)
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
index 5f18cbf7cc3d..046ce0513d31 100644
--- a/net/wireless/nl80211.c
+++ b/net/wireless/nl80211.c
@@ -20136,9 +20136,26 @@ int cfg80211_external_auth_request(struct net_device *dev,
if (!hdr)
goto nla_put_failure;
+ /* Some historical mistakes in drivers <-> userspace interface (notably
+ * between drivers and wpa_supplicant) led to a big-endian conversion
+ * being needed on NL80211_ATTR_AKM_SUITES _only_ when its value is
+ * WLAN_AKM_SUITE_SAE. This is now fixed on userspace side, but for the
+ * benefit of older wpa_supplicant versions, send this particular value
+ * in big-endian. Note that newer wpa_supplicant will also detect this
+ * particular value in big endian still, so it all continues to work.
+ */
+ if (params->key_mgmt_suite == WLAN_AKM_SUITE_SAE) {
+ if (nla_put_be32(msg, NL80211_ATTR_AKM_SUITES,
+ cpu_to_be32(WLAN_AKM_SUITE_SAE)))
+ goto nla_put_failure;
+ } else {
+ if (nla_put_u32(msg, NL80211_ATTR_AKM_SUITES,
+ params->key_mgmt_suite))
+ goto nla_put_failure;
+ }
+
if (nla_put_u32(msg, NL80211_ATTR_WIPHY, rdev->wiphy_idx) ||
nla_put_u32(msg, NL80211_ATTR_IFINDEX, dev->ifindex) ||
- nla_put_u32(msg, NL80211_ATTR_AKM_SUITES, params->key_mgmt_suite) ||
nla_put_u32(msg, NL80211_ATTR_EXTERNAL_AUTH_ACTION,
params->action) ||
nla_put(msg, NL80211_ATTR_BSSID, ETH_ALEN, params->bssid) ||
--
2.43.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/2] wifi: wilc1000: remove AKM suite be32 conversion for external auth request
2024-02-15 14:13 [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
2024-02-15 14:13 ` [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in " Alexis Lothoré
@ 2024-02-15 14:13 ` Alexis Lothoré
2024-02-21 18:54 ` Kalle Valo
2024-02-15 15:50 ` [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
2 siblings, 1 reply; 15+ messages in thread
From: Alexis Lothoré @ 2024-02-15 14:13 UTC (permalink / raw)
To: Johannes Berg
Cc: Ajay Singh, Kalle Valo, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich,
Alexis Lothoré,
kernel test robot
The driver currently raises the following sparse warning:
[...] cfg80211.c:360:42: warning: incorrect type in assignment (different
base types)
[...] cfg80211.c:360:42: expected unsigned int key_mgmt_suite
[...] cfg80211.c:360:42: got restricted __be32 [usertype]
CHECK drivers/net/wireless/microchip/wilc1000/netdev.c
This conversion was needed because historically the external supplicant
(observed with wpa_supplicant) expects AKM suite as big endian in
NL80211_CMD_EXTERNAL_AUTH message when the AKM suite is WLAN_AKM_SUITE_SAE.
This is not needed anymore:
- new (to be released) versions of wpa_supplicant now reads it in host
endian _while_ keeping compatibility for older drivers
- for new drivers used with current/old wpa_supplicant, this conversion has
been added to nl80211 to force big endian when the AKM suite is
WLAN_AKM_SUITE_SAE
Remove this not-needed-anymore conversion to fix the sparse warning.
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202308290615.lUTIgqUl-lkp@intel.com/
Tested-on: WILC1000 hwB SPI WILC_WIFI_FW_REL_16_1-13452
Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
---
drivers/net/wireless/microchip/wilc1000/cfg80211.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/wireless/microchip/wilc1000/cfg80211.c b/drivers/net/wireless/microchip/wilc1000/cfg80211.c
index f03fd15c0c97..cb564e2bad29 100644
--- a/drivers/net/wireless/microchip/wilc1000/cfg80211.c
+++ b/drivers/net/wireless/microchip/wilc1000/cfg80211.c
@@ -356,7 +356,7 @@ static int connect(struct wiphy *wiphy, struct net_device *dev,
memcpy(vif->auth.ssid.ssid, sme->ssid, sme->ssid_len);
vif->auth.ssid.ssid_len = sme->ssid_len;
}
- vif->auth.key_mgmt_suite = cpu_to_be32(sme->crypto.akm_suites[0]);
+ vif->auth.key_mgmt_suite = sme->crypto.akm_suites[0];
ether_addr_copy(vif->auth.bssid, sme->bssid);
break;
--
2.43.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH
2024-02-15 14:13 [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
2024-02-15 14:13 ` [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in " Alexis Lothoré
2024-02-15 14:13 ` [PATCH 2/2] wifi: wilc1000: remove AKM suite be32 conversion for external auth request Alexis Lothoré
@ 2024-02-15 15:50 ` Alexis Lothoré
2024-02-15 17:06 ` wireless: orphan qfnfmac and plfxlc drivers? Kalle Valo
2 siblings, 1 reply; 15+ messages in thread
From: Alexis Lothoré @ 2024-02-15 15:50 UTC (permalink / raw)
To: Johannes Berg
Cc: Ajay Singh, Kalle Valo, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich,
kernel test robot, Claudiu Beznea
On 2/15/24 15:13, Alexis Lothoré wrote:
> This small series is the follow-up to discussions started around a sparse
> warning in wilc1000 driver ([1]) and implements the solution suggested by
> Johannes. It moves a historically needed conversion to be32 in nl80211 (in
> NL80211_CMD_EXTERNAL_AUTH, specifically on NL80211_ATTR_AKM_SUITES property
> _only_ when it is set to WLAN_AKM_SUITE_SAE) The user scenario affected by
> this update is a connect process on a WPA3-protected access point with
> authentication offloaded to user-space. Two drivers are affected by the
> update: wilc1000 and qtnfmac. wilc1000 case is handled by a small
> companion patch which also fixes the sparse warning.
Adding Claudio Beznea (co-maintainer for WILC), who got lost when I prepared the
series, sorry.
Also, my mail provider returns error 550 (No Such User Here) for quantenna
driver maintainer (<imitsyanko@quantenna.com>, taken from MAINTAINERS). I've
seen no recent activity from him on the ML, is he still around ?
--
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in big endian in NL80211_CMD_EXTERNAL_AUTH
2024-02-15 14:13 ` [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in " Alexis Lothoré
@ 2024-02-15 16:58 ` Kalle Valo
2024-02-16 8:21 ` Alexis Lothoré
0 siblings, 1 reply; 15+ messages in thread
From: Kalle Valo @ 2024-02-15 16:58 UTC (permalink / raw)
To: Alexis Lothoré
Cc: Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich
Alexis Lothoré <alexis.lothore@bootlin.com> writes:
> User-space supplicant (observed at least on wpa_supplicant) historically
> parses the NL80211_ATTR_AKM_SUITES from the NL80211_CMD_EXTERNAL_AUTH
> message as big endian _only_ when its value is WLAN_AKM_SUITE_SAE, while
> processing anything else in host endian. This behavior makes any driver
> relying on SAE external auth to switch AKM suite to big endian if it is
> WLAN_AKM_SUITE_SAE. A fix bringing compatibility with both endianness
> has been brought into wpa_supplicant, however we must keep compatibility
> with older versions, while trying to reduce the occurences of this manual
> conversion in wireless drivers.
>
> Add the be32 conversion specifically on WLAN_AKM_SUITE_SAE in nl80211 layer
> to keep compatibility with older wpa_supplicant versions.
>
> Suggested-by: Johannes Berg <johannes@sipsolutions.net>
> Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
A pointer to the discussion would be nice to have:
Link: https://lore.kernel.org/all/09eeb7d4-c922-45ee-a1ac-59942153dbce@bootlin.com/
I assume Johannes can add that.
Alexis, thanks so much for working on this! This has been bugging me for
long but never found the time to investigate it.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 15+ messages in thread
* wireless: orphan qfnfmac and plfxlc drivers?
2024-02-15 15:50 ` [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
@ 2024-02-15 17:06 ` Kalle Valo
2024-02-15 17:21 ` Kalle Valo
0 siblings, 1 reply; 15+ messages in thread
From: Kalle Valo @ 2024-02-15 17:06 UTC (permalink / raw)
To: Alexis Lothoré
Cc: Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich,
kernel test robot, Claudiu Beznea, srini.raju
(changing subject)
Alexis Lothoré <alexis.lothore@bootlin.com> writes:
> Also, my mail provider returns error 550 (No Such User Here) for quantenna
> driver maintainer (<imitsyanko@quantenna.com>, taken from MAINTAINERS). I've
> seen no recent activity from him on the ML, is he still around ?
I found a bounce for imitsyanko@quantenna.com from 2022 so I guess it's
time to orphan qtnfmac?
Also the purelife maintainer Srini Raju (CCed) is bouncing, that's
another candidate to orphan.
PURELIFI PLFXLC DRIVER
M: Srinivasan Raju <srini.raju@purelifi.com>
L: linux-wireless@vger.kernel.org
S: Supported
F: drivers/net/wireless/purelifi/plfxlc/
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: wireless: orphan qfnfmac and plfxlc drivers?
2024-02-15 17:06 ` wireless: orphan qfnfmac and plfxlc drivers? Kalle Valo
@ 2024-02-15 17:21 ` Kalle Valo
2024-02-15 18:51 ` Jeff Johnson
0 siblings, 1 reply; 15+ messages in thread
From: Kalle Valo @ 2024-02-15 17:21 UTC (permalink / raw)
To: Alexis Lothoré
Cc: Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
Igor Mitsyanko, Sergey Matyukevich, kernel test robot,
Claudiu Beznea, srini.raju
Kalle Valo <kvalo@kernel.org> writes:
> (changing subject)
>
> Alexis Lothoré <alexis.lothore@bootlin.com> writes:
>
>> Also, my mail provider returns error 550 (No Such User Here) for quantenna
>> driver maintainer (<imitsyanko@quantenna.com>, taken from MAINTAINERS). I've
>> seen no recent activity from him on the ML, is he still around ?
>
> I found a bounce for imitsyanko@quantenna.com from 2022 so I guess it's
> time to orphan qtnfmac?
>
> Also the purelife maintainer Srini Raju (CCed) is bouncing, that's
> another candidate to orphan.
And confirmed, I just got bounces for both of these maintainers. Patches
welcome to orphan these drivers.
<srini.raju@purelifi.com>: host
purelifi-com.mail.protection.outlook.com[52.101.68.16] said: 550 5.4.1
Recipient address rejected: Access denied.
[DB5PEPF00014B8D.eurprd02.prod.outlook.com 2024-02-15T17:07:03.735Z
08DC2C530F4F7910] (in reply to RCPT TO command)
<imitsyanko@quantenna.com>: host mail.quantenna.com[50.87.249.29] said: 550 No
Such User Here (in reply to RCPT TO command)
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: wireless: orphan qfnfmac and plfxlc drivers?
2024-02-15 17:21 ` Kalle Valo
@ 2024-02-15 18:51 ` Jeff Johnson
2024-02-16 9:37 ` Kalle Valo
0 siblings, 1 reply; 15+ messages in thread
From: Jeff Johnson @ 2024-02-15 18:51 UTC (permalink / raw)
To: Kalle Valo, Alexis Lothoré
Cc: Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
Igor Mitsyanko, Sergey Matyukevich, kernel test robot,
Claudiu Beznea, srini.raju
On 2/15/2024 9:21 AM, Kalle Valo wrote:
> <imitsyanko@quantenna.com>: host mail.quantenna.com[50.87.249.29] said: 550 No
> Such User Here (in reply to RCPT TO command)
Apparently Quantenna was acquired by ON Semi in 2019[1], and in 2022
they closed it down[2]. Seems pretty abandoned to me.
[1]
https://www.eetimes.com/on-semi-to-acquire-wi-fi-chip-vendor-quantenna-for-1-billion/
[2]
https://www.bizjournals.com/phoenix/news/2022/09/20/onsemi-close-down-quantenna.html
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in big endian in NL80211_CMD_EXTERNAL_AUTH
2024-02-15 16:58 ` Kalle Valo
@ 2024-02-16 8:21 ` Alexis Lothoré
2024-02-16 9:34 ` Kalle Valo
0 siblings, 1 reply; 15+ messages in thread
From: Alexis Lothoré @ 2024-02-16 8:21 UTC (permalink / raw)
To: Kalle Valo
Cc: Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich
On 2/15/24 17:58, Kalle Valo wrote:
> Alexis Lothoré <alexis.lothore@bootlin.com> writes:
>
>> User-space supplicant (observed at least on wpa_supplicant) historically
>> parses the NL80211_ATTR_AKM_SUITES from the NL80211_CMD_EXTERNAL_AUTH
>> message as big endian _only_ when its value is WLAN_AKM_SUITE_SAE, while
>> processing anything else in host endian. This behavior makes any driver
>> relying on SAE external auth to switch AKM suite to big endian if it is
>> WLAN_AKM_SUITE_SAE. A fix bringing compatibility with both endianness
>> has been brought into wpa_supplicant, however we must keep compatibility
>> with older versions, while trying to reduce the occurences of this manual
>> conversion in wireless drivers.
>>
>> Add the be32 conversion specifically on WLAN_AKM_SUITE_SAE in nl80211 layer
>> to keep compatibility with older wpa_supplicant versions.
>>
>> Suggested-by: Johannes Berg <johannes@sipsolutions.net>
>> Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
>
> A pointer to the discussion would be nice to have:
>
> Link: https://lore.kernel.org/all/09eeb7d4-c922-45ee-a1ac-59942153dbce@bootlin.com/
>
> I assume Johannes can add that.
Ah yes, indeed. Johannes, please let me know if you prefer me to resend it with
the link in the commit message.
> Alexis, thanks so much for working on this! This has been bugging me for
> long but never found the time to investigate it.
I'm glad to help, especially since I have the corresponding hardware. This
warning was on my radar, and your last complaint about remaining sparse warnings
in the wireless tree eventually triggered the action :)
--
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in big endian in NL80211_CMD_EXTERNAL_AUTH
2024-02-16 8:21 ` Alexis Lothoré
@ 2024-02-16 9:34 ` Kalle Valo
0 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2024-02-16 9:34 UTC (permalink / raw)
To: Alexis Lothoré
Cc: Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
linux-kernel, Sergey Matyukevich
Alexis Lothoré <alexis.lothore@bootlin.com> writes:
>> Alexis, thanks so much for working on this! This has been bugging me for
>> long but never found the time to investigate it.
>
> I'm glad to help, especially since I have the corresponding hardware. This
> warning was on my radar, and your last complaint about remaining sparse warnings
> in the wireless tree eventually triggered the action :)
Hah, at least there's one person who reads my complaints :)
I owe you a beer on this one, if we ever meet please remind me.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: wireless: orphan qfnfmac and plfxlc drivers?
2024-02-15 18:51 ` Jeff Johnson
@ 2024-02-16 9:37 ` Kalle Valo
2024-02-26 2:13 ` Igor Mitsyanko
0 siblings, 1 reply; 15+ messages in thread
From: Kalle Valo @ 2024-02-16 9:37 UTC (permalink / raw)
To: Jeff Johnson
Cc: Alexis Lothoré,
Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
Igor Mitsyanko, Sergey Matyukevich, kernel test robot,
Claudiu Beznea, srini.raju
Jeff Johnson <quic_jjohnson@quicinc.com> writes:
> On 2/15/2024 9:21 AM, Kalle Valo wrote:
>> <imitsyanko@quantenna.com>: host mail.quantenna.com[50.87.249.29] said: 550 No
>> Such User Here (in reply to RCPT TO command)
>
> Apparently Quantenna was acquired by ON Semi in 2019[1], and in 2022
> they closed it down[2]. Seems pretty abandoned to me.
Thanks Jeff. I do wonder is anyone even using qtnfmac and plfxlc? Maybe
we should just delete the drivers as nobody seems to care about them?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/2] wifi: wilc1000: remove AKM suite be32 conversion for external auth request
2024-02-15 14:13 ` [PATCH 2/2] wifi: wilc1000: remove AKM suite be32 conversion for external auth request Alexis Lothoré
@ 2024-02-21 18:54 ` Kalle Valo
0 siblings, 0 replies; 15+ messages in thread
From: Kalle Valo @ 2024-02-21 18:54 UTC (permalink / raw)
To: Alexis Lothoré
Cc: Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
linux-kernel, Igor Mitsyanko, Sergey Matyukevich,
Alexis Lothoré,
kernel test robot
Alexis Lothoré <alexis.lothore@bootlin.com> wrote:
> The driver currently raises the following sparse warning:
> [...] cfg80211.c:360:42: warning: incorrect type in assignment (different
> base types)
> [...] cfg80211.c:360:42: expected unsigned int key_mgmt_suite
> [...] cfg80211.c:360:42: got restricted __be32 [usertype]
> CHECK drivers/net/wireless/microchip/wilc1000/netdev.c
>
> This conversion was needed because historically the external supplicant
> (observed with wpa_supplicant) expects AKM suite as big endian in
> NL80211_CMD_EXTERNAL_AUTH message when the AKM suite is WLAN_AKM_SUITE_SAE.
> This is not needed anymore:
> - new (to be released) versions of wpa_supplicant now reads it in host
> endian _while_ keeping compatibility for older drivers
> - for new drivers used with current/old wpa_supplicant, this conversion has
> been added to nl80211 to force big endian when the AKM suite is
> WLAN_AKM_SUITE_SAE
>
> Remove this not-needed-anymore conversion to fix the sparse warning.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202308290615.lUTIgqUl-lkp@intel.com/
> Tested-on: WILC1000 hwB SPI WILC_WIFI_FW_REL_16_1-13452
> Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
Patch applied to wireless-next.git, thanks.
00413dd36414 wifi: wilc1000: remove AKM suite be32 conversion for external auth request
--
https://patchwork.kernel.org/project/linux-wireless/patch/20240215-nl80211_fix_akm_suites_endianness-v1-2-57e902632f9d@bootlin.com/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: wireless: orphan qfnfmac and plfxlc drivers?
2024-02-16 9:37 ` Kalle Valo
@ 2024-02-26 2:13 ` Igor Mitsyanko
2024-02-27 14:00 ` Kalle Valo
0 siblings, 1 reply; 15+ messages in thread
From: Igor Mitsyanko @ 2024-02-26 2:13 UTC (permalink / raw)
To: Kalle Valo, Jeff Johnson
Cc: Alexis Lothoré,
Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
Sergey Matyukevich, kernel test robot, Claudiu Beznea,
srini.raju, i.mitsyanko, krystal.heaton
On 2/16/24 01:37, Kalle Valo wrote:
> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>
>> On 2/15/2024 9:21 AM, Kalle Valo wrote:
>>> <imitsyanko@quantenna.com>: host mail.quantenna.com[50.87.249.29] said: 550 No
>>> Such User Here (in reply to RCPT TO command)
>> Apparently Quantenna was acquired by ON Semi in 2019[1], and in 2022
>> they closed it down[2]. Seems pretty abandoned to me.
> Thanks Jeff. I do wonder is anyone even using qtnfmac and plfxlc? Maybe
> we should just delete the drivers as nobody seems to care about them?
>
(replacing imitsyanko@quantenna.com with my personal email)
Hi Kalle, Jeff,
that's right, Quantenna division was shutdown by ON. To my knowledge no
users of qtnfmac drivers are left after that. I would think removing
driver altogether is the right approach. I'm not associated with ON
anymore so it's just my personal opinion.
CCing Krystal Heaton who I found on ON "contacts" web page (just in case
someone from ON wants to comment if removing qtnfmac driver from Linux
kernel is a concern).
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: wireless: orphan qfnfmac and plfxlc drivers?
2024-02-26 2:13 ` Igor Mitsyanko
@ 2024-02-27 14:00 ` Kalle Valo
2024-02-28 3:02 ` Stefan Lippers-Hollmann
0 siblings, 1 reply; 15+ messages in thread
From: Kalle Valo @ 2024-02-27 14:00 UTC (permalink / raw)
To: Igor Mitsyanko
Cc: Jeff Johnson, Alexis Lothoré,
Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
Sergey Matyukevich, kernel test robot, Claudiu Beznea,
srini.raju, krystal.heaton
Igor Mitsyanko <i.mitsyanko@gmail.com> writes:
> On 2/16/24 01:37, Kalle Valo wrote:
>> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>>
>>> On 2/15/2024 9:21 AM, Kalle Valo wrote:
>>>> <imitsyanko@quantenna.com>: host mail.quantenna.com[50.87.249.29] said: 550 No
>>>> Such User Here (in reply to RCPT TO command)
>>> Apparently Quantenna was acquired by ON Semi in 2019[1], and in 2022
>>> they closed it down[2]. Seems pretty abandoned to me.
>> Thanks Jeff. I do wonder is anyone even using qtnfmac and plfxlc? Maybe
>> we should just delete the drivers as nobody seems to care about them?
>>
> (replacing imitsyanko@quantenna.com with my personal email)
>
> Hi Kalle, Jeff,
>
> that's right, Quantenna division was shutdown by ON. To my knowledge
> no users of qtnfmac drivers are left after that. I would think
> removing driver altogether is the right approach. I'm not associated
> with ON anymore so it's just my personal opinion.
>
> CCing Krystal Heaton who I found on ON "contacts" web page (just in
> case someone from ON wants to comment if removing qtnfmac driver from
> Linux kernel is a concern).
Sad news but thanks for letting us know. So unless anyone else objects
the plan is to remove qtnfmac driver in the near future.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: wireless: orphan qfnfmac and plfxlc drivers?
2024-02-27 14:00 ` Kalle Valo
@ 2024-02-28 3:02 ` Stefan Lippers-Hollmann
0 siblings, 0 replies; 15+ messages in thread
From: Stefan Lippers-Hollmann @ 2024-02-28 3:02 UTC (permalink / raw)
To: Kalle Valo
Cc: Igor Mitsyanko, Jeff Johnson, Alexis Lothoré,
Johannes Berg, Ajay Singh, Thomas Petazzoni, linux-wireless,
Sergey Matyukevich, kernel test robot, Claudiu Beznea,
srini.raju, krystal.heaton
Hi
On 2024-02-27, Kalle Valo wrote:
> Igor Mitsyanko <i.mitsyanko@gmail.com> writes:
> > On 2/16/24 01:37, Kalle Valo wrote:
> >> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
> >>> On 2/15/2024 9:21 AM, Kalle Valo wrote:
[...]
> > that's right, Quantenna division was shutdown by ON. To my knowledge
> > no users of qtnfmac drivers are left after that. I would think
> > removing driver altogether is the right approach. I'm not associated
> > with ON anymore so it's just my personal opinion.
> >
> > CCing Krystal Heaton who I found on ON "contacts" web page (just in
> > case someone from ON wants to comment if removing qtnfmac driver from
> > Linux kernel is a concern).
>
> Sad news but thanks for letting us know. So unless anyone else objects
> the plan is to remove qtnfmac driver in the near future.
In theory, there would be quite a few /potential/ users for qtnfmac
and QT3840BC 'Topaz' (QSR1000) on a variety of early 802.11ac routers
and APs (particularly on ipq8064 SOCs[0]), but as the required qtnfmac
firmware for these chipsets has never been published, there is
no wireless support for (the typically 5 GHz-) "Topaz" wlan chipset
on those devices at all.
The devices are out there (albeit not the most common- or long-lived
combination), but they never had wireless support in the first place
(first because qtnfmac didn't support QSR1000, after that was
rectified[1], because the required firmware for qtnfmac/ QSR1000 has
never been published). It's a bit sad that these never got supported.
Regards
Stefan Lippers-Hollmann
--
[0] e.g. Linksys E8350, Linksys E8400, Netgear R7500,
ZyXEL NBG6816 (Armor Z1), ZyXEL WAP6806 (Armor X1) (on mt7621)
[Disclaimer: I don't own any of these myself, but I regularly
see requests for Topaz wireless support on the r7500 and e8350]
[1] Tue Oct 16 10:23:58 2018 +0000, qtnfmac: add support for Topaz
chipsets
e401fa25cfa23df8b17960a656ff11f49facae84
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2024-02-28 3:03 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-15 14:13 [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
2024-02-15 14:13 ` [PATCH 1/2] wifi: nl80211: force WLAN_AKM_SUITE_SAE in " Alexis Lothoré
2024-02-15 16:58 ` Kalle Valo
2024-02-16 8:21 ` Alexis Lothoré
2024-02-16 9:34 ` Kalle Valo
2024-02-15 14:13 ` [PATCH 2/2] wifi: wilc1000: remove AKM suite be32 conversion for external auth request Alexis Lothoré
2024-02-21 18:54 ` Kalle Valo
2024-02-15 15:50 ` [PATCH 0/2] wifi: nl80211/wilc1000: force WLAN_AKM_SUITE_SAE to big endian in NL80211_CMD_EXTERNAL_AUTH Alexis Lothoré
2024-02-15 17:06 ` wireless: orphan qfnfmac and plfxlc drivers? Kalle Valo
2024-02-15 17:21 ` Kalle Valo
2024-02-15 18:51 ` Jeff Johnson
2024-02-16 9:37 ` Kalle Valo
2024-02-26 2:13 ` Igor Mitsyanko
2024-02-27 14:00 ` Kalle Valo
2024-02-28 3:02 ` Stefan Lippers-Hollmann
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.