qcacld: Add extra debug information for scan request and results

Added debug logs to find out why SME scan request does not start
fresh scan. Also added debug logs to find out reason for the
"Found 0 BSS" message.
This is a temporary change. We will change log levels after
debugging is complete.

Change-Id: I5d645739f556162e7d7d86174764c95946b8d2ad
CRs-Fixed: 752920

3 files changed, 36 insertions, 17 deletions

diff --git a/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c b/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c
index 65436c51ec3d..a8edd128baae 100644
--- a/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c
+++ b/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c
@@ -157,8 +157,11 @@ __limFreshScanReqd(tpAniSirGlobal pMac, tANI_U8 returnFreshResults)
     tANI_U8 validState = TRUE;
     int i;
 
+    limLog(pMac, LOG1, FL("gLimSmeState: %d, returnFreshResults 0x%x"),
+        pMac->lim.gLimSmeState, returnFreshResults);
     if(pMac->lim.gLimSmeState != eLIM_SME_IDLE_STATE)
     {
+        limLog(pMac, LOG1, FL("return FALSE"));
         return FALSE;
     }
     for(i =0; i < pMac->lim.maxBssId; i++)
@@ -166,6 +169,12 @@ __limFreshScanReqd(tpAniSirGlobal pMac, tANI_U8 returnFreshResults)
 
         if(pMac->lim.gpSession[i].valid == TRUE)
         {
+            limLog(pMac, LOG1,
+               FL("session %d, bsstype %d, limSystemRole %d, limSmeState %d"),
+               i,
+               pMac->lim.gpSession[i].bssType,
+               pMac->lim.gpSession[i].limSystemRole,
+               pMac->lim.gpSession[i].limSmeState);
             if(!( ( (  (pMac->lim.gpSession[i].bssType == eSIR_INFRASTRUCTURE_MODE) ||
                         (pMac->lim.gpSession[i].limSystemRole == eLIM_BT_AMP_STA_ROLE))&&
                        (pMac->lim.gpSession[i].limSmeState == eLIM_SME_LINK_EST_STATE) )||
@@ -187,12 +196,15 @@ __limFreshScanReqd(tpAniSirGlobal pMac, tANI_U8 returnFreshResults)
         }
     }
 
-   limLog(pMac, LOG1, FL("FreshScanReqd: %d "), validState);
-
-   if( (validState) && (returnFreshResults & SIR_BG_SCAN_RETURN_FRESH_RESULTS))
-    return TRUE;
 
-    return FALSE;
+    if((validState) &&
+       (returnFreshResults & SIR_BG_SCAN_RETURN_FRESH_RESULTS)) {
+        limLog(pMac, LOG1, FL("validState: %d, return TRUE"), validState);
+        return TRUE;
+    } else {
+        limLog(pMac, LOG1, FL("validState: %d, return FALSE"), validState);
+        return FALSE;
+    }
 }
 
 
@@ -1353,6 +1365,7 @@ __limProcessSmeScanReq(tpAniSirGlobal pMac, tANI_U32 *pMsgBuf)
      */
   if (__limFreshScanReqd(pMac, pScanReq->returnFreshResults))
   {
+      limLog(pMac, LOG1, FL("Fresh scan is required"));
       if (pMac->fScanOffload)
          limFlushp2pScanResults(pMac);
 
@@ -1566,9 +1579,13 @@ __limProcessSmeScanReq(tpAniSirGlobal pMac, tANI_U32 *pMsgBuf)
             if (pScanReq->returnFreshResults & SIR_BG_SCAN_RETURN_LFR_CACHED_RESULTS)
             {
                 pMac->lim.gLimSmeLfrScanResultLength = pMac->lim.gLimMlmLfrScanResultLength;
+                limLog(pMac, LOG1,
+                   FL("Returned scan results from LFR cache, length = %d"),
+                   pMac->lim.gLimSmeLfrScanResultLength);
+
                 if (pMac->lim.gLimSmeLfrScanResultLength == 0)
                 {
-                    limSendSmeLfrScanRsp(pMac, scanRspLen,
+                   limSendSmeLfrScanRsp(pMac, scanRspLen,
                                          eSIR_SME_SUCCESS,
                                          pScanReq->sessionId,
                                          pScanReq->transactionId);
@@ -1585,6 +1602,9 @@ __limProcessSmeScanReq(tpAniSirGlobal pMac, tANI_U32 *pMsgBuf)
             else
             {
 #endif
+                limLog(pMac, LOG1,
+                   FL("Returned scan results from normal cache, length = %d"),
+                   pMac->lim.gLimSmeScanResultLength);
                if (pMac->lim.gLimSmeScanResultLength == 0)
                {
                   limSendSmeScanRsp(pMac, scanRspLen, eSIR_SME_SUCCESS,
@@ -1606,8 +1626,6 @@ __limProcessSmeScanReq(tpAniSirGlobal pMac, tANI_U32 *pMsgBuf)
             if (pMac->fScanOffload)
                  limFlushp2pScanResults(pMac);
 
-            limLog(pMac, LOG1, FL("Cached scan results are returned "));
-
             if (pScanReq->returnFreshResults & SIR_BG_SCAN_PURGE_RESUTLS)
             {
                 // Discard previously cached scan results
diff --git a/CORE/MAC/src/pe/lim/limSendSmeRspMessages.c b/CORE/MAC/src/pe/lim/limSendSmeRspMessages.c
index ab79c333b00f..e833eec55787 100644
--- a/CORE/MAC/src/pe/lim/limSendSmeRspMessages.c
+++ b/CORE/MAC/src/pe/lim/limSendSmeRspMessages.c
@@ -766,9 +766,9 @@ limSendSmeScanRsp(tpAniSirGlobal pMac, tANI_U16 length,
     tANI_U8               *pbBuf;
     tSirBssDescription    *pDesc;
 
-    PELOG1(limLog(pMac, LOG1,
+    limLog(pMac, LOG1,
        FL("Sending message SME_SCAN_RSP with length=%d reasonCode %s"),
-       length, limResultCodeStr(resultCode));)
+       length, limResultCodeStr(resultCode));
 
     if (resultCode != eSIR_SME_SUCCESS)
     {
@@ -934,9 +934,9 @@ limSendSmeLfrScanRsp(tpAniSirGlobal pMac, tANI_U16 length,
     tANI_U8               *currentBssid =
         pMac->roam.roamSession[smesessionId].connectedProfile.bssid;
 
-    PELOG1(limLog(pMac, LOG1,
+    limLog(pMac, LOG1,
        FL("Sending message SME_SCAN_RSP with length=%d reasonCode %s\n"),
-       length, limResultCodeStr(resultCode));)
+       length, limResultCodeStr(resultCode));
 
     if (resultCode != eSIR_SME_SUCCESS)
     {
@@ -1182,9 +1182,9 @@ limPostSmeScanRspMessage(tpAniSirGlobal    pMac,
     tpSirSmeScanRsp   pSirSmeScanRsp;
     tSirMsgQ          mmhMsg;
 
-    PELOG1(limLog(pMac, LOG1,
+    limLog(pMac, LOG1,
        FL("limPostSmeScanRspMessage: send SME_SCAN_RSP (len %d, reasonCode %s). "),
-       length, limResultCodeStr(resultCode));)
+       length, limResultCodeStr(resultCode));
 
     pSirSmeScanRsp = vos_mem_malloc(length);
     if ( NULL == pSirSmeScanRsp )
diff --git a/CORE/SME/src/csr/csrApiScan.c b/CORE/SME/src/csr/csrApiScan.c
index 79ace8035237..2609cacf57d8 100644
--- a/CORE/SME/src/csr/csrApiScan.c
+++ b/CORE/SME/src/csr/csrApiScan.c
@@ -4670,7 +4670,7 @@ tCsrScanResult *csrScanSaveBssDescriptionToInterimList( tpAniSirGlobal pMac,
         smsLog(pMac, LOGW,
            FL(" Set Aging Count = %d for BSS "MAC_ADDRESS_STR" "),
            pCsrBssDescription->AgingCount,
-           MAC_ADDR_ARRAY(pCsrBssDescription->Result.BssDescriptor.bssId));
+           MAC_ADDR_ARRAY(pBSSDescription->bssId));
         vos_mem_copy(&pCsrBssDescription->Result.BssDescriptor, pBSSDescription, cbBSSDesc );
         //Save SSID separately for later use
         if( pIes->SSID.present && !csrIsNULLSSID(pIes->SSID.ssid, pIes->SSID.num_ssid) )
@@ -5027,8 +5027,9 @@ static tANI_BOOLEAN csrScanProcessScanResults( tpAniSirGlobal pMac, tSmeCmd *pCo
     }while(0);
     if ( eSIR_SME_MORE_SCAN_RESULTS_FOLLOW != pScanRsp->statusCode )
     {
-        smsLog(pMac, LOGE, "Found %d BSS",
-                           csrLLCount(&pMac->scan.tempScanResults));
+        smsLog(pMac, LOGE, "Found %d BSS, statusCode %d",
+                           csrLLCount(&pMac->scan.tempScanResults),
+                           pScanRsp->statusCode);
         smsLog(pMac, LOG1, "scan reason is %d", pCommand->u.scanCmd.reason);
         fRemoveCommand = csrScanComplete( pMac, pScanRsp );
         fRet = eANI_BOOLEAN_TRUE;