wlan: SME log level modification.
Added logs which should get printed when some unexpected error
happens. Modification in log level for SME.
CRs-Fixed: 517829
Change-Id: Id7e7f21136d7ddb8e5d1c71ee29a6a13f717d1b8
diff --git a/CORE/SME/src/ccm/ccmApi.c b/CORE/SME/src/ccm/ccmApi.c
index 27eb121..284379c 100644
--- a/CORE/SME/src/ccm/ccmApi.c
+++ b/CORE/SME/src/ccm/ccmApi.c
@@ -141,7 +141,7 @@
status = palSendMBMessage(hHdd, msg) ;
if (status != eHAL_STATUS_SUCCESS)
{
- smsLog( pMac, LOGW, FL("palSendMBMessage() failed"));
+ smsLog( pMac, LOGE, FL("palSendMBMessage() failed"));
//No need to free msg. palSendMBMessage frees it.
status = eHAL_STATUS_FAILURE ;
}
@@ -243,7 +243,7 @@
req->state = eCCM_REQ_SENT;
if (sendCfg(pMac, hHdd, req, eANI_BOOLEAN_TRUE) != eHAL_STATUS_SUCCESS)
{
- smsLog( pMac, LOGW, FL("sendCfg() failed"));
+ smsLog( pMac, LOGE, FL("sendCfg() failed"));
palSpinLockTake(hHdd, pMac->ccm.lock);
del_req(req, &pMac->ccm.reqQ) ;
palSpinLockGive(hHdd, pMac->ccm.lock);
@@ -322,7 +322,7 @@
status = sendCfg(pMac, hHdd, req, eANI_BOOLEAN_TRUE) ;
if (status != eHAL_STATUS_SUCCESS)
{
- smsLog( pMac, LOGW, FL("sendCfg() failed"));
+ smsLog( pMac, LOGE, FL("sendCfg() failed"));
palSpinLockTake(hHdd, pMac->ccm.lock);
del_req(req, &pMac->ccm.reqQ);
palSpinLockGive(hHdd, pMac->ccm.lock);
diff --git a/CORE/SME/src/csr/csrApiRoam.c b/CORE/SME/src/csr/csrApiRoam.c
index 7a25eab..828f7be 100644
--- a/CORE/SME/src/csr/csrApiRoam.c
+++ b/CORE/SME/src/csr/csrApiRoam.c
@@ -422,10 +422,10 @@
v_U8_t cntryCodeLength;
if(NULL == apCntryCode)
{
- smsLog( pMac, LOGW, FL(" Invalid country Code Pointer") );
+ smsLog( pMac, LOGE, FL(" Invalid country Code Pointer") );
return eHAL_STATUS_FAILURE;
}
- smsLog( pMac, LOGW, FL(" country Code %.2s"), apCntryCode );
+ smsLog( pMac, LOG1, FL(" country Code %.2s"), apCntryCode );
/* To get correct Regulatory domain from NV table
* 2 character Country code should be used
* 3rd charater is optional for indoor/outdoor setting */
@@ -2790,7 +2790,8 @@
status = csrSendMBDeauthReqMsg( pMac, sessionId, bssId, eSIR_MAC_DEAUTH_LEAVING_BSS_REASON );
if(!HAL_STATUS_SUCCESS(status))
{
- smsLog(pMac, LOGW, FL("csrSendMBDeauthReqMsg failed with status %d"), status);
+ smsLog(pMac, LOGE, FL("csrSendMBDeauthReqMsg failed with status %d Session ID: %d"
+ MAC_ADDRESS_STR ), status, sessionId, MAC_ADDR_ARRAY(bssId));
}
return (status);
@@ -4041,7 +4042,8 @@
if(!HAL_STATUS_SUCCESS(csrRoamIssueDisassociate( pMac, sessionId,
eCSR_ROAM_SUBSTATE_DISASSOC_REQ, FALSE )))
{
- smsLog(pMac, LOGW, FL(" fail to issue disassociate"));
+ smsLog(pMac, LOGE, FL(" fail to issue disassociate with Session ID %d"),
+ sessionId);
eRoamState = eCsrStopRoaming;
}
}
@@ -5493,7 +5495,7 @@
pSirBssDesc = &pSmeStartBssRsp->bssDescription;
if( !HAL_STATUS_SUCCESS(csrGetParsedBssDescriptionIEs( pMac, pSirBssDesc, &pIes )) )
{
- smsLog(pMac, LOG2, FL("cannot parse IBSS IEs"));
+ smsLog(pMac, LOGW, FL("cannot parse IBSS IEs"));
roamInfo.pBssDesc = pSirBssDesc;
//We need to associate_complete it first, becasue Associate_start already indicated.
csrRoamCallCallback( pMac, sessionId, &roamInfo, pCommand->u.roamCmd.roamId,
@@ -7501,7 +7503,7 @@
tCsrRoamSession *pSession;
if(NULL == pEntry)
{
- smsLog(pMac, LOGW, " CFG_CNF with active list empty");
+ smsLog(pMac, LOGE, " CFG_CNF with active list empty");
return;
}
pCommand = GET_BASE_ADDR(pEntry, tSmeCmd, Link);
@@ -7550,7 +7552,7 @@
&pSession->bssParams, &pCommand->u.roamCmd.roamProfile,
pBssDesc, pCommand->u.roamCmd.roamId )))
{
- smsLog(pMac, LOGW, " CSR start BSS failed");
+ smsLog(pMac, LOGE, " CSR start BSS failed");
//We need to complete the command
csrRoamComplete(pMac, eCsrStartBssFailure, NULL);
}
@@ -7559,7 +7561,7 @@
{
if (!pCommand->u.roamCmd.pRoamBssEntry)
{
- smsLog(pMac, LOGW, " pRoamBssEntry is NULL");
+ smsLog(pMac, LOGE, " pRoamBssEntry is NULL");
//We need to complete the command
csrRoamComplete(pMac, eCsrJoinFailure, NULL);
return;
@@ -8772,7 +8774,7 @@
}
else
{
- smsLog(pMac, LOGW, FL("Channel list empty"));
+ smsLog(pMac, LOGE, FL("Channel list empty"));
status = eHAL_STATUS_FAILURE;
break;
}
@@ -11698,7 +11700,7 @@
if(Channel == 0)
{
- smsLog(pMac, LOGW, " CSR cannot find a channel to start IBSS");
+ smsLog(pMac, LOGE, " CSR cannot find a channel to start IBSS");
}
else
{
@@ -14598,7 +14600,7 @@
pMsg = vos_mem_malloc(sizeof(tAniGetPEStatsReq));
if ( NULL == pMsg )
{
- smsLog(pMac, LOG1, " csrSendMBStatsReqMsg: failed to allocate mem for stats req ");
+ smsLog(pMac, LOGE, FL( "Failed to allocate mem for stats req "));
return eHAL_STATUS_FAILURE;
}
// need to initiate a stats request to PE
@@ -14609,7 +14611,7 @@
status = palSendMBMessage(pMac->hHdd, pMsg );
if(!HAL_STATUS_SUCCESS(status))
{
- smsLog(pMac, LOG1, " csrSendMBStatsReqMsg: failed to send down the stats req ");
+ smsLog(pMac, LOG1, FL("Failed to send down the stats req "));
}
return status;
}
@@ -15114,7 +15116,7 @@
pMsg = vos_mem_malloc(sizeof(tAniGetRssiReq));
if ( NULL == pMsg )
{
- smsLog(pMac, LOGE, " csrGetRoamRssi: failed to allocate mem for req");
+ smsLog(pMac, LOGE, FL("Failed to allocate mem for req"));
return eHAL_STATUS_FAILURE;
}
// need to initiate a stats request to PE
@@ -15127,7 +15129,7 @@
status = palSendMBMessage(pMac->hHdd, pMsg );
if(!HAL_STATUS_SUCCESS(status))
{
- smsLog(pMac, LOG1, " csrGetRoamRssi: failed to send down the rssi req");
+ smsLog(pMac, LOGE, FL(" Failed to send down get rssi req"));
//pMsg is freed by palSendMBMessage
status = eHAL_STATUS_FAILURE;
}
@@ -16162,7 +16164,7 @@
if(!pEntry)
{
//list empty
- smsLog(pMac, LOGW, "csrRoamRemoveEntryFromPeStatsReqList: List empty, no stats req for PE");
+ smsLog(pMac, LOGE, FL(" List empty, no stats req for PE"));
return;
}
while( pEntry )
@@ -16170,7 +16172,7 @@
pTempStaEntry = GET_BASE_ADDR( pEntry, tCsrPeStatsReqInfo, link );
if( pTempStaEntry && pTempStaEntry->statsMask == pPeStaEntry->statsMask)
{
- smsLog(pMac, LOGW, "csrRoamRemoveEntryFromPeStatsReqList: match found");
+ smsLog(pMac, LOGW, FL("Match found"));
if(pTempStaEntry->timerRunning)
{
vosStatus = vos_timer_stop( &pTempStaEntry->hPeStatsTimer );
@@ -16248,12 +16250,12 @@
tANI_U8 counter = 0;
if(!callback)
{
- smsLog(pMac, LOGE, FL("csrRoamReportStatistics:cannot report callback NULL"));
+ smsLog(pMac, LOGE, FL("Cannot report callback NULL"));
return;
}
if(!statsMask)
{
- smsLog(pMac, LOGE, FL("csrRoamReportStatistics:cannot report statsMask is 0"));
+ smsLog(pMac, LOGE, FL("Cannot report statsMask is 0"));
return;
}
pStats = stats;
@@ -16266,43 +16268,43 @@
switch(counter)
{
case eCsrSummaryStats:
- smsLog( pMac, LOG2, FL("csrRoamReportStatistics:summary stats"));
+ smsLog( pMac, LOG2, FL("Summary stats"));
vos_mem_copy( pStats, (tANI_U8 *)&pMac->roam.summaryStatsInfo,
sizeof(tCsrSummaryStatsInfo));
pStats += sizeof(tCsrSummaryStatsInfo);
break;
case eCsrGlobalClassAStats:
- smsLog( pMac, LOG2, FL("csrRoamReportStatistics:ClassA stats"));
+ smsLog( pMac, LOG2, FL("ClassA stats"));
vos_mem_copy( pStats, (tANI_U8 *)&pMac->roam.classAStatsInfo,
sizeof(tCsrGlobalClassAStatsInfo));
pStats += sizeof(tCsrGlobalClassAStatsInfo);
break;
case eCsrGlobalClassBStats:
- smsLog( pMac, LOG2, FL("csrRoamReportStatistics:ClassB stats"));
+ smsLog( pMac, LOG2, FL("ClassB stats"));
vos_mem_copy( pStats, (tANI_U8 *)&pMac->roam.classBStatsInfo,
sizeof(tCsrGlobalClassBStatsInfo));
pStats += sizeof(tCsrGlobalClassBStatsInfo);
break;
case eCsrGlobalClassCStats:
- smsLog( pMac, LOG2, FL("csrRoamReportStatistics:ClassC stats"));
+ smsLog( pMac, LOG2, FL("ClassC stats"));
vos_mem_copy( pStats, (tANI_U8 *)&pMac->roam.classCStatsInfo,
sizeof(tCsrGlobalClassCStatsInfo));
pStats += sizeof(tCsrGlobalClassCStatsInfo);
break;
case eCsrGlobalClassDStats:
- smsLog( pMac, LOG2, FL("csrRoamReportStatistics:ClassD stats"));
+ smsLog( pMac, LOG2, FL("ClassD stats"));
vos_mem_copy( pStats, (tANI_U8 *)&pMac->roam.classDStatsInfo,
sizeof(tCsrGlobalClassDStatsInfo));
pStats += sizeof(tCsrGlobalClassDStatsInfo);
break;
case eCsrPerStaStats:
- smsLog( pMac, LOG2, FL("csrRoamReportStatistics:PerSta stats"));
+ smsLog( pMac, LOG2, FL("PerSta stats"));
vos_mem_copy( pStats, (tANI_U8 *)&pMac->roam.perStaStatsInfo[staId],
sizeof(tCsrPerStaStatsInfo));
pStats += sizeof(tCsrPerStaStatsInfo);
break;
default:
- smsLog( pMac, LOG1, FL("csrRoamReportStatistics:unknown stats type"));
+ smsLog( pMac, LOGE, FL("Unknown stats type and counter %d"), counter);
break;
}
}
@@ -16825,16 +16827,14 @@
tCsrRoamInfo roamInfo;
#endif
-#if defined WLAN_FEATURE_VOWIFI_11R_DEBUG
- smsLog( pMac, LOGE, FL("Preauth response status code 0x%x"), pFTPreAuthRsp->status);
-#endif
#ifdef WLAN_FEATURE_NEIGHBOR_ROAMING
status = csrNeighborRoamPreauthRspHandler(pMac, pFTPreAuthRsp->status);
if (status != eHAL_STATUS_SUCCESS) {
/*
* Bail out if pre-auth was not even processed.
*/
- smsLog(pMac, LOGW, FL("Preauth was not processed: %d"), status);
+ smsLog(pMac, LOGE,FL("Preauth was not processed: %d SessionID: %d"),
+ status, pFTPreAuthRsp->smeSessionId);
return;
}
#endif
diff --git a/CORE/SME/src/csr/csrApiScan.c b/CORE/SME/src/csr/csrApiScan.c
index 4efab15..3459763 100644
--- a/CORE/SME/src/csr/csrApiScan.c
+++ b/CORE/SME/src/csr/csrApiScan.c
@@ -2573,7 +2573,7 @@
if (!HAL_STATUS_SUCCESS(csrGetCfgValidChannels(pMac,
pMac->roam.validChannelList, &len)))
{
- smsLog( pMac, LOG1, "Failed to get Channel list from CFG");
+ smsLog( pMac, LOGE, "Failed to get Channel list from CFG");
}
pEntry = csrLLPeekHead( &pMac->scan.scanResultList, LL_ACCESS_LOCK );
diff --git a/CORE/SME/src/csr/csrNeighborRoam.c b/CORE/SME/src/csr/csrNeighborRoam.c
index dd49214..05a0c76 100644
--- a/CORE/SME/src/csr/csrNeighborRoam.c
+++ b/CORE/SME/src/csr/csrNeighborRoam.c
@@ -375,7 +375,7 @@
if (!VOS_IS_STATUS_SUCCESS(vosStatus))
{
//err msg
- smsLog(pMac, LOGW, FL(" Couldn't register csrNeighborRoamNeighborLookupDOWNCallback with TL: Status = %d"), vosStatus);
+ smsLog(pMac, LOGE, FL(" Couldn't register csrNeighborRoamNeighborLookupDOWNCallback with TL: Status = %d"), vosStatus);
vosStatus = VOS_STATUS_E_FAILURE;
}
#ifdef WLAN_FEATURE_ROAM_SCAN_OFFLOAD
@@ -436,7 +436,7 @@
if (!VOS_IS_STATUS_SUCCESS(vosStatus))
{
//err msg
- smsLog(pMac, LOGW, FL(" Couldn't register csrNeighborRoamNeighborLookupDOWNCallback with TL: Status = %d"), vosStatus);
+ smsLog(pMac, LOGE, FL(" Couldn't register csrNeighborRoamNeighborLookupDOWNCallback with TL: Status = %d"), vosStatus);
vosStatus = VOS_STATUS_E_FAILURE;
}
#ifdef WLAN_FEATURE_ROAM_SCAN_OFFLOAD
@@ -509,7 +509,7 @@
if (!VOS_IS_STATUS_SUCCESS(vosStatus))
{
//err msg
- smsLog(pMac, LOGW, FL(" Couldn't register csrNeighborRoamNeighborLookupDOWNCallback with TL: Status = %d"), vosStatus);
+ smsLog(pMac, LOGE, FL(" Couldn't register csrNeighborRoamNeighborLookupDOWNCallback with TL: Status = %d"), vosStatus);
vosStatus = VOS_STATUS_E_FAILURE;
}
#ifdef WLAN_FEATURE_ROAM_SCAN_OFFLOAD
@@ -1001,7 +1001,7 @@
if (NULL == pNeighborBssNode)
{
- smsLog(pMac, LOG1, FL("Roamable AP list is empty.. "));
+ smsLog(pMac, LOGW, FL("Roamable AP list is empty.. "));
return eHAL_STATUS_FAILURE;
}
else
@@ -1740,7 +1740,6 @@
/* Stop neighbor scan timer */
vos_timer_stop(&pNeighborRoamInfo->neighborScanTimer);
-
/*
* Increase the neighbor lookup threshold by 3 dB
* after every scan cycle. NOTE: uEmptyScanCount
diff --git a/CORE/SME/src/csr/csrUtil.c b/CORE/SME/src/csr/csrUtil.c
index a13dfda..417140f 100644
--- a/CORE/SME/src/csr/csrUtil.c
+++ b/CORE/SME/src/csr/csrUtil.c
@@ -3116,7 +3116,7 @@
break;
default :
- smsLog(pMac, LOG1, FL(" Persona not supported : %d"),currBssPersona);
+ smsLog(pMac, LOGE, FL(" Persona not supported : %d"),currBssPersona);
return eHAL_STATUS_FAILURE;
}
}
@@ -6143,7 +6143,7 @@
}
else
{
- smsLog(pMac, LOGW, FL(" doesn't match country %c%c"), pCountry[0], pCountry[1]);
+ smsLog(pMac, LOGW, FL(" Couldn't find domain for country code %c%c"), pCountry[0], pCountry[1]);
status = eHAL_STATUS_INVALID_PARAMETER;
}
}
diff --git a/CORE/SME/src/p2p/p2p_Api.c b/CORE/SME/src/p2p/p2p_Api.c
index 98c4d44..d5d88e1 100644
--- a/CORE/SME/src/p2p/p2p_Api.c
+++ b/CORE/SME/src/p2p/p2p_Api.c
@@ -1595,7 +1595,7 @@
pMac->p2pContext[sessionId].listenDuration = P2P_LISTEN_TIMEOUT_HIGH;
if (pMac->p2pContext[sessionId].state == eP2P_STATE_DISCONNECTED)
{
- VOS_TRACE(VOS_MODULE_ID_SME, VOS_TRACE_LEVEL_INFO, "%s Calling RemainOnChannel with diration %d on channel %d",
+ VOS_TRACE(VOS_MODULE_ID_SME, VOS_TRACE_LEVEL_INFO, "%s Calling RemainOnChannel with duration %d on channel %d",
__func__, pMac->p2pContext[sessionId].listenDuration, pMac->p2pContext[sessionId].P2PListenChannel);
p2pRemainOnChannel( pMac, pMac->p2pContext[sessionId].SMEsessionId, pMac->p2pContext[sessionId].P2PListenChannel,
pMac->p2pContext[sessionId].listenDuration, p2pListenStateDiscoverableCallback,
@@ -1627,7 +1627,7 @@
if (pMac->p2pContext[sessionId].state == eP2P_STATE_DISCONNECTED)
{
- VOS_TRACE(VOS_MODULE_ID_SME, VOS_TRACE_LEVEL_INFO, "%s Calling RemainOnChannel with diration %d on channel %d",
+ VOS_TRACE(VOS_MODULE_ID_SME, VOS_TRACE_LEVEL_INFO, "%s Calling RemainOnChannel with duration %d on channel %d",
__func__, pMac->p2pContext[sessionId].listenDuration, pMac->p2pContext[sessionId].P2PListenChannel);
p2pRemainOnChannel( pMac, pMac->p2pContext[sessionId].SMEsessionId, pMac->p2pContext[sessionId].P2PListenChannel,
pMac->p2pContext[sessionId].listenDuration, p2pListenStateDiscoverableCallback,
diff --git a/CORE/SME/src/sme_common/sme_Api.c b/CORE/SME/src/sme_common/sme_Api.c
index 2749b00..caff98f 100644
--- a/CORE/SME/src/sme_common/sme_Api.c
+++ b/CORE/SME/src/sme_common/sme_Api.c
@@ -4033,7 +4033,7 @@
MTRACE(vos_trace(VOS_MODULE_ID_SME,
TRACE_CODE_SME_RX_HDD_REQUEST_STANDBY, NO_SESSION, 0));
- smsLog( pMac, LOG1, FL("") );
+ smsLog( pMac, LOG1, FL(" called") );
status = sme_AcquireGlobalLock( &pMac->sme );
if ( HAL_STATUS_SUCCESS( status ) )
{