qcacmn: Rate limit excessive logs in WBM error path
Rate limit excessive logging prints in WBM error path Change-Id: Ifb69959b78740bf4a0fca7b7611a2da0ef740413 CRs-Fixed: 2908240
This commit is contained in:

committed by
snandini

parent
ca2545b8a1
commit
1759c5d6cb
@@ -1019,13 +1019,13 @@ uint8_t dp_rx_process_invalid_peer(struct dp_soc *soc, qdf_nbuf_t mpdu,
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (qdf_nbuf_len(mpdu) < sizeof(struct ieee80211_frame)) {
|
if (qdf_nbuf_len(mpdu) < sizeof(struct ieee80211_frame)) {
|
||||||
dp_rx_err("%pK: Invalid nbuf length", soc);
|
dp_rx_info_rl("%pK: Invalid nbuf length", soc);
|
||||||
goto free;
|
goto free;
|
||||||
}
|
}
|
||||||
|
|
||||||
pdev = dp_get_pdev_for_lmac_id(soc, mac_id);
|
pdev = dp_get_pdev_for_lmac_id(soc, mac_id);
|
||||||
if (!pdev) {
|
if (!pdev) {
|
||||||
dp_rx_err("%pK: PDEV not found", soc);
|
dp_rx_info_rl("%pK: PDEV not found", soc);
|
||||||
goto free;
|
goto free;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -1040,7 +1040,7 @@ uint8_t dp_rx_process_invalid_peer(struct dp_soc *soc, qdf_nbuf_t mpdu,
|
|||||||
qdf_spin_unlock_bh(&pdev->vdev_list_lock);
|
qdf_spin_unlock_bh(&pdev->vdev_list_lock);
|
||||||
|
|
||||||
if (!vdev) {
|
if (!vdev) {
|
||||||
dp_rx_err("%pK: VDEV not found", soc);
|
dp_rx_info_rl("%pK: VDEV not found", soc);
|
||||||
goto free;
|
goto free;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@@ -76,6 +76,8 @@
|
|||||||
#define dp_rx_warn(params...) QDF_TRACE_WARN(QDF_MODULE_ID_DP_RX, params)
|
#define dp_rx_warn(params...) QDF_TRACE_WARN(QDF_MODULE_ID_DP_RX, params)
|
||||||
#define dp_rx_info(params...) \
|
#define dp_rx_info(params...) \
|
||||||
__QDF_TRACE_FL(QDF_TRACE_LEVEL_INFO_HIGH, QDF_MODULE_ID_DP_RX, ## params)
|
__QDF_TRACE_FL(QDF_TRACE_LEVEL_INFO_HIGH, QDF_MODULE_ID_DP_RX, ## params)
|
||||||
|
#define dp_rx_info_rl(params...) \
|
||||||
|
__QDF_TRACE_RL(QDF_TRACE_LEVEL_INFO_HIGH, QDF_MODULE_ID_DP_RX, ## params)
|
||||||
#define dp_rx_debug(params...) QDF_TRACE_DEBUG(QDF_MODULE_ID_DP_RX, params)
|
#define dp_rx_debug(params...) QDF_TRACE_DEBUG(QDF_MODULE_ID_DP_RX, params)
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@@ -38,6 +38,8 @@
|
|||||||
#define dp_rx_err_warn(params...) QDF_TRACE_WARN(QDF_MODULE_ID_DP_RX_ERROR, params)
|
#define dp_rx_err_warn(params...) QDF_TRACE_WARN(QDF_MODULE_ID_DP_RX_ERROR, params)
|
||||||
#define dp_rx_err_info(params...) \
|
#define dp_rx_err_info(params...) \
|
||||||
__QDF_TRACE_FL(QDF_TRACE_LEVEL_INFO_HIGH, QDF_MODULE_ID_DP_RX_ERROR, ## params)
|
__QDF_TRACE_FL(QDF_TRACE_LEVEL_INFO_HIGH, QDF_MODULE_ID_DP_RX_ERROR, ## params)
|
||||||
|
#define dp_rx_err_info_rl(params...) \
|
||||||
|
__QDF_TRACE_RL(QDF_TRACE_LEVEL_INFO_HIGH, QDF_MODULE_ID_DP_RX_ERROR, ## params)
|
||||||
#define dp_rx_err_debug(params...) QDF_TRACE_DEBUG(QDF_MODULE_ID_DP_RX_ERROR, params)
|
#define dp_rx_err_debug(params...) QDF_TRACE_DEBUG(QDF_MODULE_ID_DP_RX_ERROR, params)
|
||||||
|
|
||||||
#ifndef QCA_HOST_MODE_WIFI_DISABLED
|
#ifndef QCA_HOST_MODE_WIFI_DISABLED
|
||||||
@@ -1359,8 +1361,8 @@ dp_rx_process_rxdma_err(struct dp_soc *soc, qdf_nbuf_t nbuf,
|
|||||||
|
|
||||||
vdev = peer->vdev;
|
vdev = peer->vdev;
|
||||||
if (!vdev) {
|
if (!vdev) {
|
||||||
dp_rx_err_err("%pK: INVALID vdev %pK OR osif_rx", soc,
|
dp_rx_err_info_rl("%pK: INVALID vdev %pK OR osif_rx", soc,
|
||||||
vdev);
|
vdev);
|
||||||
/* Drop & free packet */
|
/* Drop & free packet */
|
||||||
qdf_nbuf_free(nbuf);
|
qdf_nbuf_free(nbuf);
|
||||||
DP_STATS_INC(soc, rx.err.invalid_vdev, 1);
|
DP_STATS_INC(soc, rx.err.invalid_vdev, 1);
|
||||||
|
Reference in New Issue
Block a user