mbox series

[v6,0/3] Additional processing in NL80211_CMD_SET_BEACON

Message ID 20220518203922.26417-1-quic_alokad@quicinc.com
Headers show
Series Additional processing in NL80211_CMD_SET_BEACON | expand

Message

Aloka Dixit May 18, 2022, 8:39 p.m. UTC 
FILS discovery and unsolicited broadcast probe response transmissions
are configured as part of NL80211_CMD_START_AP, however both stop
after userspace uses the NL80211_CMD_SET_BEACON command as these
attributes are not processed as part of this command.

- Modify the local variable in nl80211_set_beacon() and input parameter
to rdev_change_beacon() from type struct cfg80211_beacon_data to
type struct cfg80211_ap_settings to support the new processing.
- Modify ieee80211_change_beacon() to reflect the new input parameter type.
- Modify driver specific functions pointed by change_beacon to
reflect the new input parameter type.
- Add the missing implementation in nl80211 and mac80211 to process
FILS discovery and unsolicited broadcast probe response configuration.

Aloka Dixit (3):
  cfg80211: modify prototype for change_beacon
  nl80211: additional processing in NL80211_CMD_SET_BEACON
  mac80211: additional processing in ieee80211_change_beacon

 drivers/net/wireless/ath/ath6kl/cfg80211.c    |  4 +-
 drivers/net/wireless/ath/wil6210/cfg80211.c   |  3 +-
 .../broadcom/brcm80211/brcmfmac/cfg80211.c    |  4 +-
 .../net/wireless/marvell/mwifiex/cfg80211.c   |  3 +-
 .../wireless/microchip/wilc1000/cfg80211.c    |  4 +-
 .../net/wireless/quantenna/qtnfmac/cfg80211.c |  4 +-
 include/net/cfg80211.h                        |  2 +-
 net/mac80211/cfg.c                            | 34 +++++++++---
 net/wireless/nl80211.c                        | 28 ++++++++--
 net/wireless/rdev-ops.h                       |  2 +-
 net/wireless/trace.h                          | 52 +++++++++++--------
 11 files changed, 94 insertions(+), 46 deletions(-)


base-commit: 9335156ac0e174721921c404bd173526c8509124

Comments

Johannes Berg July 1, 2022, 8:26 a.m. UTC | #1 
Hi Aloka,

I'm sorry, but the MLO work that ended up getting priority here has some
conflicts with this, and I'm not sure I'm comfortable rebasing this
myself, I also don't know how to test it.

Could you rebase this please?

Thanks,
johannes