mac80211: fix deadlock in driver-managed RX BA session start
authorJohannes Berg <johannes.berg@intel.com>
Wed, 6 Sep 2017 13:01:42 +0000 (15:01 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 6 Sep 2017 13:22:02 +0000 (15:22 +0200)
commitbde59c475e0883e4c4294bcd9b9c7e08ae18c828
tree2bc99ff875407d538dd1c6c8f2cced8878528ad9
parent98e93e968e4947cd71c2eb69e323682daa453ee7
mac80211: fix deadlock in driver-managed RX BA session start

When an RX BA session is started by the driver, and it has to tell
mac80211 about it, the corresponding bit in tid_rx_manage_offl gets
set and the BA session work is scheduled. Upon testing this bit, it
will call __ieee80211_start_rx_ba_session(), thus deadlocking as it
already holds the ampdu_mlme.mtx, which that acquires again.

Fix this by adding ___ieee80211_start_rx_ba_session(), a version of
the function that requires the mutex already held.

Cc: stable@vger.kernel.org
Fixes: 699cb58c8a52 ("mac80211: manage RX BA session offload without SKB queue")
Reported-by: Matteo Croce <mcroce@redhat.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/agg-rx.c
net/mac80211/ht.c
net/mac80211/ieee80211_i.h