ath11k: Fix rx_filter flags setting for per peer rx_stats
authorMaharaja Kennadyrajan <mkenna@codeaurora.org>
Fri, 10 Apr 2020 17:06:45 +0000 (22:36 +0530)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 15 Apr 2020 08:00:29 +0000 (11:00 +0300)
Rx_filter flags are set with default filter flags during
wifi up/down sequence even though the 'ext_rx_stats' debugfs
is enabled as 1. So, that we are not getting proper per peer
rx_stats.
Hence, fixing this by setting the missing rx_filter when
ext_rx_stats is already set/enabled.

Signed-off-by: Maharaja Kennadyrajan <mkenna@codeaurora.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1586538405-16226-3-git-send-email-mkenna@codeaurora.org
drivers/net/wireless/ath/ath11k/core.h
drivers/net/wireless/ath/ath11k/debug.c
drivers/net/wireless/ath/ath11k/debug.h
drivers/net/wireless/ath/ath11k/mac.c

index b4c3e04..a8ef95f 100644 (file)
@@ -392,6 +392,7 @@ struct ath11k_debug {
        u32 pktlog_mode;
        u32 pktlog_peer_valid;
        u8 pktlog_peer_addr[ETH_ALEN];
+       u32 rx_filter;
 };
 
 struct ath11k_per_peer_tx_stats {
index 825e7ba..a2e3dfe 100644 (file)
@@ -698,6 +698,8 @@ static ssize_t ath11k_write_extd_rx_stats(struct file *file,
                tlv_filter = ath11k_mac_mon_status_filter_default;
        }
 
+       ar->debug.rx_filter = tlv_filter.rx_filter;
+
        ring_id = ar->dp.rx_mon_status_refill_ring.refill_buf_ring.ring_id;
        ret = ath11k_dp_tx_htt_rx_filter_setup(ar->ab, ring_id, ar->dp.mac_id,
                                               HAL_RXDMA_MONITOR_STATUS,
index 4a3ff82..45454fc 100644 (file)
@@ -188,6 +188,11 @@ static inline int ath11k_debug_is_extd_rx_stats_enabled(struct ath11k *ar)
        return ar->debug.extd_rx_stats;
 }
 
+static inline int ath11k_debug_rx_filter(struct ath11k *ar)
+{
+       return ar->debug.rx_filter;
+}
+
 void ath11k_sta_add_debugfs(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
                            struct ieee80211_sta *sta, struct dentry *dir);
 void
@@ -269,6 +274,11 @@ static inline bool ath11k_debug_is_pktlog_peer_valid(struct ath11k *ar, u8 *addr
        return false;
 }
 
+static inline int ath11k_debug_rx_filter(struct ath11k *ar)
+{
+       return 0;
+}
+
 static inline void
 ath11k_accumulate_per_peer_tx_stats(struct ath11k_sta *arsta,
                                    struct ath11k_per_peer_tx_stats *peer_stats,
index 0834089..065b7d6 100644 (file)
@@ -3881,8 +3881,10 @@ static int ath11k_mac_config_mon_status_default(struct ath11k *ar, bool enable)
        struct htt_rx_ring_tlv_filter tlv_filter = {0};
        u32 ring_id;
 
-       if (enable)
+       if (enable) {
                tlv_filter = ath11k_mac_mon_status_filter_default;
+               tlv_filter.rx_filter = ath11k_debug_rx_filter(ar);
+       }
 
        ring_id = ar->dp.rx_mon_status_refill_ring.refill_buf_ring.ring_id;