From e4239809d9394bb8129592698ee1d816693aba16 Mon Sep 17 00:00:00 2001 From: Yun Park Date: Tue, 9 Jan 2018 11:01:40 -0800 Subject: [PATCH] qcacld-3.0: Cleanup function entry/exit logs from HDD IPA module Move module init/deinit and function entry/exit logs to DEBUG level, and keep only minimum logs in kernel log buffer. Change-Id: Ia9fe82934638683079d308acfc9e7014e1d1a0e3 CRs-Fixed: 2169416 --- core/dp/htt/htt.c | 4 +-- core/hdd/src/wlan_hdd_ipa.c | 46 ++++++++++++++-------------- core/hdd/src/wlan_hdd_softap_tx_rx.c | 4 +-- 3 files changed, 27 insertions(+), 27 deletions(-) diff --git a/core/dp/htt/htt.c b/core/dp/htt/htt.c index 16c99f3d661d..a3e792f86122 100644 --- a/core/dp/htt/htt.c +++ b/core/dp/htt/htt.c @@ -880,7 +880,7 @@ int htt_ipa_uc_attach(struct htt_pdev_t *pdev) */ void htt_ipa_uc_detach(struct htt_pdev_t *pdev) { - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, "%s: enter", + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG, "%s: enter", __func__); /* TX IPA micro controller detach */ @@ -889,7 +889,7 @@ void htt_ipa_uc_detach(struct htt_pdev_t *pdev) /* RX IPA micro controller detach */ htt_rx_ipa_uc_detach(pdev); - QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_INFO, "%s: exit", + QDF_TRACE(QDF_MODULE_ID_HTT, QDF_TRACE_LEVEL_DEBUG, "%s: exit", __func__); } diff --git a/core/hdd/src/wlan_hdd_ipa.c b/core/hdd/src/wlan_hdd_ipa.c index 305693caf742..026f40f2053d 100644 --- a/core/hdd/src/wlan_hdd_ipa.c +++ b/core/hdd/src/wlan_hdd_ipa.c @@ -1885,7 +1885,7 @@ static int hdd_ipa_uc_enable_pipes(struct hdd_ipa_priv *hdd_ipa) void *soc = cds_get_context(QDF_MODULE_ID_SOC); struct ol_txrx_pdev_t *pdev = cds_get_context(QDF_MODULE_ID_TXRX); - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (!hdd_ipa->ipa_pipes_down) { /* @@ -1911,7 +1911,7 @@ static int hdd_ipa_uc_enable_pipes(struct hdd_ipa_priv *hdd_ipa) cdp_ipa_enable_autonomy(soc, (struct cdp_pdev *)pdev); end: - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit: ipa_pipes_down=%d", + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit: ipa_pipes_down=%d", hdd_ipa->ipa_pipes_down); return result; @@ -1929,7 +1929,7 @@ static int hdd_ipa_uc_disable_pipes(struct hdd_ipa_priv *hdd_ipa) struct ol_txrx_pdev_t *pdev = cds_get_context(QDF_MODULE_ID_TXRX); int result = 0; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (hdd_ipa->ipa_pipes_down) { /* @@ -1954,7 +1954,7 @@ static int hdd_ipa_uc_disable_pipes(struct hdd_ipa_priv *hdd_ipa) hdd_ipa->ipa_pipes_down = true; end: - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit: ipa_pipes_down=%d", + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit: ipa_pipes_down=%d", hdd_ipa->ipa_pipes_down); return result; } @@ -1969,7 +1969,7 @@ static int hdd_ipa_uc_handle_first_con(struct hdd_ipa_priv *hdd_ipa) { struct hdd_context *hdd_ctx = hdd_ipa->hdd_ctx; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); hdd_ipa->activated_fw_pipe = 0; hdd_ipa->resource_loading = true; @@ -2006,7 +2006,7 @@ static int hdd_ipa_uc_handle_first_con(struct hdd_ipa_priv *hdd_ipa) hdd_ipa->resource_loading = false; } - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit"); return 0; } @@ -2021,7 +2021,7 @@ static void hdd_ipa_uc_handle_last_discon(struct hdd_ipa_priv *hdd_ipa) void *soc = cds_get_context(QDF_MODULE_ID_SOC); void *pdev = cds_get_context(QDF_MODULE_ID_TXRX); - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (!pdev) { HDD_IPA_LOG(QDF_TRACE_LEVEL_ERROR, "txrx context is NULL"); @@ -2036,7 +2036,7 @@ static void hdd_ipa_uc_handle_last_discon(struct hdd_ipa_priv *hdd_ipa) HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "Disable FW TX PIPE"); cdp_ipa_set_active(soc, (struct cdp_pdev *)pdev, false, true); - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit: IPA WDI Pipes deactivated"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit: IPA WDI Pipes deactivated"); } /** @@ -3197,7 +3197,7 @@ int hdd_ipa_uc_ol_deinit(struct hdd_context *hdd_ctx) int ret = 0; QDF_STATUS status; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (!hdd_ipa_uc_is_enabled(hdd_ctx)) return ret; @@ -3221,7 +3221,7 @@ int hdd_ipa_uc_ol_deinit(struct hdd_context *hdd_ctx) hdd_ipa_cleanup_pending_event(hdd_ipa); qdf_mutex_release(&hdd_ipa->ipa_lock); - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit: ret=%d", ret); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit: ret=%d", ret); return ret; } @@ -3240,7 +3240,7 @@ static void __hdd_ipa_uc_force_pipe_shutdown(struct hdd_context *hdd_ctx) { struct hdd_ipa_priv *hdd_ipa; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (!hdd_ipa_is_enabled(hdd_ctx) || !hdd_ctx->hdd_ipa) return; @@ -3255,7 +3255,7 @@ static void __hdd_ipa_uc_force_pipe_shutdown(struct hdd_context *hdd_ctx) "IPA pipes are down, do nothing"); } - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit"); } /** @@ -3353,7 +3353,7 @@ static int hdd_ipa_uc_disconnect_client(struct hdd_adapter *adapter) int ret = 0; int i; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); for (i = 0; i < WLAN_MAX_STA_COUNT; i++) { if (qdf_is_macaddr_broadcast(&adapter->sta_info[i].sta_mac)) continue; @@ -3365,7 +3365,7 @@ static int hdd_ipa_uc_disconnect_client(struct hdd_adapter *adapter) hdd_ipa->sap_num_connected_sta--; } } - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit: sap_num_connected_sta=%d", + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit: sap_num_connected_sta=%d", hdd_ipa->sap_num_connected_sta); return ret; @@ -3384,12 +3384,12 @@ static int hdd_ipa_uc_disconnect_ap(struct hdd_adapter *adapter) { int ret = 0; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (adapter->ipa_context) { hdd_ipa_uc_send_evt(adapter, WLAN_AP_DISCONNECT, adapter->dev->dev_addr); } - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit"); return ret; } @@ -3408,14 +3408,14 @@ static int hdd_ipa_uc_disconnect_sta(struct hdd_adapter *adapter) struct hdd_ipa_priv *hdd_ipa = ghdd_ipa; int ret = 0; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (hdd_ipa_uc_sta_is_enabled(hdd_ipa->hdd_ctx) && hdd_ipa->sta_connected) { sta_ctx = WLAN_HDD_GET_STATION_CTX_PTR(adapter); hdd_ipa_uc_send_evt(adapter, WLAN_STA_DISCONNECT, sta_ctx->conn_info.bssId.bytes); } - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit"); return ret; } @@ -3460,7 +3460,7 @@ static int __hdd_ipa_uc_ssr_deinit(void) struct hdd_ipa_iface_context *iface_context; struct hdd_context *hdd_ctx; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (!hdd_ipa) return 0; @@ -3503,7 +3503,7 @@ static int __hdd_ipa_uc_ssr_deinit(void) hdd_ipa->uc_op_work[idx].msg = NULL; } - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit"); return 0; } @@ -3541,7 +3541,7 @@ static int __hdd_ipa_uc_ssr_reinit(struct hdd_context *hdd_ctx) int i; struct hdd_ipa_iface_context *iface_context = NULL; - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "enter"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "enter"); if (!hdd_ipa || !hdd_ipa_uc_is_enabled(hdd_ctx)) return 0; @@ -3566,7 +3566,7 @@ static int __hdd_ipa_uc_ssr_reinit(struct hdd_context *hdd_ctx) hdd_ipa->uc_loaded = true; } - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit"); return 0; } @@ -5975,7 +5975,7 @@ static QDF_STATUS __hdd_ipa_init(struct hdd_context *hdd_ctx) init_completion(&hdd_ipa->ipa_resource_comp); - HDD_IPA_LOG(QDF_TRACE_LEVEL_INFO, "exit: success"); + HDD_IPA_LOG(QDF_TRACE_LEVEL_DEBUG, "exit: success"); return QDF_STATUS_SUCCESS; fail_create_sys_pipe: diff --git a/core/hdd/src/wlan_hdd_softap_tx_rx.c b/core/hdd/src/wlan_hdd_softap_tx_rx.c index 82f35f34e9b1..e1c86d0911ac 100644 --- a/core/hdd/src/wlan_hdd_softap_tx_rx.c +++ b/core/hdd/src/wlan_hdd_softap_tx_rx.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012-2017 The Linux Foundation. All rights reserved. + * Copyright (c) 2012-2018 The Linux Foundation. All rights reserved. * * Previously licensed under the ISC license by Qualcomm Atheros, Inc. * @@ -730,7 +730,7 @@ QDF_STATUS hdd_softap_rx_packet_cbk(void *context, qdf_nbuf_t rxBuf) skb->next = NULL; #ifdef QCA_WIFI_QCA6290 /* Debug code, remove later */ - QDF_TRACE(QDF_MODULE_ID_HDD_DATA, QDF_TRACE_LEVEL_INFO, + QDF_TRACE(QDF_MODULE_ID_HDD_DATA, QDF_TRACE_LEVEL_DEBUG, "%s: skb %pK skb->len %d\n", __func__, skb, skb->len); #endif