diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2023-08-27 14:05:28 +0300 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2023-09-11 12:32:39 +0200 |
| commit | 428e8976a15f849ad92b1c1e38dda2a684350ff7 (patch) | |
| tree | 6a8b0f358f7e9b4c21e2f2ffc2294b6a375bd28e /net/mac80211/sta_info.c | |
| parent | 8107807891eae9d9de67ff35a40d09354ca265ec (diff) | |
| download | linux-428e8976a15f849ad92b1c1e38dda2a684350ff7.tar.gz linux-428e8976a15f849ad92b1c1e38dda2a684350ff7.tar.bz2 linux-428e8976a15f849ad92b1c1e38dda2a684350ff7.zip | |
wifi: mac80211: fix # of MSDU in A-MSDU calculation
During my refactoring I wanted to get rid of the switch,
but replaced it with the wrong calculation. Fix that.
Fixes: 175ad2ec89fe ("wifi: mac80211: limit A-MSDU subframes for client too")
Reported-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20230827135854.51bf1b8b0adb.Iffbd337fdad2b86ae12f5a39c69fb82b517f7486@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/sta_info.c')
| -rw-r--r-- | net/mac80211/sta_info.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/mac80211/sta_info.c b/net/mac80211/sta_info.c index 0c59ab9f9aaa..7243c6aa2161 100644 --- a/net/mac80211/sta_info.c +++ b/net/mac80211/sta_info.c @@ -2986,7 +2986,7 @@ void ieee80211_sta_set_max_amsdu_subframes(struct sta_info *sta, WLAN_EXT_CAPA9_MAX_MSDU_IN_AMSDU_MSB) << 1; if (val) - sta->sta.max_amsdu_subframes = 4 << val; + sta->sta.max_amsdu_subframes = 4 << (4 - val); } #ifdef CONFIG_LOCKDEP |