diff options
| author | Selvaraj, Sridhar <sselvara@codeaurora.org> | 2017-01-10 11:37:29 +0530 |
|---|---|---|
| committer | qcabuildsw <qcabuildsw@localhost> | 2017-01-10 09:53:48 -0800 |
| commit | ac9ed3f5fdac650c9b1cb90f7cbb11e2408352c1 (patch) | |
| tree | 2c836109e5c7bc22f8f814fd6917e088d23fcd61 | |
| parent | 76e2ba74dbc75ff47cc4b998c6543340021e38d2 (diff) | |
qcacld-3.0: Logging enhancements to LFR3 Roaming
1) Add enhancements to the existing log prints for lFR3
with respect to host driver code.
2) Fix memory leak in Wma roaming config handler.
Change-Id: If3e26a7f374673a4c7c11862e94f042bcf9d3c58
CRs-Fixed: 1109354
| -rw-r--r-- | core/sme/src/csr/csr_api_roam.c | 15 | ||||
| -rw-r--r-- | core/wma/src/wma_scan_roam.c | 1 |
2 files changed, 15 insertions, 1 deletions
diff --git a/core/sme/src/csr/csr_api_roam.c b/core/sme/src/csr/csr_api_roam.c index f079cf3c8adf..62a12105c21e 100644 --- a/core/sme/src/csr/csr_api_roam.c +++ b/core/sme/src/csr/csr_api_roam.c @@ -17329,7 +17329,11 @@ csr_create_roam_scan_offload_request(tpAniSirGlobal mac_ctx, (csr_is_auth_type_ese(req_buf-> ConnectedNetwork.authentication))); QDF_TRACE(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_DEBUG, - FL("LFR3:IsEseAssoc=%d\n"), req_buf->IsESEAssoc); + FL("IsEseAssoc=%d middle of roaming %d ese_neighbor_list_recvd %d cur no of chan %d"), + req_buf->IsESEAssoc, + req_buf->middle_of_roaming, + ese_neighbor_list_recvd, + curr_ch_lst_info->numOfChannels); #endif if (ese_neighbor_list_recvd || curr_ch_lst_info->numOfChannels == 0) { /* @@ -17340,6 +17344,9 @@ csr_create_roam_scan_offload_request(tpAniSirGlobal mac_ctx, status = csr_fetch_ch_lst_from_ini(mac_ctx, roam_info, req_buf); if (!QDF_IS_STATUS_SUCCESS(status)) { + QDF_TRACE(QDF_MODULE_ID_SME, + QDF_TRACE_LEVEL_DEBUG, + FL("Fetch channel list from ini failed")); qdf_mem_free(req_buf); return NULL; } @@ -17376,6 +17383,8 @@ csr_create_roam_scan_offload_request(tpAniSirGlobal mac_ctx, /* Maintain the Valid Channels List */ status = csr_fetch_valid_ch_lst(mac_ctx, req_buf); if (!QDF_IS_STATUS_SUCCESS(status)) { + QDF_TRACE(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_ERROR, + FL("Fetch channel list fail")); qdf_mem_free(req_buf); return NULL; } @@ -17844,6 +17853,9 @@ csr_roam_offload_scan(tpAniSirGlobal mac_ctx, uint8_t session_id, uint8_t i; uint8_t op_channel; + QDF_TRACE(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_DEBUG, + "RSO Command %d, Session id %d, Reason %d", + command, session_id, reason); if (NULL == session) { QDF_TRACE(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_ERROR, FL("session is null")); @@ -17866,6 +17878,7 @@ csr_roam_offload_scan(tpAniSirGlobal mac_ctx, uint8_t session_id, * expecting any WMI commands when the roam synch is in progress */ b_roam_scan_offload_started = false; + sms_log(mac_ctx, LOG1, FL("Roam sync in progress")); return QDF_STATUS_SUCCESS; } #endif diff --git a/core/wma/src/wma_scan_roam.c b/core/wma/src/wma_scan_roam.c index 453d44b672f7..b8e1b268cfe9 100644 --- a/core/wma/src/wma_scan_roam.c +++ b/core/wma/src/wma_scan_roam.c @@ -1732,6 +1732,7 @@ QDF_STATUS wma_process_roaming_config(tp_wma_handle wma_handle, if (NULL == pMac) { WMA_LOGE("%s: pMac is NULL", __func__); + qdf_mem_free(roam_req); return QDF_STATUS_E_FAILURE; } |