wlan: Missing error and info logs in PE disconnect path (Part 2)
added info and error logs for disconnect path
in Dissoc req/rsp and Deauth req/rsp
Change-Id: Ib2e1d4e6feca4bb762ca7f72d3abc508cd153153
CRs-Fixed: 608938
diff --git a/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c b/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c
index 05b1837..10d64ef 100644
--- a/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c
+++ b/CORE/MAC/src/pe/lim/limProcessSmeReqMessages.c
@@ -2351,7 +2351,6 @@
tANI_U8 smesessionId;
tANI_U16 smetransactionId;
- PELOG1(limLog(pMac, LOG1,FL("received DISASSOC_REQ message"));)
if (pMsgBuf == NULL)
{
@@ -2381,19 +2380,20 @@
return;
}
-
- PELOG1(limLog(pMac, LOG1, FL("received DISASSOC_REQ message. Reason: %d global SmeState: %d"),
- smeDisassocReq.reasonCode, pMac->lim.gLimSmeState);)
-
-
if((psessionEntry = peFindSessionByBssid(pMac,smeDisassocReq.bssId,&sessionId))== NULL)
{
- limLog(pMac, LOGE,FL("session does not exist for given bssId"));
+ limLog(pMac, LOGE,FL("session does not exist for given bssId "MAC_ADDRESS_STR),
+ MAC_ADDR_ARRAY(smeDisassocReq.bssId));
retCode = eSIR_SME_INVALID_PARAMETERS;
disassocTrigger = eLIM_HOST_DISASSOC;
goto sendDisassoc;
}
+ limLog(pMac, LOG1, FL("received DISASSOC_REQ message on sessionid %d"
+ "Systemrole %d Reason: %u SmeState: %d from: "MAC_ADDRESS_STR),
+ smesessionId,psessionEntry->limSystemRole,
+ smeDisassocReq.reasonCode, pMac->lim.gLimSmeState,
+ MAC_ADDR_ARRAY(smeDisassocReq.peerMacAddr));
#ifdef FEATURE_WLAN_DIAG_SUPPORT_LIM //FEATURE_WLAN_DIAG_SUPPORT
limDiagEventReport(pMac, WLAN_PE_DIAG_DISASSOC_REQ_EVENT, psessionEntry, 0, smeDisassocReq.reasonCode);
@@ -2419,6 +2419,8 @@
limDeleteTDLSPeers(pMac, psessionEntry);
#endif
MTRACE(macTrace(pMac, TRACE_CODE_SME_STATE, psessionEntry->peSessionId, psessionEntry->limSmeState));
+ limLog(pMac, LOG1, FL("Rcvd SME_DISASSOC_REQ while in "
+ "limSmeState: %d "),psessionEntry->limSmeState);
break;
case eLIM_SME_WT_DEAUTH_STATE:
@@ -2429,7 +2431,8 @@
*/
psessionEntry->limSmeState= eLIM_SME_WT_DISASSOC_STATE;
MTRACE(macTrace(pMac, TRACE_CODE_SME_STATE, psessionEntry->peSessionId, psessionEntry->limSmeState));
- limLog(pMac, LOG1, FL("Rcvd SME_DISASSOC_REQ while in SME_WT_DEAUTH_STATE. "));
+ limLog(pMac, LOG1, FL("Rcvd SME_DISASSOC_REQ while in "
+ "SME_WT_DEAUTH_STATE. "));
break;
case eLIM_SME_WT_DISASSOC_STATE:
@@ -2442,11 +2445,14 @@
* It will send a disassoc, which is ok. However, we can use the global flag
* sendDisassoc to not send disassoc frame.
*/
- limLog(pMac, LOG1, FL("Rcvd SME_DISASSOC_REQ while in SME_WT_DISASSOC_STATE. "));
+ limLog(pMac, LOG1, FL("Rcvd SME_DISASSOC_REQ while in "
+ "SME_WT_DISASSOC_STATE. "));
break;
case eLIM_SME_JOIN_FAILURE_STATE: {
/** Return Success as we are already in Disconnected State*/
+ limLog(pMac, LOG1, FL("Rcvd SME_DISASSOC_REQ while in "
+ "eLIM_SME_JOIN_FAILURE_STATE. "));
if (pMac->lim.gLimRspReqd) {
retCode = eSIR_SME_SUCCESS;
disassocTrigger = eLIM_HOST_DISASSOC;
@@ -2498,6 +2504,7 @@
if (smeDisassocReq.reasonCode == eLIM_LINK_MONITORING_DISASSOC)
{
/// Disassociation is triggered by Link Monitoring
+ limLog(pMac, LOG1, FL("**** Lost link with AP ****"));
disassocTrigger = eLIM_LINK_MONITORING_DISASSOC;
reasonCode = eSIR_MAC_DISASSOC_DUE_TO_INACTIVITY_REASON;
}
@@ -2509,6 +2516,7 @@
if (smeDisassocReq.doNotSendOverTheAir)
{
+ limLog(pMac, LOG1, FL("do not send dissoc over the air"));
sendDisassocFrame = 0;
}
// Trigger Disassociation frame to peer MAC entity
@@ -2594,7 +2602,7 @@
if (!limIsSmeDisassocCnfValid(pMac, &smeDisassocCnf, psessionEntry))
{
- limLog(pMac, LOGW, FL("received invalid SME_DISASSOC_CNF message"));
+ limLog(pMac, LOGE, FL("received invalid SME_DISASSOC_CNF message"));
return;
}
@@ -2643,8 +2651,9 @@
pStaDs = dphLookupHashEntry(pMac, smeDisassocCnf.peerMacAddr, &aid, &psessionEntry->dph.dphHashTable);
if (pStaDs == NULL)
{
- PELOGW(limLog(pMac, LOGW, FL("received DISASSOC_CNF for a STA that does not have context, addr= "));
- limPrintMacAddr(pMac, smeDisassocCnf.peerMacAddr, LOGW);)
+ PELOGE(limLog(pMac, LOGE, FL("received DISASSOC_CNF for a STA that "
+ "does not have context, addr= "MAC_ADDRESS_STR),
+ MAC_ADDR_ARRAY(smeDisassocCnf.peerMacAddr));)
return;
}
/* Delete FT session if there exists one */
@@ -2689,10 +2698,8 @@
tANI_U8 smesessionId;
tANI_U16 smetransactionId;
- PELOG1(limLog(pMac, LOG1,FL("received DEAUTH_REQ message"));)
status = limDeauthReqSerDes(pMac, &smeDeauthReq,(tANI_U8 *) pMsgBuf);
-
limGetSessionInfo(pMac,(tANI_U8 *)pMsgBuf,&smesessionId,&smetransactionId);
//We need to get a session first but we don't even know if the message is correct.
@@ -2719,7 +2726,11 @@
return;
}
-
+ limLog(pMac, LOG1,FL("received DEAUTH_REQ message on sessionid %d "
+ "Systemrole %d with reasoncode %u in limSmestate %d from "
+ MAC_ADDRESS_STR), smesessionId, psessionEntry->limSystemRole,
+ smeDeauthReq.reasonCode, psessionEntry->limSmeState,
+ MAC_ADDR_ARRAY(smeDeauthReq.peerMacAddr));
#ifdef FEATURE_WLAN_DIAG_SUPPORT_LIM //FEATURE_WLAN_DIAG_SUPPORT
limDiagEventReport(pMac, WLAN_PE_DIAG_DEAUTH_REQ_EVENT, psessionEntry, 0, smeDeauthReq.reasonCode);
#endif //FEATURE_WLAN_DIAG_SUPPORT
@@ -2755,7 +2766,8 @@
* peer. Log error and send response to host.
*/
limLog(pMac, LOGE,
- FL("received unexp SME_DEAUTH_REQ in state %X"),psessionEntry->limSmeState);
+ FL("received unexp SME_DEAUTH_REQ in state %X"),
+ psessionEntry->limSmeState);
limPrintSmeState(pMac, LOGE, psessionEntry->limSmeState);
if (pMac->lim.gLimRspReqd)
@@ -2783,7 +2795,8 @@
default:
limLog(pMac, LOGE,
- FL("received unexpected SME_DEAUTH_REQ for role %X"),psessionEntry->limSystemRole);
+ FL("received unexpected SME_DEAUTH_REQ for role %X"),
+ psessionEntry->limSystemRole);
return;
} // end switch (pMac->lim.gLimSystemRole)