lisa/android_kernel_xiaomi_sm8350
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacld-3.0: Handle dummy wmi_radio_link_stats_link wmi event

Browse Source 
Sometimes wlan FW fails to return wmi_radio_link_stats_link because of
buffer, which makes timeout happen in wlan_hdd_cfg80211_ll_stats_get.

To fix this issue, FW returns wmi_radio_link_stats_link with dummy
information, and host need handle dummy wmi_radio_link_stats_link wmi
event.

Change-Id: Ibafc7f19ffc6d182601001f28dd7e85654155d02
CRs-Fixed: 2599725
This commit is contained in:
bings 2020-01-08 14:16:32 +08:00 committed by nshrivas
parent 8134b2fc69
commit a27327b2d8
1 changed files with 37 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

56
core/wma/src/wma_utils.c
View File

@ -2027,6 +2027,20 @@ static int wma_unified_link_radio_stats_event_handler(void *handle,
* num_channels * size of(struct wmi_channel_stats) * num_channels * size of(struct wmi_channel_stats)
*/ */
fixed_param = param_tlvs->fixed_param; fixed_param = param_tlvs->fixed_param;
if (fixed_param && !fixed_param->num_radio &&
!fixed_param->more_radio_events) {
WMA_LOGD("FW indicates dummy link radio stats");
if (!wma_handle->link_stats_results) {
wma_handle->link_stats_results = qdf_mem_malloc(
sizeof(*link_stats_results));
if (!wma_handle->link_stats_results)
return -ENOMEM;
}
link_stats_results = wma_handle->link_stats_results;
link_stats_results->num_radio = fixed_param->num_radio;
goto link_radio_stats_cb;
}
radio_stats = param_tlvs->radio_stats; radio_stats = param_tlvs->radio_stats;
channel_stats = param_tlvs->channel_stats; channel_stats = param_tlvs->channel_stats;
@ -2099,21 +2113,6 @@ static int wma_unified_link_radio_stats_event_handler(void *handle,
WMA_LOGD("on_time_host_scan: %u, on_time_lpi_scan: %u", WMA_LOGD("on_time_host_scan: %u, on_time_lpi_scan: %u",
radio_stats->on_time_host_scan, radio_stats->on_time_lpi_scan); radio_stats->on_time_host_scan, radio_stats->on_time_lpi_scan);
link_stats_results->paramId = WMI_LINK_STATS_RADIO;
link_stats_results->rspId = fixed_param->request_id;
link_stats_results->ifaceId = 0;
link_stats_results->peer_event_number = 0;
/*
* Backward compatibility:
* There are firmware(s) which will send Radio stats only with
* more_radio_events set to 0 and firmware which sends Radio stats
* followed by tx_power level stats with more_radio_events set to 1.
* if more_radio_events is set to 1, buffer the radio stats and
* wait for tx_power_level stats.
*/
link_stats_results->moreResultToFollow = fixed_param->more_radio_events;
results = (uint8_t *) link_stats_results->results; results = (uint8_t *) link_stats_results->results;
t_radio_stats = (uint8_t *) radio_stats; t_radio_stats = (uint8_t *) radio_stats;
t_channel_stats = (uint8_t *) channel_stats; t_channel_stats = (uint8_t *) channel_stats;
@ -2172,15 +2171,34 @@ static int wma_unified_link_radio_stats_event_handler(void *handle,
} }
} }
link_radio_stats_cb:
link_stats_results->paramId = WMI_LINK_STATS_RADIO;
link_stats_results->rspId = fixed_param->request_id;
link_stats_results->ifaceId = 0;
link_stats_results->peer_event_number = 0;
/*
* Backward compatibility:
* There are firmware(s) which will send Radio stats only with
* more_radio_events set to 0 and firmware which sends Radio stats
* followed by tx_power level stats with more_radio_events set to 1.
* if more_radio_events is set to 1, buffer the radio stats and
* wait for tx_power_level stats.
*/
link_stats_results->moreResultToFollow = fixed_param->more_radio_events;
if (link_stats_results->moreResultToFollow) { if (link_stats_results->moreResultToFollow) {
/* More results coming, don't post yet */ /* More results coming, don't post yet */
return 0; return 0;
} }
link_stats_results->nr_received++; if (link_stats_results->num_radio) {
link_stats_results->nr_received++;
if (link_stats_results->num_radio != link_stats_results->nr_received) { if (link_stats_results->num_radio !=
/* Not received all radio stats yet, don't post yet */ link_stats_results->nr_received) {
return 0; /* Not received all radio stats yet, don't post yet */
return 0;
}
} }
mac->sme.link_layer_stats_cb(mac->hdd_handle, mac->sme.link_layer_stats_cb(mac->hdd_handle,