mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-09-21 20:22:13 +08:00
wifi: mac80211: fix off-by-one link setting
[ Upstream commitcf08e29db7
] The convention for find_first_bit() is 0-based, while ffs() is 1-based, so this is now off-by-one. I cannot reproduce the gcc-9 problem, but since the -1 is now removed, I'm hoping it will still avoid the original issue. Reported-by: Alexander Lobakin <alexandr.lobakin@intel.com> Fixes:1d8d4af434
("wifi: mac80211: avoid u32_encode_bits() warning") Signed-off-by: Johannes Berg <johannes.berg@intel.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
2a0200daec
commit
558496d79b
@ -4395,7 +4395,7 @@ static void ieee80211_mlo_multicast_tx(struct net_device *dev,
|
||||
u32 ctrl_flags = IEEE80211_TX_CTRL_MCAST_MLO_FIRST_TX;
|
||||
|
||||
if (hweight16(links) == 1) {
|
||||
ctrl_flags |= u32_encode_bits(find_first_bit(&links, 16) - 1,
|
||||
ctrl_flags |= u32_encode_bits(__ffs(links),
|
||||
IEEE80211_TX_CTRL_MLO_LINK);
|
||||
|
||||
__ieee80211_subif_start_xmit(skb, sdata->dev, 0, ctrl_flags,
|
||||
|
Loading…
Reference in New Issue
Block a user