ath9k: fix tx99 with monitor mode interface
authorFelix Fietkau <nbd@nbd.name>
Mon, 20 Aug 2018 09:37:51 +0000 (11:37 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 28 Aug 2018 13:45:40 +0000 (16:45 +0300)
Tx99 is typically configured via a monitor mode interface, which does
not get added to the driver as a vif. Since the code currently expects
a configured virtual interface for tx99, enabling tx99 via debugfs fails.
Since the vif is not needed anyway, remove all checks for it.

Signed-off-by: Felix Fietkau <nbd@nbd.name>
[kvalo@codeaurora.org: s/CPTCFG/CONFIG/]
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ath/ath9k/ath9k.h
drivers/net/wireless/ath/ath9k/main.c
drivers/net/wireless/ath/ath9k/tx99.c
drivers/net/wireless/ath/ath9k/xmit.c

index 21ba20981a80bfd4b96133c0b0a9ecd6c51f6c41..003e9fb456ac59e902c08ae95051453a6e88864e 100644 (file)
@@ -1074,7 +1074,6 @@ struct ath_softc {
 
        struct ath_spec_scan_priv spec_priv;
 
-       struct ieee80211_vif *tx99_vif;
        struct sk_buff *tx99_skb;
        bool tx99_state;
        s16 tx99_power;
index 1049773378f274e2f8c4ccf1050cb58f50858366..6ce4b9f1dcb44e0e7a2036e477793a61b6665d1d 100644 (file)
@@ -1251,15 +1251,10 @@ static int ath9k_add_interface(struct ieee80211_hw *hw,
        struct ath_vif *avp = (void *)vif->drv_priv;
        struct ath_node *an = &avp->mcast_node;
 
-       mutex_lock(&sc->mutex);
+       if (IS_ENABLED(CONFIG_ATH9K_TX99))
+               return -EOPNOTSUPP;
 
-       if (IS_ENABLED(CONFIG_ATH9K_TX99)) {
-               if (sc->cur_chan->nvifs >= 1) {
-                       mutex_unlock(&sc->mutex);
-                       return -EOPNOTSUPP;
-               }
-               sc->tx99_vif = vif;
-       }
+       mutex_lock(&sc->mutex);
 
        ath_dbg(common, CONFIG, "Attach a VIF of type: %d\n", vif->type);
        sc->cur_chan->nvifs++;
@@ -1342,7 +1337,6 @@ static void ath9k_remove_interface(struct ieee80211_hw *hw,
        ath9k_p2p_remove_vif(sc, vif);
 
        sc->cur_chan->nvifs--;
-       sc->tx99_vif = NULL;
        if (!ath9k_is_chanctx_enabled())
                list_del(&avp->list);
 
index ce50d8f5835e03cf22cbb7f19c2bb1bc0b61296c..9b05ffb68c34a1f1f89072bcc9c80b5f688ce1b7 100644 (file)
@@ -54,12 +54,6 @@ static struct sk_buff *ath9k_build_tx99_skb(struct ath_softc *sc)
        struct ieee80211_hdr *hdr;
        struct ieee80211_tx_info *tx_info;
        struct sk_buff *skb;
-       struct ath_vif *avp;
-
-       if (!sc->tx99_vif)
-               return NULL;
-
-       avp = (struct ath_vif *)sc->tx99_vif->drv_priv;
 
        skb = alloc_skb(len, GFP_KERNEL);
        if (!skb)
@@ -77,14 +71,11 @@ static struct sk_buff *ath9k_build_tx99_skb(struct ath_softc *sc)
        memcpy(hdr->addr2, hw->wiphy->perm_addr, ETH_ALEN);
        memcpy(hdr->addr3, hw->wiphy->perm_addr, ETH_ALEN);
 
-       hdr->seq_ctrl |= cpu_to_le16(avp->seq_no);
-
        tx_info = IEEE80211_SKB_CB(skb);
        memset(tx_info, 0, sizeof(*tx_info));
        rate = &tx_info->control.rates[0];
        tx_info->band = sc->cur_chan->chandef.chan->band;
        tx_info->flags = IEEE80211_TX_CTL_NO_ACK;
-       tx_info->control.vif = sc->tx99_vif;
        rate->count = 1;
        if (ah->curchan && IS_CHAN_HT(ah->curchan)) {
                rate->flags |= IEEE80211_TX_RC_MCS;
index 43b6c8508e4938f0dc95d0ac345044824d2fb7b2..66b6a8872c067d2443600c3a480c13e3944c0ac3 100644 (file)
@@ -2973,7 +2973,7 @@ int ath9k_tx99_send(struct ath_softc *sc, struct sk_buff *skb,
                return -EINVAL;
        }
 
-       ath_set_rates(sc->tx99_vif, NULL, bf);
+       ath_set_rates(NULL, NULL, bf);
 
        ath9k_hw_set_desc_link(sc->sc_ah, bf->bf_desc, bf->bf_daddr);
        ath9k_hw_tx99_start(sc->sc_ah, txctl->txq->axq_qnum);