mac80211: remove bogus skb vif assignment
authorFelix Fietkau <nbd@nbd.name>
Fri, 4 Nov 2016 09:27:53 +0000 (10:27 +0100)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 15 Nov 2016 13:37:21 +0000 (14:37 +0100)
The call to ieee80211_txq_enqueue overwrites the vif pointer with the
codel enqueue time, so setting it just before that call makes no sense.

Signed-off-by: Felix Fietkau <nbd@nbd.name>
Acked-by: Toke Høiland-Jørgensen <toke@toke.dk>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/tx.c

index d08a8492a846fc96c65e52ae28c281a2bd4fb9c9..fb73e86bdf41bc61f0f0e0a627ff3e16223066de 100644 (file)
@@ -1501,7 +1501,6 @@ static bool ieee80211_queue_skb(struct ieee80211_local *local,
                                struct sta_info *sta,
                                struct sk_buff *skb)
 {
-       struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb);
        struct fq *fq = &local->fq;
        struct ieee80211_vif *vif;
        struct txq_info *txqi;
@@ -1526,8 +1525,6 @@ static bool ieee80211_queue_skb(struct ieee80211_local *local,
        if (!txqi)
                return false;
 
-       info->control.vif = vif;
-
        spin_lock_bh(&fq->lock);
        ieee80211_txq_enqueue(local, txqi, skb);
        spin_unlock_bh(&fq->lock);