mac80211: Fix Tx aggregation session tear down with ITXQs
authorIlan Peer <ilan.peer@intel.com>
Wed, 6 Feb 2019 11:17:21 +0000 (13:17 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 11 Feb 2019 14:50:56 +0000 (15:50 +0100)
commit6157ca0d6bfe437691b1e98a62e2efe12b6714da
tree9cc5020354d597ad9222b53f580c0a9d949da15a
parent1fc9b7253382ce1a83d9a3e63e88d656eb63f263
mac80211: Fix Tx aggregation session tear down with ITXQs

When mac80211 requests the low level driver to stop an ongoing
Tx aggregation, the low level driver is expected to call
ieee80211_stop_tx_ba_cb_irqsafe() to indicate that it is ready
to stop the session. The callback in turn schedules a worker
to complete the session tear down, which in turn also handles
the relevant state for the intermediate Tx queue.

However, as this flow in asynchronous, the intermediate queue
should be stopped and not continue servicing frames, as in
such a case frames that are dequeued would be marked as part
of an aggregation, although the aggregation is already been
stopped.

Fix this by stopping the intermediate Tx queue, before
calling the low level driver to stop the Tx aggregation.

Signed-off-by: Ilan Peer <ilan.peer@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/agg-tx.c