qcacld-3.0: Reduce excessive kmsg logging in roaming path

Reduce excessive logging for roaming path to reduce latency.
Also remove function and line prints in roam debug related
logging.

Change-Id: Ida6fc55886c944126720f8bdd4b944765426ccc6
CRs-Fixed: 2847773
This commit is contained in:
Pragaspapthi Thilagaraj
2020-12-29 12:44:04 +05:30
committed by snandini
parent be88be800e
commit dd019ef5ed
6 changed files with 33 additions and 34 deletions

View File

@@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2018-2020 The Linux Foundation. All rights reserved. * Copyright (c) 2018-2021 The Linux Foundation. All rights reserved.
* *
* Permission to use, copy, modify, and/or distribute this software for * Permission to use, copy, modify, and/or distribute this software for
* any purpose with or without fee is hereby granted, provided that the * any purpose with or without fee is hereby granted, provided that the
@@ -180,7 +180,7 @@ tgt_mc_cp_stats_prepare_raw_peer_rssi(struct wlan_objmgr_psoc *psoc,
get_peer_rssi_cb = last_req->u.get_peer_rssi_cb; get_peer_rssi_cb = last_req->u.get_peer_rssi_cb;
if (!get_peer_rssi_cb) { if (!get_peer_rssi_cb) {
cp_stats_err("get_peer_rssi_cb is null"); cp_stats_debug("get_peer_rssi_cb is null");
return; return;
} }
@@ -213,8 +213,8 @@ tgt_mc_cp_stats_prepare_raw_peer_rssi(struct wlan_objmgr_psoc *psoc,
peer = wlan_objmgr_get_peer(psoc, last_req->pdev_id, peer = wlan_objmgr_get_peer(psoc, last_req->pdev_id,
mac_addr, WLAN_CP_STATS_ID); mac_addr, WLAN_CP_STATS_ID);
if (!peer) { if (!peer) {
cp_stats_err("peer["QDF_MAC_ADDR_FMT"] is null", cp_stats_debug("peer[" QDF_MAC_ADDR_FMT "] is null",
QDF_MAC_ADDR_REF(mac_addr)); QDF_MAC_ADDR_REF(mac_addr));
goto end; goto end;
} }
@@ -505,9 +505,8 @@ static void tgt_mc_cp_stats_extract_peer_stats(struct wlan_objmgr_psoc *psoc,
/* no matched peer */ /* no matched peer */
if (!QDF_IS_ADDR_BROADCAST(last_req.peer_mac_addr) && if (!QDF_IS_ADDR_BROADCAST(last_req.peer_mac_addr) &&
selected == ev->num_peer_stats) { selected == ev->num_peer_stats)
cp_stats_rl_err("peer not found for stats"); cp_stats_debug("peer not found for stats");
}
extd2_stats: extd2_stats:

View File

@@ -2865,8 +2865,8 @@ hdd_add_latency_critical_client(struct hdd_adapter *adapter,
if (adapter->device_mode == QDF_STA_MODE) if (adapter->device_mode == QDF_STA_MODE)
qdf_atomic_inc(&hdd_ctx->num_latency_critical_clients); qdf_atomic_inc(&hdd_ctx->num_latency_critical_clients);
hdd_info("Adding latency critical connection for vdev %d", hdd_debug("Adding latency critical connection for vdev %d",
adapter->vdev_id); adapter->vdev_id);
cdp_vdev_inform_ll_conn(cds_get_context(QDF_MODULE_ID_SOC), cdp_vdev_inform_ll_conn(cds_get_context(QDF_MODULE_ID_SOC),
adapter->vdev_id, adapter->vdev_id,
CDP_VDEV_LL_CONN_ADD); CDP_VDEV_LL_CONN_ADD);

View File

@@ -18371,7 +18371,7 @@ int hdd_get_rssi_snr_by_bssid(struct hdd_adapter *adapter, const uint8_t *bssid,
roam_profile, bssid, rssi, snr, roam_profile, bssid, rssi, snr,
adapter->vdev_id); adapter->vdev_id);
if (QDF_STATUS_SUCCESS != status) { if (QDF_STATUS_SUCCESS != status) {
hdd_warn("sme_get_rssi_snr_by_bssid failed"); hdd_debug("sme_get_rssi_snr_by_bssid failed");
return -EINVAL; return -EINVAL;
} }

View File

@@ -2528,8 +2528,6 @@ void csr_init_occupied_channels_list(struct mac_context *mac_ctx,
&mac_ctx->scan.occupiedChannels[sessionId], &mac_ctx->scan.occupiedChannels[sessionId],
true); true);
list = ucfg_scan_get_result(pdev, filter); list = ucfg_scan_get_result(pdev, filter);
if (list)
sme_debug("num_entries %d", qdf_list_size(list));
if (!list || (list && !qdf_list_size(list))) { if (!list || (list && !qdf_list_size(list))) {
goto err; goto err;
} }

View File

@@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2013-2020 The Linux Foundation. All rights reserved. * Copyright (c) 2013-2021 The Linux Foundation. All rights reserved.
* *
* Permission to use, copy, modify, and/or distribute this software for * Permission to use, copy, modify, and/or distribute this software for
* any purpose with or without fee is hereby granted, provided that the * any purpose with or without fee is hereby granted, provided that the
@@ -1877,7 +1877,7 @@ wma_rso_print_trigger_info(struct wmi_roam_trigger_info *data, uint8_t vdev_id)
wma_get_trigger_detail_str(data, buf); wma_get_trigger_detail_str(data, buf);
mlme_get_converted_timestamp(data->timestamp, time); mlme_get_converted_timestamp(data->timestamp, time);
wma_info("%s [ROAM_TRIGGER]: VDEV[%d] %s", time, vdev_id, buf); wma_nofl_info("%s [ROAM_TRIGGER]: VDEV[%d] %s", time, vdev_id, buf);
qdf_mem_free(buf); qdf_mem_free(buf);
} }
@@ -1898,9 +1898,9 @@ wma_rso_print_btm_rsp_info(struct roam_btm_response_data *data,
char time[TIME_STRING_LEN]; char time[TIME_STRING_LEN];
mlme_get_converted_timestamp(data->timestamp, time); mlme_get_converted_timestamp(data->timestamp, time);
wma_info("%s [BTM RSP]: VDEV[%d], Status: %d, VSIE reason: %d, BSSID: "QDF_MAC_ADDR_FMT, wma_nofl_info("%s [BTM RSP]: VDEV[%d], Status: %d, VSIE reason: %d, BSSID: " QDF_MAC_ADDR_FMT,
time, vdev_id, data->btm_status, data->vsie_reason, time, vdev_id, data->btm_status, data->vsie_reason,
QDF_MAC_ADDR_REF(data->target_bssid.bytes)); QDF_MAC_ADDR_REF(data->target_bssid.bytes));
} }
/** /**
@@ -1917,9 +1917,9 @@ static void
wma_rso_print_roam_initial_info(struct roam_initial_data *data, wma_rso_print_roam_initial_info(struct roam_initial_data *data,
uint8_t vdev_id) uint8_t vdev_id)
{ {
wma_info("[ROAM INIT INFO]: VDEV[%d], roam_full_scan_count: %d, rssi_th: %d, cu_th: %d, fw_cancel_timer_bitmap: %d", wma_nofl_info("[ROAM INIT INFO]: VDEV[%d], roam_full_scan_count: %d, rssi_th: %d, cu_th: %d, fw_cancel_timer_bitmap: %d",
vdev_id, data->roam_full_scan_count, data->rssi_th, vdev_id, data->roam_full_scan_count, data->rssi_th,
data->cu_th, data->fw_cancel_timer_bitmap); data->cu_th, data->fw_cancel_timer_bitmap);
} }
/** /**
@@ -2045,9 +2045,9 @@ wma_rso_print_scan_info(struct wmi_roam_scan_data *scan, uint8_t vdev_id,
scan->next_rssi_threshold); scan->next_rssi_threshold);
mlme_get_converted_timestamp(timestamp, time); mlme_get_converted_timestamp(timestamp, time);
wma_info("%s [ROAM_SCAN]: VDEV[%d] Scan_type: %s %s %s", wma_nofl_info("%s [ROAM_SCAN]: VDEV[%d] Scan_type: %s %s %s",
time, vdev_id, mlme_get_roam_scan_type_str(scan->type), time, vdev_id, mlme_get_roam_scan_type_str(scan->type),
buf1, buf); buf1, buf);
wma_log_roam_scan_candidates(scan->ap, scan->num_ap); wma_log_roam_scan_candidates(scan->ap, scan->num_ap);
qdf_mem_free(buf); qdf_mem_free(buf);
@@ -2079,8 +2079,9 @@ wma_rso_print_roam_result(struct wmi_roam_result *res,
mlme_get_roam_fail_reason_str(res->fail_reason)); mlme_get_roam_fail_reason_str(res->fail_reason));
mlme_get_converted_timestamp(res->timestamp, time); mlme_get_converted_timestamp(res->timestamp, time);
wma_info("%s [ROAM_RESULT]: VDEV[%d] %s %s", wma_nofl_info("%s [ROAM_RESULT]: VDEV[%d] %s %s",
time, vdev_id, mlme_get_roam_status_str(res->status), buf); time, vdev_id, mlme_get_roam_status_str(res->status),
buf);
qdf_mem_free(buf); qdf_mem_free(buf);
} }
@@ -2131,17 +2132,18 @@ wma_rso_print_11kv_info(struct wmi_neighbor_report_data *neigh_rpt,
} }
mlme_get_converted_timestamp(neigh_rpt->req_time, time); mlme_get_converted_timestamp(neigh_rpt->req_time, time);
wma_info("%s [%s] VDEV[%d]", time, wma_nofl_info("%s [%s] VDEV[%d]", time,
(type == 1) ? "BTM_QUERY" : "NEIGH_RPT_REQ", vdev_id); (type == 1) ? "BTM_QUERY" : "NEIGH_RPT_REQ", vdev_id);
if (neigh_rpt->resp_time) { if (neigh_rpt->resp_time) {
mlme_get_converted_timestamp(neigh_rpt->resp_time, time1); mlme_get_converted_timestamp(neigh_rpt->resp_time, time1);
wma_info("%s [%s] VDEV[%d] %s", time1, wma_nofl_info("%s [%s] VDEV[%d] %s", time1,
(type == 1) ? "BTM_REQ" : "NEIGH_RPT_RSP", vdev_id, (type == 1) ? "BTM_REQ" : "NEIGH_RPT_RSP",
(num_ch > 0) ? buf : "NO Ch update"); vdev_id,
(num_ch > 0) ? buf : "NO Ch update");
} else { } else {
wma_info("%s No response received from AP", wma_nofl_info("%s No response received from AP",
(type == 1) ? "BTM" : "NEIGH_RPT"); (type == 1) ? "BTM" : "NEIGH_RPT");
} }
qdf_mem_free(buf); qdf_mem_free(buf);
} }

View File

@@ -1,5 +1,5 @@
/* /*
* Copyright (c) 2011-2020 The Linux Foundation. All rights reserved. * Copyright (c) 2011-2021 The Linux Foundation. All rights reserved.
* *
* Permission to use, copy, modify, and/or distribute this software for * Permission to use, copy, modify, and/or distribute this software for
* any purpose with or without fee is hereby granted, provided that the * any purpose with or without fee is hereby granted, provided that the
@@ -429,7 +429,7 @@ wlan_cfg80211_mc_cp_stats_get_peer_rssi(struct wlan_objmgr_vdev *vdev,
*errno = osif_request_wait_for_response(request); *errno = osif_request_wait_for_response(request);
if (*errno) { if (*errno) {
osif_err("wait failed or timed out ret: %d", *errno); osif_debug("wait failed or timed out ret: %d", *errno);
goto get_peer_rssi_fail; goto get_peer_rssi_fail;
} }