qcacmn: Reduce target_if info logging
Excessive logging in target_if has been identified as the root cause of
a watchdog timeout. Reduce target_if logging in normal operation paths
to debug level to avoid excessive CPU use.
Change-Id: If384474270bb4b860100b98e5eea81d54b7c2184
CRs-Fixed: 2286630
diff --git a/target_if/init_deinit/src/init_cmd_api.c b/target_if/init_deinit/src/init_cmd_api.c
index 7228a55..b2c3364 100644
--- a/target_if/init_deinit/src/init_cmd_api.c
+++ b/target_if/init_deinit/src/init_cmd_api.c
@@ -103,9 +103,8 @@
info->mem_chunks[idx].req_id = req_id;
}
}
- target_if_info(
- "req_id %d idx %d num_units %d unit_len %d",
- req_id, idx, num_units, unit_len);
+ target_if_debug("req_id %d idx %d num_units %d unit_len %d",
+ req_id, idx, num_units, unit_len);
return num_units;
}
@@ -250,11 +249,11 @@
}
}
- target_if_info("idx %d req %d num_units %d num_unit_info %d unit size %d actual units %d",
- idx, mem_reqs[idx].req_id,
- mem_reqs[idx].num_units,
- mem_reqs[idx].num_unit_info,
- mem_reqs[idx].unit_size, num_units);
+ target_if_debug("idx %d req %d num_units %d num_unit_info %d unit size %d actual units %d",
+ idx, mem_reqs[idx].req_id,
+ mem_reqs[idx].num_units,
+ mem_reqs[idx].num_unit_info,
+ mem_reqs[idx].unit_size, num_units);
status = init_deinit_alloc_host_mem(psoc, tgt_hdl,
mem_reqs[idx].req_id, num_units,
@@ -384,7 +383,7 @@
(WMI_HOST_WLAN_5G_CAPABILITY |
WMI_HOST_WLAN_2G_CAPABILITY)) {
/*Supports both 5G and 2G. Use freq from both radios*/
- target_if_info("Supports both 2G and 5G");
+ target_if_debug("Supports both 2G and 5G");
band_to_mac[i].pdev_id = mac_phy_cap->pdev_id;
band_to_mac[i].start_freq =
reg_cap[i].low_2ghz_chan;
@@ -402,11 +401,10 @@
reg_cap[mac_phy_cap->phy_id].low_5ghz_chan = 0;
reg_cap[mac_phy_cap->phy_id].high_5ghz_chan = 0;
- target_if_info(
- "2G radio - pdev_id = %d start_freq = %d end_freq= %d",
- band_to_mac[i].pdev_id,
- band_to_mac[i].start_freq,
- band_to_mac[i].end_freq);
+ target_if_debug("2G radio - pdev_id = %d start_freq = %d end_freq= %d",
+ band_to_mac[i].pdev_id,
+ band_to_mac[i].start_freq,
+ band_to_mac[i].end_freq);
} else if (mac_phy_cap->supported_bands ==
WMI_HOST_WLAN_5G_CAPABILITY) {
@@ -419,11 +417,10 @@
reg_cap[mac_phy_cap->phy_id].low_2ghz_chan = 0;
reg_cap[mac_phy_cap->phy_id].high_2ghz_chan = 0;
- target_if_info(
- "5G radio -pdev_id = %d start_freq = %d end_freq =%d\n",
- band_to_mac[i].pdev_id,
- band_to_mac[i].start_freq,
- band_to_mac[i].end_freq);
+ target_if_debug("5G radio -pdev_id = %d start_freq = %d end_freq =%d\n",
+ band_to_mac[i].pdev_id,
+ band_to_mac[i].start_freq,
+ band_to_mac[i].end_freq);
}
}
}
@@ -480,8 +477,7 @@
return;
target_if_info("FW version 0x%x ", info->target_caps.fw_version);
- if (init_deinit_is_service_ext_msg(psoc, tgt_hdl) ==
- QDF_STATUS_SUCCESS)
+ if (init_deinit_is_service_ext_msg(psoc, tgt_hdl) == QDF_STATUS_SUCCESS)
target_if_info("0x%x\n",
info->service_ext_param.fw_build_vers_ext);
else
diff --git a/target_if/init_deinit/src/init_event_handler.c b/target_if/init_deinit/src/init_event_handler.c
index a78a2fe..a27e571 100644
--- a/target_if/init_deinit/src/init_event_handler.c
+++ b/target_if/init_deinit/src/init_event_handler.c
@@ -101,7 +101,7 @@
if (wmi_service_enabled(wmi_handle, wmi_service_check_cal_version))
wlan_psoc_nif_fw_ext_cap_set(psoc, WLAN_SOC_CEXT_SW_CAL);
- target_if_info(" TT support %d, Wide BW Scan %d, SW cal %d",
+ target_if_debug(" TT support %d, Wide BW Scan %d, SW cal %d",
wlan_psoc_nif_fw_ext_cap_get(psoc, WLAN_SOC_CEXT_TT_SUPPORT),
wlan_psoc_nif_fw_ext_cap_get(psoc, WLAN_SOC_CEXT_WIDEBAND_SCAN),
wlan_psoc_nif_fw_ext_cap_get(psoc, WLAN_SOC_CEXT_SW_CAL));
@@ -140,11 +140,11 @@
if (wmi_service_enabled(wmi_handle, wmi_service_mgmt_tx_wmi)) {
wlan_psoc_nif_fw_ext_cap_set(psoc, WLAN_SOC_CEXT_WMI_MGMT_REF);
- target_if_info("WMI mgmt service enabled");
+ target_if_debug("WMI mgmt service enabled");
} else {
wlan_psoc_nif_fw_ext_cap_clear(psoc,
WLAN_SOC_CEXT_WMI_MGMT_REF);
- target_if_info("WMI mgmt service disabled");
+ target_if_debug("WMI mgmt service disabled");
}
err_code = init_deinit_handle_host_mem_req(psoc, tgt_hdl, event);
@@ -153,13 +153,13 @@
target_if_reg_set_offloaded_info(psoc);
if (!wmi_service_enabled(wmi_handle, wmi_service_ext_msg)) {
- target_if_info("No EXT message, send init command");
+ target_if_debug("No EXT message, send init command");
tgt_hdl->info.wmi_service_ready = TRUE;
target_psoc_set_num_radios(tgt_hdl, 1);
/* send init command */
init_deinit_prepare_send_init_cmd(psoc, tgt_hdl);
} else {
- target_if_info("Wait for EXT message");
+ target_if_debug("Wait for EXT message");
}
target_if_smart_log_enable(psoc, tgt_hdl, event);
diff --git a/target_if/init_deinit/src/service_ready_util.c b/target_if/init_deinit/src/service_ready_util.c
index ced5421..cf0f428 100644
--- a/target_if/init_deinit/src/service_ready_util.c
+++ b/target_if/init_deinit/src/service_ready_util.c
@@ -153,8 +153,8 @@
hw_mode_id = hw_cap->hw_mode_id;
phy_bit_map = hw_cap->phy_id_map;
- target_if_info("hw_mode_id %d phy_bit_map 0x%x",
- hw_mode_id, phy_bit_map);
+ target_if_debug("hw_mode_id %d phy_bit_map 0x%x",
+ hw_mode_id, phy_bit_map);
mac_phy_id = 0;
while (phy_bit_map) {
@@ -177,7 +177,7 @@
phy_bit_map &= (phy_bit_map - 1);
mac_phy_id++;
}
- target_if_info("total_mac_phy_cnt %d", info->total_mac_phy_cnt);
+ target_if_debug("total_mac_phy_cnt %d", info->total_mac_phy_cnt);
return 0;
}
@@ -227,7 +227,7 @@
target_if_err("invalid num_hw_modes %d", num_hw_modes);
return -EINVAL;
}
- target_if_info("num_hw_modes %d", num_hw_modes);
+ target_if_debug("num_hw_modes %d", num_hw_modes);
qdf_mem_zero(&hw_mode_caps, sizeof(hw_mode_caps));
@@ -250,12 +250,11 @@
if ((preferred_mode != WMI_HOST_HW_MODE_MAX) &&
(hw_mode_caps[hw_idx].hw_mode_id == preferred_mode)) {
info->num_radios = info->total_mac_phy_cnt;
- target_if_info("num radios is %d\n", info->num_radios);
+ target_if_debug("num radios is %d\n", info->num_radios);
}
}
status = get_sar_version(wmi_handle, event, &info->service_ext_param);
- target_if_info("sar version %d",
- info->service_ext_param.sar_version);
+ target_if_debug("sar version %d", info->service_ext_param.sar_version);
return_exit:
return qdf_status_to_os_return(status);
@@ -271,7 +270,7 @@
num_dbr_ring_caps = info->service_ext_param.num_dbr_ring_caps;
- target_if_info("Num DMA Capabilities = %d", num_dbr_ring_caps);
+ target_if_debug("Num DMA Capabilities = %d", num_dbr_ring_caps);
if (!num_dbr_ring_caps)
return 0;
@@ -340,8 +339,8 @@
reg_cap[0].phy_id = 0;
qdf_mem_copy(&(reg_cap[0].eeprom_reg_domain), &cap,
sizeof(struct wlan_psoc_hal_reg_capability));
- target_if_info("FW wireless modes 0x%x",
- reg_cap[0].wireless_modes);
+ target_if_debug("FW wireless modes 0x%x",
+ reg_cap[0].wireless_modes);
} else {
num_phy_reg_cap = info->service_ext_param.num_phy;
if (num_phy_reg_cap > PSOC_MAX_PHY_REG_CAP) {
@@ -349,7 +348,7 @@
num_phy_reg_cap);
return -EINVAL;
}
- target_if_info("num_phy_reg_cap %d", num_phy_reg_cap);
+ target_if_debug("num_phy_reg_cap %d", num_phy_reg_cap);
for (reg_idx = 0; reg_idx < num_phy_reg_cap; reg_idx++) {
status = wmi_extract_reg_cap_service_ready_ext(handle,
diff --git a/target_if/regulatory/src/target_if_reg.c b/target_if/regulatory/src/target_if_reg.c
index c1aceb9..a42613d 100644
--- a/target_if/regulatory/src/target_if_reg.c
+++ b/target_if/regulatory/src/target_if_reg.c
@@ -356,8 +356,7 @@
reg_cap->high_2ghz_chan = 0;
}
- target_if_info(
- "phy_id = %d - low_2ghz_chan = %d high_2ghz_chan = %d low_5ghz_chan = %d high_5ghz_chan = %d",
+ target_if_debug("phy_id = %d - low_2ghz_chan = %d high_2ghz_chan = %d low_5ghz_chan = %d high_5ghz_chan = %d",
reg_cap->phy_id,
reg_cap->low_2ghz_chan,
reg_cap->high_2ghz_chan,
diff --git a/target_if/scan/src/target_if_scan.c b/target_if/scan/src/target_if_scan.c
index 0974f99..2b7f6eb 100644
--- a/target_if/scan/src/target_if_scan.c
+++ b/target_if/scan/src/target_if_scan.c
@@ -116,8 +116,8 @@
}
nlo_event = param_buf->fixed_param;
- target_if_info("PNO complete event received for vdev %d",
- nlo_event->vdev_id);
+ target_if_debug("PNO complete event received for vdev %d",
+ nlo_event->vdev_id);
event_info->event.type = SCAN_EVENT_TYPE_NLO_COMPLETE;
event_info->event.vdev_id = nlo_event->vdev_id;
@@ -166,8 +166,8 @@
}
nlo_event = param_buf->fixed_param;
- target_if_info("PNO match event received for vdev %d",
- nlo_event->vdev_id);
+ target_if_debug("PNO match event received for vdev %d",
+ nlo_event->vdev_id);
event_info->event.type = SCAN_EVENT_TYPE_NLO_MATCH;
event_info->event.vdev_id = nlo_event->vdev_id;