| Message ID | 20211129133248.83829-1-luca@coelho.fi |
|---|---|
| Headers | show |
| Series | cfg80211/mac80211 patches from our internal tree 2021-11-29 | expand |
On Mon, 2021-11-29 at 15:32 +0200, Luca Coelho wrote: > From: Johannes Berg <johannes.berg@intel.com> > > Add more logging in places that affect HT/VHT/HE state, so > things get easier to debug. > > Signed-off-by: Johannes Berg <johannes.berg@intel.com> > Signed-off-by: Luca Coelho <luciano.coelho@intel.com> > --- Sorry, there was a merge mistake in this patch and it broke compilation. I'll send v2 in a sec. -- Cheers, Luca.
Luca Coelho <luca@coelho.fi> writes: > From: Ilan Peer <ilan.peer@intel.com> > > The function cfg80211_reg_can_beacon_relax() expects wiphy > mutex to be held when it is being called. However, when > reg_leave_invalid_chans() is called the mutex is not held. > Fix it by acquiring the lock before calling the function. > > Fixes: a05829a7222e ("cfg80211: avoid holding the RTNL when calling the driver") > Signed-off-by: Ilan Peer <ilan.peer@intel.com> > Fixes: a05829a7222e ("cfg80211: avoid holding the RTNL when calling the driver") Duplicate Fixes tags.
On Mon, 2021-11-29 at 15:32 +0200, Luca Coelho wrote: > From: Ayala Beker <ayala.beker@intel.com> > > A non-collocated AP whose primary channel is not a PSC channel > may transmit a duplicated beacon on the corresponding PSC channel > in which it would indicate its true primary channel. > Use this inforamtion contained in the HE operation IE to determine > the primary channel of the AP. > In case of invalid infomration ignore it and use the channel > the frame was received on. > > Signed-off-by: Ayala Beker <ayala.beker@intel.com> > Signed-off-by: Luca Coelho <luciano.coelho@intel.com> > --- As you know already (but for the record), this is totally broken. There were some conflicts due to the refactor that happened in this function and I accidentally ran the wrong script to test compilation before sending this series out... :( V2 coming up soon. -- Cheers, Luca.
On Wed, 2021-12-01 at 15:47 +0200, Kalle Valo wrote: > Luca Coelho <luca@coelho.fi> writes: > > > From: Ilan Peer <ilan.peer@intel.com> > > > > The function cfg80211_reg_can_beacon_relax() expects wiphy > > mutex to be held when it is being called. However, when > > reg_leave_invalid_chans() is called the mutex is not held. > > Fix it by acquiring the lock before calling the function. > > > > Fixes: a05829a7222e ("cfg80211: avoid holding the RTNL when calling the driver") > > Signed-off-by: Ilan Peer <ilan.peer@intel.com> > > Fixes: a05829a7222e ("cfg80211: avoid holding the RTNL when calling the driver") > > Duplicate Fixes tags. Thanks for noticing! My script adds the tag automatically, but it doesn't check (yet) whether the tag was already there... I'll send v2 without it. -- Cheers, Luca.
From: Luca Coelho <luciano.coelho@intel.com> Hi, A bunch of patches with mac80211 and cfg80211 changes from our internal tree. Please review, though you have already reviewed (or even written!) most if not all of them. ;) Thanks! Cheers, Luca. Ayala Beker (1): cfg80211: Use the HE operation IE to determine a 6GHz BSS channel Ilan Peer (6): cfg80211: Add support for notifying association comeback mac80211: Notify cfg80211 about association comeback cfg80211: Fix order of enum nl80211_band_iftype_attr documentation mac80211: Remove a couple of obsolete TODO mac80211: Fix the size used for building probe request cfg80211: Acquire wiphy mutex on regulatory work Johannes Berg (7): mac80211: add more HT/VHT/HE state logging [BUGFIX] cfg80211: check fixed size before ieee80211_he_oper_size() mac80211: mark TX-during-stop for TX in in_reconfig mac80211: do drv_reconfig_complete() before restarting all cfg80211: simplify cfg80211_chandef_valid() mac80211: fix lookup when adding AddBA extension element mac80211: agg-tx: don't schedule_and_wake_txq() under sta->lock Mordechay Goodstein (1): mac80211: update channel context before station state Nathan Errera (1): mac80211: introduce channel switch disconnect function include/net/cfg80211.h | 12 +++++++ include/net/mac80211.h | 12 +++++++ include/uapi/linux/nl80211.h | 10 ++++-- net/mac80211/agg-rx.c | 5 +-- net/mac80211/agg-tx.c | 10 ++++-- net/mac80211/cfg.c | 14 +++++++- net/mac80211/driver-ops.h | 5 ++- net/mac80211/main.c | 13 +++----- net/mac80211/mlme.c | 54 +++++++++++++++++++++---------- net/mac80211/sta_info.c | 15 +++++---- net/mac80211/util.c | 16 +++++----- net/wireless/chan.c | 62 +++++++++++++++++++----------------- net/wireless/nl80211.c | 38 ++++++++++++++++++++++ net/wireless/reg.c | 8 +++-- net/wireless/scan.c | 48 +++++++++++++++++++++++++--- net/wireless/trace.h | 17 ++++++++++ 16 files changed, 256 insertions(+), 83 deletions(-)