On 2020-04-24 16:59, Sergey Matyukevich wrote:
Hello Johannes, Tamizh, and all
This patch series contains several minor fixes and enhancements for TID
specific configuration functionality. The first three patches include
minor fixes and TID specific AMSDU configuration.
The 4th patch is somewhat controversial, so the series is marked as
RFC.
This patch simplifies current override logic. It suggests to make no
difference between 'specific peer' and 'all peers' cases and to apply
new TID configuration immediately after resetting the previous one.
The 5th patch enables access to new kernel functionality from iw tool.
Note that nl80211.h changes are not included into iw. So to make this
change work in iw , header should be updated upfront, including new
AMSDU attribute from this patch series.
Regards,
Sergey
v1 -> v2
- add policy for new AMSDU attribute
- add patch for iw with support for per-tid per-node configuration
Sergey Matyukevich (5):
cfg80211: fix mask type in cfg80211_tid_cfg structure
mac80211: fix variable names in TID config methods
cfg80211: add support for TID specific AMSDU configuration
nl80211: simplify peer specific TID configuration
Patches looks good to me
Reviewed-by: Tamizh Chelvam <tamizhr@xxxxxxxxxxxxxx>
iw: add TID specific configuration command
Looks neat to me :)
Reviewed-by: Tamizh Chelvam <tamizhr@xxxxxxxxxxxxxx>
include/net/cfg80211.h | 6 ++++--
include/uapi/linux/nl80211.h | 20 +++++++++++---------
net/mac80211/cfg.c | 6 +++---
net/mac80211/driver-ops.h | 4 ++--
net/wireless/nl80211.c | 13 +++++++++----
5 files changed, 29 insertions(+), 20 deletions(-)