ath9k: fix handling of broken descriptors
authorFelix Fietkau <nbd@openwrt.org>
Sun, 7 Apr 2013 22:04:11 +0000 (00:04 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 10 Apr 2013 18:10:33 +0000 (14:10 -0400)
As the comment in ath_get_next_rx_buf indicates, if a descriptor with
the done bit set follows one with the done bit cleared, both descriptors
should be discarded, however the driver is not doing that yet.

To fix this, use the rs->rs_more flag as an indicator that the following
frame should be discarded. This also helps with the split buffer case:
if the first part of the frame is discarded, the following parts need to
be discarded as well, since they contain no valid header or usable data.

Signed-off-by: Felix Fietkau <nbd@openwrt.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath/ath9k/ath9k.h
drivers/net/wireless/ath/ath9k/recv.c

index a56b2416e2f90836943fe0d69c5c64466889859a..86d35726edb423cbc4e4c42810654df27b108f81 100644 (file)
@@ -311,6 +311,7 @@ struct ath_rx_edma {
 struct ath_rx {
        u8 defant;
        u8 rxotherant;
+       bool discard_next;
        u32 *rxlink;
        u32 num_pkts;
        unsigned int rxfilter;
index 3d0f02ddcf5bd6727915c361730c8563afe9bb4d..c593a3e0cd95f36cd5887433c5b33282114c3853 100644 (file)
@@ -727,6 +727,13 @@ static struct ath_buf *ath_get_next_rx_buf(struct ath_softc *sc,
                ret = ath9k_hw_rxprocdesc(ah, tds, &trs);
                if (ret == -EINPROGRESS)
                        return NULL;
+
+               /*
+                * mark descriptor as zero-length and set the 'more'
+                * flag to ensure that both buffers get discarded
+                */
+               rs->rs_datalen = 0;
+               rs->rs_more = true;
        }
 
        list_del(&bf->list);
@@ -933,14 +940,20 @@ static void ath9k_process_rssi(struct ath_common *common,
  * up the frame up to let mac80211 handle the actual error case, be it no
  * decryption key or real decryption error. This let us keep statistics there.
  */
-static int ath9k_rx_skb_preprocess(struct ath_common *common,
-                                  struct ieee80211_hw *hw,
+static int ath9k_rx_skb_preprocess(struct ath_softc *sc,
                                   struct ieee80211_hdr *hdr,
                                   struct ath_rx_status *rx_stats,
                                   struct ieee80211_rx_status *rx_status,
                                   bool *decrypt_error)
 {
-       struct ath_hw *ah = common->ah;
+       struct ieee80211_hw *hw = sc->hw;
+       struct ath_hw *ah = sc->sc_ah;
+       struct ath_common *common = ath9k_hw_common(ah);
+       bool discard_current = sc->rx.discard_next;
+
+       sc->rx.discard_next = rx_stats->rs_more;
+       if (discard_current)
+               return -EINVAL;
 
        /*
         * everything but the rate is checked here, the rate check is done
@@ -966,6 +979,7 @@ static int ath9k_rx_skb_preprocess(struct ath_common *common,
        if (rx_stats->rs_moreaggr)
                rx_status->flag |= RX_FLAG_NO_SIGNAL_VAL;
 
+       sc->rx.discard_next = false;
        return 0;
 }
 
@@ -1243,8 +1257,8 @@ int ath_rx_tasklet(struct ath_softc *sc, int flush, bool hp)
                        }
                }
 
-               retval = ath9k_rx_skb_preprocess(common, hw, hdr, &rs,
-                                                rxs, &decrypt_error);
+               retval = ath9k_rx_skb_preprocess(sc, hdr, &rs, rxs,
+                                                &decrypt_error);
                if (retval)
                        goto requeue_drop_frag;