From c749f7d136839c12efcac05422fff51b851f9721 Mon Sep 17 00:00:00 2001 From: gaurank kathpalia Date: Thu, 20 Feb 2020 12:10:26 +0530 Subject: [PATCH] qcacld-3.0: Print the peer only when rx tx is going Currently the TDLS module prints the peer info every defined time interval which leads to too much of information even when no rx tx is ongoing with the peer which leads to excessive logging. Fix is to limit the logs and print only when there is activity going on. Change-Id: I6c1bf5a1ce8a858dfe895168178154a6c8b49c1f CRs-Fixed: 2625632 --- components/tdls/core/src/wlan_tdls_ct.c | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/components/tdls/core/src/wlan_tdls_ct.c b/components/tdls/core/src/wlan_tdls_ct.c index a7dc7bd1a2b2..48bcd6000397 100644 --- a/components/tdls/core/src/wlan_tdls_ct.c +++ b/components/tdls/core/src/wlan_tdls_ct.c @@ -759,13 +759,12 @@ static void tdls_ct_process_cap_supported(struct tdls_peer *curr_peer, struct tdls_vdev_priv_obj *tdls_vdev, struct tdls_soc_priv_obj *tdls_soc_obj) { - tdls_debug("tx %d rx %d thr.pkt %d/idle %d rssi %d thr.trig %d/tear %d", - curr_peer->tx_pkt, curr_peer->rx_pkt, - tdls_vdev->threshold_config.tx_packet_n, - tdls_vdev->threshold_config.idle_packet_n, - curr_peer->rssi, - tdls_vdev->threshold_config.rssi_trigger_threshold, - tdls_vdev->threshold_config.rssi_teardown_threshold); + if (curr_peer->rx_pkt || curr_peer->tx_pkt) + tdls_debug(QDF_MAC_ADDR_STR "link_status %d tdls_support %d tx %d rx %d rssi %d", + QDF_MAC_ADDR_ARRAY(curr_peer->peer_mac.bytes), + curr_peer->link_status, curr_peer->tdls_support, + curr_peer->tx_pkt, curr_peer->rx_pkt, + curr_peer->rssi); switch (curr_peer->link_status) { case TDLS_LINK_IDLE: @@ -805,9 +804,11 @@ static void tdls_ct_process_cap_unknown(struct tdls_peer *curr_peer, (!curr_peer->is_forced_peer)) return; - tdls_debug("threshold tx pkt = %d peer tx_pkt = %d & rx_pkt = %d ", - tdls_vdev->threshold_config.tx_packet_n, curr_peer->tx_pkt, - curr_peer->rx_pkt); + if (curr_peer->rx_pkt || curr_peer->tx_pkt) + tdls_debug(QDF_MAC_ADDR_STR "link_status %d tdls_support %d tx %d rx %d", + QDF_MAC_ADDR_ARRAY(curr_peer->peer_mac.bytes), + curr_peer->link_status, curr_peer->tdls_support, + curr_peer->tx_pkt, curr_peer->rx_pkt); if (!TDLS_IS_LINK_CONNECTED(curr_peer) && ((curr_peer->tx_pkt + curr_peer->rx_pkt) >= @@ -848,10 +849,6 @@ static void tdls_ct_process_peers(struct tdls_peer *curr_peer, struct tdls_vdev_priv_obj *tdls_vdev_obj, struct tdls_soc_priv_obj *tdls_soc_obj) { - tdls_debug(QDF_MAC_ADDR_STR " link_status %d tdls_support %d", - QDF_MAC_ADDR_ARRAY(curr_peer->peer_mac.bytes), - curr_peer->link_status, curr_peer->tdls_support); - switch (curr_peer->tdls_support) { case TDLS_CAP_SUPPORTED: tdls_ct_process_cap_supported(curr_peer, tdls_vdev_obj,