Explorar el Código

qcacld-3.0: Remove sta index in few prints

Remove sta index in few prints and replace it
with MAC address as part of sta_index cleanup.

Change-Id: I689ede9dc345667f1a054f76bf3d08b236131c13
CRs-Fixed: 2524505
Yeshwanth Sriram Guntuka hace 5 años
padre
commit
e6b54245d6

+ 1 - 1
components/tdls/core/src/wlan_tdls_ct.c

@@ -585,7 +585,7 @@ tdls_ct_process_idle_handler(struct wlan_objmgr_vdev *vdev,
 		return;
 
 	if (INVALID_TDLS_PEER_ID == tdls_info->sta_id) {
-		tdls_err("peer (staidx %u) doesn't exists", tdls_info->sta_id);
+		tdls_err("peer doesn't exists");
 		return;
 	}
 

+ 4 - 3
core/mac/src/pe/lim/lim_admit_control.c

@@ -604,9 +604,10 @@ lim_validate_access_policy(struct mac_context *mac,
 	}
 
 	if (retval != QDF_STATUS_SUCCESS)
-		pe_warn("accPol: %d staId: %d lle: %d wme: %d wsm: %d",
-			accessPolicy, pSta->staIndex, pSta->lleEnabled,
-			pSta->wmeEnabled, pSta->wsmEnabled);
+		pe_warn("accPol: %d lle: %d wme: %d wsm: %d sta mac "
+			QDF_MAC_ADDR_STR, accessPolicy, pSta->lleEnabled,
+			pSta->wmeEnabled, pSta->wsmEnabled,
+			QDF_MAC_ADDR_ARRAY(pSta->staAddr));
 
 	return retval;
 }

+ 5 - 3
core/mac/src/pe/lim/lim_assoc_utils.c

@@ -3106,7 +3106,8 @@ lim_delete_dph_hash_entry(struct mac_context *mac_ctx, tSirMacAddr sta_addr,
 		return;
 	}
 
-	pe_debug("Deleting DPH Hash entry for STAID: %X", sta_id);
+	pe_debug("Deleting DPH Hash entry sta mac " QDF_MAC_ADDR_STR,
+		 QDF_MAC_ADDR_ARRAY(sta_addr));
 	/*
 	 * update the station count and perform associated actions
 	 * do this before deleting the dph hash entry
@@ -3155,8 +3156,9 @@ lim_delete_dph_hash_entry(struct mac_context *mac_ctx, tSirMacAddr sta_addr,
 
 #ifdef WLAN_FEATURE_11W
 		if (sta_ds->rmfEnabled) {
-			pe_debug("delete pmf timer sta-idx:%d assoc-id:%d",
-				 sta_ds->staIndex, sta_ds->assocId);
+			pe_debug("delete pmf timer assoc-id:%d sta mac "
+				 QDF_MAC_ADDR_STR, sta_ds->assocId,
+				 QDF_MAC_ADDR_ARRAY(sta_ds->staAddr));
 			tx_timer_delete(&sta_ds->pmfSaQueryTimer);
 		}
 #endif

+ 2 - 2
core/mac/src/pe/lim/lim_ibss_peer_mgmt.c

@@ -1087,8 +1087,8 @@ void lim_process_ibss_del_sta_rsp(struct mac_context *mac_ctx,
 		status = eSIR_SME_REFUSED;
 		goto skip_event;
 	}
-	pe_debug("Deleted STA associd %d staId %d MAC " QDF_MAC_ADDR_STR,
-		sta_ds->assocId, sta_ds->staIndex,
+	pe_debug("Deleted STA associd %d MAC " QDF_MAC_ADDR_STR,
+		sta_ds->assocId,
 		QDF_MAC_ADDR_ARRAY(sta_ds->staAddr));
 
 	lim_delete_dph_hash_entry(mac_ctx, sta_ds->staAddr,

+ 5 - 4
core/mac/src/pe/lim/lim_process_assoc_req_frame.c

@@ -1781,8 +1781,8 @@ static bool lim_update_sta_ds(struct mac_context *mac_ctx, tpSirMacMgmtHdr hdr,
 		return false;
 	}
 	if (sta_ds->rmfEnabled)
-	    pe_debug("Created pmf timer sta-idx:%d assoc-id:%d",
-		     sta_ds->staIndex, sta_ds->assocId);
+	    pe_debug("Created pmf timer assoc-id:%d sta mac" QDF_MAC_ADDR_STR,
+		     sta_ds->assocId, QDF_MAC_ADDR_ARRAY(sta_ds->staAddr));
 #endif
 
 	if (assoc_req->ExtCap.present) {
@@ -1862,8 +1862,9 @@ static bool lim_update_sta_ctx(struct mac_context *mac_ctx, struct pe_session *s
 				eLIM_MLM_WT_ASSOC_DEL_STA_RSP_STATE;
 			if (lim_del_sta(mac_ctx, sta_ds, true, session)
 					!= QDF_STATUS_SUCCESS) {
-				pe_err("Couldn't DEL STA, assocId: %d staId: %d",
-					sta_ds->assocId, sta_ds->staIndex);
+				pe_err("Couldn't DEL STA, assocId: %d sta mac"
+				       QDF_MAC_ADDR_STR, sta_ds->assocId,
+				       QDF_MAC_ADDR_ARRAY(sta_ds->staAddr));
 				lim_reject_association(mac_ctx, sta_ds->staAddr,
 					sta_ds->mlmStaContext.subType, true,
 					sta_ds->mlmStaContext.authType,

+ 6 - 5
core/mac/src/pe/lim/lim_process_auth_frame.c

@@ -483,9 +483,9 @@ static void lim_process_auth_frame_type1(struct mac_context *mac_ctx,
 #endif
 		   ) {
 			pe_err("STA is already connected but received auth frame"
-					"Send the Deauth and lim Delete Station Context"
-					"(staId: %d, associd: %d) ",
-				sta_ds_ptr->staIndex, associd);
+			       "Send the Deauth and lim Delete Station Context"
+			       "(associd: %d) sta mac" QDF_MAC_ADDR_STR,
+			       associd, QDF_MAC_ADDR_ARRAY(mac_hdr->sa));
 			lim_send_deauth_mgmt_frame(mac_ctx,
 				eSIR_MAC_UNSPEC_FAILURE_REASON,
 				(uint8_t *) mac_hdr->sa,
@@ -547,8 +547,9 @@ static void lim_process_auth_frame_type1(struct mac_context *mac_ctx,
 			&& !sta_ds_ptr->rmfEnabled
 #endif
 		   ) {
-			pe_debug("lim Delete Station Context staId: %d associd: %d",
-				sta_ds_ptr->staIndex, associd);
+			pe_debug("lim Del Sta Ctx associd: %d sta mac"
+				 QDF_MAC_ADDR_STR, associd,
+				 QDF_MAC_ADDR_ARRAY(sta_ds_ptr->staAddr));
 			lim_send_deauth_mgmt_frame(mac_ctx,
 				eSIR_MAC_UNSPEC_FAILURE_REASON,
 				(uint8_t *)auth_node->peerMacAddr,

+ 6 - 4
core/mac/src/pe/lim/lim_utils.c

@@ -6010,13 +6010,15 @@ void lim_del_pmf_sa_query_timer(struct mac_context *mac_ctx, struct pe_session *
 		if (!sta_ds)
 			continue;
 		if (!sta_ds->rmfEnabled) {
-			pe_debug("no PMF timer for sta-idx:%d assoc-id:%d",
-				 sta_ds->staIndex, sta_ds->assocId);
+			pe_debug("no PMF timer for assoc-id:%d sta mac"
+				 QDF_MAC_ADDR_STR, sta_ds->assocId,
+				 QDF_MAC_ADDR_ARRAY(sta_ds->staAddr));
 			continue;
 		}
 
-		pe_debug("Deleting pmfSaQueryTimer for sta-idx:%d assoc-id:%d",
-			sta_ds->staIndex, sta_ds->assocId);
+		pe_debug("Deleting pmfSaQueryTimer for assoc-id:%d sta mac"
+			 QDF_MAC_ADDR_STR, sta_ds->assocId,
+			 QDF_MAC_ADDR_ARRAY(sta_ds->staAddr));
 		tx_timer_deactivate(&sta_ds->pmfSaQueryTimer);
 		tx_timer_delete(&sta_ds->pmfSaQueryTimer);
 	}

+ 8 - 8
core/wma/src/wma_dev_if.c

@@ -2944,9 +2944,9 @@ int wma_peer_assoc_conf_handler(void *handle, uint8_t *cmd_param_info,
 
 		/* peer assoc conf event means the cmd succeeds */
 		params->status = QDF_STATUS_SUCCESS;
-		WMA_LOGD(FL("Send ADD_STA_RSP: statype %d vdev_id %d aid %d bssid %pM staIdx %d status %d"),
+		WMA_LOGD(FL("Send ADD_STA_RSP: statype %d vdev_id %d aid %d bssid %pM status %d")
 			 params->staType, params->smesessionId,
-			 params->assocId, params->bssId, params->staIdx,
+			 params->assocId, params->bssId,
 			 params->status);
 		wma_send_msg_high_priority(wma, WMA_ADD_STA_RSP,
 					   (void *)params, 0);
@@ -4119,8 +4119,8 @@ static void wma_add_tdls_sta(tp_wma_handle wma, tpAddStaParams add_sta)
 	bool peer_assoc_cnf = false;
 	void *soc = cds_get_context(QDF_MODULE_ID_SOC);
 
-	WMA_LOGD("%s: staType: %d, staIdx: %d, updateSta: %d, bssId: %pM, staMac: %pM",
-		 __func__, add_sta->staType, add_sta->staIdx,
+	WMA_LOGD("%s: staType: %d, updateSta: %d, bssId: %pM, staMac: %pM",
+		 __func__, add_sta->staType,
 		 add_sta->updateSta, add_sta->bssId, add_sta->staMac);
 
 	pdev = cds_get_context(QDF_MODULE_ID_TXRX);
@@ -4174,8 +4174,8 @@ static void wma_add_tdls_sta(tp_wma_handle wma, tpAddStaParams add_sta)
 		}
 
 		add_sta->staIdx = cdp_peer_get_local_peer_id(soc, peer);
-		WMA_LOGD("%s: addSta, after calling cdp_local_peer_id, staIdx: %d, staMac: %pM",
-			 __func__, add_sta->staIdx, add_sta->staMac);
+		WMA_LOGD("%s: addSta, after calling cdp_local_peer_id, staMac: %pM",
+			 __func__, add_sta->staMac);
 
 		peer_state = qdf_mem_malloc(sizeof(*peer_state));
 		if (!peer_state) {
@@ -4249,9 +4249,9 @@ send_rsp:
 	if (peer_assoc_cnf)
 		return;
 
-	WMA_LOGD(FL("statype %d vdev_id %d aid %d bssid %pM staIdx %d status %d"),
+	WMA_LOGD(FL("statype %d vdev_id %d aid %d bssid %pM status %d"),
 		 add_sta->staType, add_sta->smesessionId,
-		 add_sta->assocId, add_sta->bssId, add_sta->staIdx,
+		 add_sta->assocId, add_sta->bssId,
 		 add_sta->status);
 	wma_send_msg_high_priority(wma, WMA_ADD_STA_RSP, (void *)add_sta, 0);
 }