qcacld-3.0: Dont send add peer before peer is deleted in firmware

When IBSS peer is deleted, host send peer delete to firmware
and continue with delete in host. In this case we may hit the
situation that peer is not deleted in firmware and host tries to
add the peer again.

To fix this wait for peer delete resp before proceeding with host
cleanup

Change-Id: I500e4c4a840156295aa00125643f87e23b5168b8
CRs-Fixed: 2047244

core/mac/src/pe/lim/lim_ibss_peer_mgmt.c

@@ -997,32 +997,29 @@ lim_ibss_sta_add(tpAniSirGlobal pMac, void *pBody, tpPESession psessionEntry)
 
 	return retCode;
 }
+
 /**
- * __lim_ibss_search_and_delete_peer()- to cleanup the IBSS peer
+ * lim_ibss_search_and_delete_peer()- to cleanup the IBSS
+ * peer from lim ibss peer list
  *
  * @mac_ptr: Pointer to Global MAC structure
  * @session_entry: Session entry
  * @mac_addr: Mac Address of the IBSS peer
  *
- * This function is called to cleanup the IBSS peer while
- * operating in IBSS mode.
+ * This function is called to cleanup the IBSS peer from
+ * lim ibss peer list
  *
  * Return: None
  *
  */
 static void
-__lim_ibss_search_and_delete_peer(tpAniSirGlobal mac_ptr,
+lim_ibss_search_and_delete_peer(tpAniSirGlobal mac_ctx,
 			tpPESession session_entry, tSirMacAddr mac_addr)
 {
 	tLimIbssPeerNode *temp_node, *prev_node;
 	tLimIbssPeerNode *temp_next_node = NULL;
-	tpDphHashNode sta = NULL;
-	uint16_t peer_idx = 0;
-	uint16_t sta_index = 0;
-	uint8_t uc_ucast_sig;
-	uint8_t uc_bcast_sig;
 
-	prev_node = temp_node = mac_ptr->lim.gLimIbssPeerList;
+	prev_node = temp_node = mac_ctx->lim.gLimIbssPeerList;
 
 	pe_debug(" PEER ADDR :" MAC_ADDRESS_STR,
 		MAC_ADDR_ARRAY(mac_addr));
@@ -1035,55 +1032,22 @@ __lim_ibss_search_and_delete_peer(tpAniSirGlobal mac_ptr,
 		if (!qdf_mem_cmp((uint8_t *) mac_addr,
 				    (uint8_t *) &temp_node->peerMacAddr,
 				    sizeof(tSirMacAddr))) {
-			sta = dph_lookup_hash_entry(mac_ptr, mac_addr,
-							&peer_idx,
-							&session_entry->dph.
-							dphHashTable);
-			if (sta) {
-				sta_index = sta->staIndex;
-				uc_ucast_sig = sta->ucUcastSig;
-				uc_bcast_sig = sta->ucBcastSig;
-				/*
-				 * Send DEL STA only if ADD STA
-				 * was success i.e staid is Valid.
-				 */
-				if (STA_INVALID_IDX != sta_index)
-					lim_del_sta(mac_ptr, sta,
-						  false /*asynchronous */,
-						  session_entry);
-				lim_delete_dph_hash_entry(mac_ptr, sta->staAddr,
-						  peer_idx, session_entry);
-				lim_release_peer_idx(mac_ptr,
-						peer_idx, session_entry);
-
-				/*
-				 * Send indication to upper layers only if ADD
-				 * STA was success i.e staid is Valid.
-				 */
-				if (STA_INVALID_IDX != sta_index)
-					ibss_status_chg_notify(mac_ptr,
-						mac_addr, sta_index,
-						uc_ucast_sig, uc_bcast_sig,
-						eWNI_SME_IBSS_PEER_DEPARTED_IND,
-						session_entry->
-						smeSessionId);
-				if (temp_node ==
-					 mac_ptr->lim.gLimIbssPeerList) {
-					mac_ptr->lim.gLimIbssPeerList =
-						temp_node->next;
-					prev_node =
-						mac_ptr->lim.gLimIbssPeerList;
-				} else
-					prev_node->next = temp_node->next;
-				if (temp_node->beacon)
-					qdf_mem_free(temp_node->beacon);
-
-				qdf_mem_free(temp_node);
-				mac_ptr->lim.gLimNumIbssPeers--;
-
-				temp_node = temp_next_node;
-				break;
-			}
+			if (temp_node ==
+			   mac_ctx->lim.gLimIbssPeerList) {
+				mac_ctx->lim.gLimIbssPeerList =
+					temp_node->next;
+				prev_node =
+					mac_ctx->lim.gLimIbssPeerList;
+			} else
+				prev_node->next = temp_node->next;
+			if (temp_node->beacon)
+				qdf_mem_free(temp_node->beacon);
+
+			qdf_mem_free(temp_node);
+			mac_ctx->lim.gLimNumIbssPeers--;
+
+			temp_node = temp_next_node;
+			break;
 		}
 		prev_node = temp_node;
 		temp_node = temp_next_node;
@@ -1092,12 +1056,120 @@ __lim_ibss_search_and_delete_peer(tpAniSirGlobal mac_ptr,
 	 * if it is the last peer walking out, we better
 	 * we set IBSS state to inactive.
 	 */
-	if (0 == mac_ptr->lim.gLimNumIbssPeers) {
+	if (0 == mac_ctx->lim.gLimNumIbssPeers) {
 		pe_debug("Last STA from IBSS walked out");
 		session_entry->limIbssActive = false;
 	}
 }
 
+/**
+ * lim_ibss_delete_peer()- to delete IBSS peer
+ *
+ * @mac_ptr: Pointer to Global MAC structure
+ * @session_entry: Session entry
+ * @mac_addr: Mac Address of the IBSS peer
+ *
+ * This function is called delete IBSS peer.
+ *
+ * Return: None
+ *
+ */
+static void
+lim_ibss_delete_peer(tpAniSirGlobal mac_ctx,
+			tpPESession session_entry, tSirMacAddr mac_addr)
+{
+	tpDphHashNode sta = NULL;
+	uint16_t peer_idx = 0;
+
+	pe_debug("Delete peer :" MAC_ADDRESS_STR,
+		MAC_ADDR_ARRAY(mac_addr));
+
+	sta = dph_lookup_hash_entry(mac_ctx, mac_addr,
+			&peer_idx,
+			&session_entry->dph.
+			dphHashTable);
+
+	if (!sta) {
+		pe_err("DPH Entry for STA %pM is missing",
+			mac_addr);
+		return;
+	}
+
+	if (STA_INVALID_IDX != sta->staIndex) {
+		lim_del_sta(mac_ctx, sta,
+			  true, session_entry);
+	} else {
+		/*
+		 * This mean ADD STA failed, thus remove the sta from
+		 * from database and no need to send del sta to firmware
+		 * and peer departed indication to upper layer.
+		 */
+		lim_delete_dph_hash_entry(mac_ctx, sta->staAddr,
+			  peer_idx, session_entry);
+		lim_release_peer_idx(mac_ctx,
+			peer_idx, session_entry);
+		lim_ibss_search_and_delete_peer(mac_ctx,
+			session_entry, mac_addr);
+	}
+
+}
+
+void lim_process_ibss_del_sta_rsp(tpAniSirGlobal mac_ctx,
+	struct scheduler_msg *lim_msg,
+	tpPESession pe_session)
+{
+	tpDphHashNode sta_ds = NULL;
+	tpDeleteStaParams del_sta_params = (tpDeleteStaParams) lim_msg->bodyptr;
+	tSirResultCodes status = eSIR_SME_SUCCESS;
+
+	if (!del_sta_params) {
+		pe_err("del_sta_params is NULL");
+		return;
+	}
+	if (!LIM_IS_IBSS_ROLE(pe_session)) {
+		pe_err("Session %d is not IBSS role", del_sta_params->assocId);
+		status = eSIR_SME_REFUSED;
+		goto skip_event;
+	}
+
+	sta_ds = dph_get_hash_entry(mac_ctx, del_sta_params->assocId,
+			&pe_session->dph.dphHashTable);
+	if (!sta_ds) {
+		pe_err("DPH Entry for STA %X is missing",
+			del_sta_params->assocId);
+		status = eSIR_SME_REFUSED;
+		goto skip_event;
+	}
+
+	if (QDF_STATUS_SUCCESS != del_sta_params->status) {
+		pe_err("DEL STA failed!");
+		status = eSIR_SME_REFUSED;
+		goto skip_event;
+	}
+	pe_debug("Deleted STA associd %d staId %d MAC " MAC_ADDRESS_STR,
+		sta_ds->assocId, sta_ds->staIndex,
+		MAC_ADDR_ARRAY(sta_ds->staAddr));
+
+	lim_delete_dph_hash_entry(mac_ctx, sta_ds->staAddr,
+			  del_sta_params->assocId, pe_session);
+	lim_release_peer_idx(mac_ctx,
+			del_sta_params->assocId, pe_session);
+
+	ibss_status_chg_notify(mac_ctx,
+		del_sta_params->staMac,
+		sta_ds->staIndex,
+		sta_ds->ucUcastSig, sta_ds->ucBcastSig,
+		eWNI_SME_IBSS_PEER_DEPARTED_IND,
+		pe_session->smeSessionId);
+
+	lim_ibss_search_and_delete_peer(mac_ctx,
+				pe_session, del_sta_params->staMac);
+
+skip_event:
+	qdf_mem_free(del_sta_params);
+	lim_msg->bodyptr = NULL;
+}
+
 /* handle the response from HAL for an ADD STA request */
 tSirRetStatus
 lim_ibss_add_sta_rsp(tpAniSirGlobal pMac, void *msg, tpPESession psessionEntry)
@@ -1126,7 +1198,7 @@ lim_ibss_add_sta_rsp(tpAniSirGlobal pMac, void *msg, tpPESession psessionEntry)
 		pe_err("IBSS: ADD_STA_RSP error: %x for MAC:"MAC_ADDRESS_STR,
 			pAddStaParams->status,
 			MAC_ADDR_ARRAY(pAddStaParams->staMac));
-		__lim_ibss_search_and_delete_peer(pMac,
+		lim_ibss_delete_peer(pMac,
 			psessionEntry, pAddStaParams->staMac);
 		qdf_mem_free(pAddStaParams);
 		return eSIR_FAILURE;
@@ -1353,7 +1425,7 @@ lim_ibss_coalesce(tpAniSirGlobal pMac,
 		 */
 		pPeerNode = ibss_peer_find(pMac, pHdr->sa);
 		if (NULL != pPeerNode) {
-			__lim_ibss_search_and_delete_peer(pMac, psessionEntry,
+			lim_ibss_delete_peer(pMac, psessionEntry,
 							  pHdr->sa);
 			pe_warn("Peer attempting to reconnect before HB timeout, deleted");
 			return eSIR_LIM_IGNORE_BEACON;
@@ -1687,7 +1759,7 @@ __lim_ibss_peer_inactivity_handler(tpAniSirGlobal pMac,
 	}
 
 	/* delete the peer for which heartbeat is observed */
-	__lim_ibss_search_and_delete_peer(pMac, psessionEntry,
+	lim_ibss_delete_peer(pMac, psessionEntry,
 					  peerInactivityInd->peer_addr.bytes);
 }
 

core/mac/src/pe/lim/lim_ibss_peer_mgmt.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2012, 2014, 2016 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2011-2012, 2014-2017 The Linux Foundation. All rights reserved.
  *
  * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
  *
@@ -46,6 +46,21 @@ tSirRetStatus lim_ibss_coalesce(tpAniSirGlobal, tpSirMacMgmtHdr,
 				tpPESession);
 tSirRetStatus lim_ibss_sta_add(tpAniSirGlobal, void *, tpPESession);
 tSirRetStatus lim_ibss_add_sta_rsp(tpAniSirGlobal, void *, tpPESession);
+
+/**
+ * lim_process_ibss_del_sta_rsp()- Handle ibss delete
+ * peer resp from firmware
+ *
+ * @mac_ptr: Pointer to Global MAC structure
+ * @lim_msg: delete sta response
+ * @pe_session: pe session
+ *
+ * Return: None
+ *
+ */
+void lim_process_ibss_del_sta_rsp(tpAniSirGlobal mac_ctx,
+	struct scheduler_msg *lim_msg,
+	tpPESession pe_session);
 tLimIbssPeerNode *lim_ibss_peer_find(tpAniSirGlobal pMac, tSirMacAddr macAddr);
 void lim_ibss_del_bss_rsp(tpAniSirGlobal, void *, tpPESession);
 void lim_ibss_del_bss_rsp_when_coalescing(tpAniSirGlobal, void *, tpPESession);

core/mac/src/pe/lim/lim_process_mlm_rsp_messages.c

@@ -1766,6 +1766,11 @@ void lim_process_mlm_del_sta_rsp(tpAniSirGlobal mac_ctx,
 				session_entry);
 		return;
 	}
+	if (LIM_IS_IBSS_ROLE(session_entry)) {
+		lim_process_ibss_del_sta_rsp(mac_ctx, msg,
+				session_entry);
+		return;
+	}
 	if (LIM_IS_NDI_ROLE(session_entry)) {
 		lim_process_ndi_del_sta_rsp(mac_ctx, msg, session_entry);
 		return;