فهرست منبع

qcacld-3.0: Reduce the log spam in lim

Move the logs to appropriate log levels to reduce
the log spam in lim_send_frames_host_roam.c and
lim_api.c

Change-Id: Iaaa63ec8be4354d721eafe6c3ca83bf3d1dc4ae9
CRs-Fixed: 1093093
Nishank Aggarwal 8 سال پیش
والد
کامیت
8d4c720cd8
2فایلهای تغییر یافته به همراه67 افزوده شده و 88 حذف شده
  1. 53 72
      core/mac/src/pe/lim/lim_api.c
  2. 14 16
      core/mac/src/pe/lim/lim_send_frames_host_roam.c

+ 53 - 72
core/mac/src/pe/lim/lim_api.c

@@ -268,7 +268,7 @@ static void __lim_init_assoc_vars(tpAniSirGlobal pMac)
 	uint32_t val;
 	if (wlan_cfg_get_int(pMac, WNI_CFG_ASSOC_STA_LIMIT, &val)
 		!= eSIR_SUCCESS)
-		lim_log(pMac, LOGP, FL("cfg get assoc sta limit failed"));
+		lim_log(pMac, LOGE, FL("cfg get assoc sta limit failed"));
 	pMac->lim.gLimAssocStaLimit = val;
 	pMac->lim.gLimIbssStaLimit = val;
 	/* Place holder for current authentication request */
@@ -341,16 +341,14 @@ static tSirRetStatus __lim_init_config(tpAniSirGlobal pMac)
 	 */
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HT_CAP_INFO, &val1) != eSIR_SUCCESS) {
-		PELOGE(lim_log(pMac, LOGE, FL("could not retrieve HT Cap CFG"));)
+		lim_log(pMac, LOGE, FL("could not retrieve HT Cap CFG"));
 		return eSIR_FAILURE;
 	}
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_CHANNEL_BONDING_MODE, &val2) !=
 	    eSIR_SUCCESS) {
-		PELOGE(lim_log
-			       (pMac, LOGE,
-			       FL("could not retrieve Channel Bonding CFG"));
-		       )
+		lim_log(pMac, LOGE,
+			FL("could not retrieve Channel Bonding CFG"));
 		return eSIR_FAILURE;
 	}
 	val16 = (uint16_t) val1;
@@ -364,17 +362,13 @@ static tSirRetStatus __lim_init_config(tpAniSirGlobal pMac)
 	if (cfg_set_int
 		    (pMac, WNI_CFG_HT_CAP_INFO, *(uint16_t *) pHTCapabilityInfo)
 	    != eSIR_SUCCESS) {
-		PELOGE(lim_log
-			       (pMac, LOGE, FL("could not update HT Cap Info CFG"));
-		       )
+		lim_log(pMac, LOGE, FL("could not update HT Cap Info CFG"));
 		return eSIR_FAILURE;
 	}
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HT_INFO_FIELD1, &val1) != eSIR_SUCCESS) {
-		PELOGE(lim_log
-			       (pMac, LOGE,
-			       FL("could not retrieve HT INFO Field1 CFG"));
-		       )
+		lim_log(pMac, LOGE,
+			FL("could not retrieve HT INFO Field1 CFG"));
 		return eSIR_FAILURE;
 	}
 
@@ -384,7 +378,7 @@ static tSirRetStatus __lim_init_config(tpAniSirGlobal pMac)
 		(uint8_t) pHTCapabilityInfo->supportedChannelWidthSet;
 	if (cfg_set_int(pMac, WNI_CFG_HT_INFO_FIELD1, *(uint8_t *) pHTInfoField1)
 	    != eSIR_SUCCESS) {
-		PELOGE(lim_log(pMac, LOGE, FL("could not update HT Info Field"));)
+		lim_log(pMac, LOGE, FL("could not update HT Info Field"));
 		return eSIR_FAILURE;
 	}
 
@@ -392,11 +386,8 @@ static tSirRetStatus __lim_init_config(tpAniSirGlobal pMac)
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HEART_BEAT_THRESHOLD, &val1) !=
 	    eSIR_SUCCESS) {
-		PELOGE(lim_log
-			       (pMac, LOGE,
-			       FL
-				       ("could not retrieve WNI_CFG_HEART_BEAT_THRESHOLD CFG"));
-		       )
+		lim_log(pMac, LOGE,
+			FL("could not retrieve WNI_CFG_HEART_BEAT_THRESHOLD CFG"));
 		return eSIR_FAILURE;
 	}
 	if (!val1) {
@@ -438,7 +429,7 @@ static tSirRetStatus __lim_init_config(tpAniSirGlobal pMac)
 	/* WNI_CFG_SHORT_PREAMBLE - this one is not updated in
 	   lim_handle_cf_gparam_update do we want to update this? */
 	if (wlan_cfg_get_int(pMac, WNI_CFG_SHORT_PREAMBLE, &val1) != eSIR_SUCCESS) {
-		lim_log(pMac, LOGP, FL("cfg get short preamble failed"));
+		lim_log(pMac, LOGE, FL("cfg get short preamble failed"));
 		return eSIR_FAILURE;
 	}
 
@@ -450,34 +441,34 @@ static tSirRetStatus __lim_init_config(tpAniSirGlobal pMac)
 	if (eSIR_SUCCESS !=
 	    wlan_cfg_get_int(pMac, WNI_CFG_DISABLE_LDPC_WITH_TXBF_AP,
 			     (uint32_t *) &pMac->lim.disableLDPCWithTxbfAP)) {
-		lim_log(pMac, LOGP, FL("cfg get disableLDPCWithTxbfAP failed"));
+		lim_log(pMac, LOGE, FL("cfg get disableLDPCWithTxbfAP failed"));
 		return eSIR_FAILURE;
 	}
 #ifdef FEATURE_WLAN_TDLS
 	if (eSIR_SUCCESS != wlan_cfg_get_int(pMac, WNI_CFG_TDLS_BUF_STA_ENABLED,
 					     (uint32_t *) &pMac->lim.
 					     gLimTDLSBufStaEnabled)) {
-		lim_log(pMac, LOGP, FL("cfg get LimTDLSBufStaEnabled failed"));
+		lim_log(pMac, LOGE, FL("cfg get LimTDLSBufStaEnabled failed"));
 		return eSIR_FAILURE;
 	}
 	if (eSIR_SUCCESS !=
 	    wlan_cfg_get_int(pMac, WNI_CFG_TDLS_QOS_WMM_UAPSD_MASK,
 			     (uint32_t *) &pMac->lim.gLimTDLSUapsdMask)) {
-		lim_log(pMac, LOGP, FL("cfg get LimTDLSUapsdMask failed"));
+		lim_log(pMac, LOGE, FL("cfg get LimTDLSUapsdMask failed"));
 		return eSIR_FAILURE;
 	}
 	if (eSIR_SUCCESS !=
 	    wlan_cfg_get_int(pMac, WNI_CFG_TDLS_OFF_CHANNEL_ENABLED,
 			     (uint32_t *) &pMac->lim.
 			     gLimTDLSOffChannelEnabled)) {
-		lim_log(pMac, LOGP, FL("cfg get LimTDLSUapsdMask failed"));
+		lim_log(pMac, LOGE, FL("cfg get LimTDLSUapsdMask failed"));
 		return eSIR_FAILURE;
 	}
 
 	if (eSIR_SUCCESS != wlan_cfg_get_int(pMac, WNI_CFG_TDLS_WMM_MODE_ENABLED,
 					     (uint32_t *) &pMac->lim.
 					     gLimTDLSWmmMode)) {
-		lim_log(pMac, LOGP, FL("cfg get LimTDLSWmmMode failed"));
+		lim_log(pMac, LOGE, FL("cfg get LimTDLSWmmMode failed"));
 		return eSIR_FAILURE;
 	}
 #endif
@@ -493,7 +484,7 @@ tSirRetStatus lim_start(tpAniSirGlobal pMac)
 {
 	tSirResultCodes retCode = eSIR_SUCCESS;
 
-	PELOG1(lim_log(pMac, LOG1, FL(" enter"));)
+	lim_log(pMac, LOGD, FL(" enter"));
 
 	if (pMac->lim.gLimSmeState == eLIM_SME_OFFLINE_STATE) {
 		pMac->lim.gLimSmeState = eLIM_SME_IDLE_STATE;
@@ -703,7 +694,7 @@ static void lim_state_info_dump(char **buf_ptr, uint16_t *size)
 
 	mac = PMAC_STRUCT(hal);
 
-	lim_log(mac, LOG1, FL("size of buffer: %d"), *size);
+	lim_log(mac, LOGD, FL("size of buffer: %d"), *size);
 
 	len += qdf_scnprintf(buf + len, *size - len,
 		"\n SmeState: %d", mac->lim.gLimSmeState);
@@ -773,16 +764,16 @@ tSirRetStatus pe_open(tpAniSirGlobal pMac, struct cds_config_info *cds_cfg)
 	qdf_spinlock_create(&pMac->sys.bbt_mgmt_lock);
 
 	if ((pMac->lim.maxBssId == 0) || (pMac->lim.maxStation == 0)) {
-		PELOGE(lim_log(pMac, LOGE,
-			FL("max number of Bssid or Stations cannot be zero!"));)
+		lim_log(pMac, LOGE,
+			FL("max number of Bssid or Stations cannot be zero!"));
 		return eSIR_FAILURE;
 	}
 
 	pMac->lim.limTimers.gpLimCnfWaitTimer =
 		qdf_mem_malloc(sizeof(TX_TIMER) * (pMac->lim.maxStation + 1));
 	if (NULL == pMac->lim.limTimers.gpLimCnfWaitTimer) {
-		PELOGE(lim_log(pMac, LOGE,
-			FL("gpLimCnfWaitTimer memory allocate failed!"));)
+		lim_log(pMac, LOGE,
+			FL("gpLimCnfWaitTimer memory allocate failed!"));
 		return eSIR_MEM_ALLOC_FAILED;
 	}
 
@@ -800,7 +791,7 @@ tSirRetStatus pe_open(tpAniSirGlobal pMac, struct cds_config_info *cds_cfg)
 	pMac->lim.deferredMsgCnt = 0;
 
 	if (!QDF_IS_STATUS_SUCCESS(qdf_mutex_create(&pMac->lim.lkPeGlobalLock))) {
-		PELOGE(lim_log(pMac, LOGE, FL("pe lock init failed!"));)
+		lim_log(pMac, LOGE, FL("pe lock init failed!"));
 		status = eSIR_FAILURE;
 		goto pe_open_lock_fail;
 	}
@@ -1148,22 +1139,22 @@ static QDF_STATUS pe_handle_mgmt_frame(struct wlan_objmgr_psoc *psoc,
 
 	mHdr = WMA_GET_RX_MAC_HEADER(pRxPacketInfo);
 	if (mHdr->fc.type == SIR_MAC_MGMT_FRAME) {
-		lim_log(pMac, LOG2, FL
+		lim_log(pMac, LOGD, FL
 		  ("RxBd=%p mHdr=%p Type: %d Subtype: %d  Sizes:FC%zu Mgmt%zu"),
 		  pRxPacketInfo, mHdr, mHdr->fc.type, mHdr->fc.subType,
 		  sizeof(tSirMacFrameCtl), sizeof(tSirMacMgmtHdr));
 
-		lim_log(pMac, LOG2, FL("mpdu_len:%d hdr_len:%d data_len:%d"),
+		lim_log(pMac, LOGD, FL("mpdu_len:%d hdr_len:%d data_len:%d"),
 		       WMA_GET_RX_MPDU_LEN(pRxPacketInfo),
 		       WMA_GET_RX_MPDU_HEADER_LEN(pRxPacketInfo),
 		       WMA_GET_RX_PAYLOAD_LEN(pRxPacketInfo));
 
 		if (WMA_GET_ROAMCANDIDATEIND(pRxPacketInfo))
-			lim_log(pMac, LOG1, FL("roamCandidateInd %d"),
+			lim_log(pMac, LOGD, FL("roamCandidateInd %d"),
 				WMA_GET_ROAMCANDIDATEIND(pRxPacketInfo));
 
 		if (WMA_GET_OFFLOADSCANLEARN(pRxPacketInfo))
-			lim_log(pMac, LOG1, FL("offloadScanLearn %d"),
+			lim_log(pMac, LOGD, FL("offloadScanLearn %d"),
 				 WMA_GET_OFFLOADSCANLEARN(pRxPacketInfo));
 	}
 
@@ -1247,7 +1238,7 @@ void pe_register_callbacks_with_wma(tpAniSirGlobal pMac,
 			ready_req->csr_roam_synch_cb,
 			ready_req->pe_roam_synch_cb);
 	if (retStatus != QDF_STATUS_SUCCESS)
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Registering roaming callbacks with WMA failed"));
 
 	retStatus = wma_register_ndp_cb(lim_handle_ndp_event_message);
@@ -1351,7 +1342,7 @@ lim_update_overlap_sta_param(tpAniSirGlobal pMac, tSirMacAddr bssId,
 	}
 
 	if (i == LIM_PROT_STA_OVERLAP_CACHE_SIZE) {
-		PELOG1(lim_log(pMac, LOGW, FL("Overlap cache is full"));)
+		lim_log(pMac, LOGW, FL("Overlap cache is full"));
 	} else {
 		qdf_mem_copy(pMac->lim.protStaOverlapCache[i].addr,
 			     bssId, sizeof(tSirMacAddr));
@@ -1448,13 +1439,11 @@ lim_handle_ibss_coalescing(tpAniSirGlobal pMac,
 	    (psessionEntry->currentOperChannel != pBeacon->channelNumber))
 		retCode = eSIR_LIM_IGNORE_BEACON;
 	else if (lim_ibss_enc_type_matched(pBeacon, psessionEntry) != eSIR_TRUE) {
-		PELOG3(lim_log(pMac, LOG3,
-			       FL
-				       ("peer privacy %d peer wpa %d peer rsn %d self encType %d"),
+		lim_log(pMac, LOGD,
+			FL("peer privacy %d peer wpa %d peer rsn %d self encType %d"),
 			       pBeacon->capabilityInfo.privacy,
 			       pBeacon->wpaPresent, pBeacon->rsnPresent,
 			       psessionEntry->encryptType);
-		       )
 		retCode = eSIR_LIM_IGNORE_BEACON;
 	} else {
 		uint32_t ieLen;
@@ -1463,10 +1452,8 @@ lim_handle_ibss_coalescing(tpAniSirGlobal pMac,
 		ieLen = WMA_GET_RX_PAYLOAD_LEN(pRxPacketInfo);
 		tsfLater = WMA_GET_RX_TSF_LATER(pRxPacketInfo);
 		pIEs = WMA_GET_RX_MPDU_DATA(pRxPacketInfo);
-		PELOG3(lim_log
-			       (pMac, LOG3, FL("BEFORE Coalescing tsfLater val :%d"),
+		lim_log(pMac, LOGD, FL("BEFORE Coalescing tsfLater val :%d"),
 			       tsfLater);
-		       )
 		retCode =
 			lim_ibss_coalesce(pMac, pHdr, pBeacon, pIEs, ieLen, tsfLater,
 					  psessionEntry);
@@ -1647,12 +1634,10 @@ lim_detect_change_in_ap_capabilities(tpAniSirGlobal pMac,
 		qdf_mem_copy(apNewCaps.bssId.bytes,
 			     psessionEntry->bssId, QDF_MAC_ADDR_SIZE);
 		if (newChannel != psessionEntry->currentOperChannel) {
-			PELOGE(lim_log
-				       (pMac, LOGE,
-				       FL("Channel Change from %d --> %d  - "
+			lim_log(pMac, LOGE,
+				FL("Channel Change from %d --> %d  - "
 					  "Ignoring beacon!"),
-				       psessionEntry->currentOperChannel, newChannel);
-			       )
+				psessionEntry->currentOperChannel, newChannel);
 			return;
 		}
 
@@ -1668,11 +1653,9 @@ lim_detect_change_in_ap_capabilities(tpAniSirGlobal pMac,
 		 */
 		else if ((SIR_MAC_GET_PRIVACY(apNewCaps.capabilityInfo) == 0) &&
 			 (pBeacon->rsnPresent || pBeacon->wpaPresent)) {
-			PELOGE(lim_log
-				       (pMac, LOGE,
-				       FL("BSS Caps (Privacy) bit 0 in beacon,"
-					  " but WPA or RSN IE present, Ignore Beacon!"));
-			       )
+			lim_log(pMac, LOGE,
+				FL("BSS Caps (Privacy) bit 0 in beacon,"
+				" but WPA or RSN IE present, Ignore Beacon!"));
 			return;
 		} else
 			apNewCaps.channelId = psessionEntry->currentOperChannel;
@@ -1691,7 +1674,7 @@ lim_detect_change_in_ap_capabilities(tpAniSirGlobal pMac,
 		 * will come here. If beacon is with broadcast ssid then fWaitForProbeRsp
 		 * will be false, the control will not come here*/
 
-		lim_log(pMac, LOG1, FL("capabilities in probe response are"
+		lim_log(pMac, LOGD, FL("capabilities in probe response are"
 				       "matching with the current setting,"
 				       "Ignoring subsequent capability"
 				       "mismatch"));
@@ -1732,7 +1715,7 @@ tSirRetStatus lim_update_short_slot(tpAniSirGlobal pMac,
 	/* Check Admin mode first. If it is disabled just return */
 	if (wlan_cfg_get_int(pMac, WNI_CFG_11G_SHORT_SLOT_TIME_ENABLED, &val)
 	    != eSIR_SUCCESS) {
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("cfg get WNI_CFG_11G_SHORT_SLOT_TIME failed"));
 		return eSIR_FAILURE;
 	}
@@ -1771,11 +1754,9 @@ tSirRetStatus lim_update_short_slot(tpAniSirGlobal pMac,
 
 	if (nShortSlot != psessionEntry->shortSlotTimeSupported) {
 		/* Short slot time capability of AP has changed. Adopt to it. */
-		PELOG1(lim_log
-			       (pMac, LOG1,
-			       FL("Shortslot capability of AP changed: %d"),
+		lim_log(pMac, LOGD,
+			FL("Shortslot capability of AP changed: %d"),
 			       nShortSlot);
-		       )
 			((tpSirMacCapabilityInfo) & psessionEntry->
 			limCurrentBssCaps)->shortSlotTime = (uint16_t) nShortSlot;
 		psessionEntry->shortSlotTimeSupported = nShortSlot;
@@ -1796,7 +1777,7 @@ void lim_send_heart_beat_timeout_ind(tpAniSirGlobal pMac, tpPESession psessionEn
 	msg.type = (uint16_t) SIR_LIM_HEART_BEAT_TIMEOUT;
 	msg.bodyptr = psessionEntry;
 	msg.bodyval = 0;
-	lim_log(pMac, LOGE, FL("Heartbeat failure from Fw"));
+	lim_log(pMac, LOG1, FL("Heartbeat failure from Fw"));
 
 	statusCode = lim_post_msg_api(pMac, &msg);
 
@@ -1833,8 +1814,8 @@ void lim_ps_offload_handle_missed_beacon_ind(tpAniSirGlobal pMac,
 
 	/* Set Beacon Miss in Powersave Offload */
 	psessionEntry->pmmOffloadInfo.bcnmiss = true;
-	PELOGE(lim_log(pMac, LOGE,
-		       FL("Received Heart Beat Failure"));)
+	lim_log(pMac, LOG1,
+		FL("Received Heart Beat Failure"));
 
 	/*  Do AP probing immediately */
 	lim_send_heart_beat_timeout_ind(pMac, psessionEntry);
@@ -1911,9 +1892,9 @@ lim_roam_fill_bss_descr(tpAniSirGlobal pMac,
 		return QDF_STATUS_E_FAILURE;
 	}
 
-	QDF_TRACE(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_INFO,
+	QDF_TRACE(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
 		"LFR3:Beacon/Prb Rsp:%d", roam_offload_synch_ind_ptr->isBeacon);
-	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_INFO,
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
 	bcn_proberesp_ptr, roam_offload_synch_ind_ptr->beaconProbeRespLength);
 	if (roam_offload_synch_ind_ptr->isBeacon) {
 		if (sir_parse_beacon_ie(pMac, parsed_frm_ptr,
@@ -2065,8 +2046,8 @@ QDF_STATUS pe_roam_synch_callback(tpAniSirGlobal mac_ctx,
 		lim_log(mac_ctx, LOGE, FL("LFR3:roam_sync_ind_ptr is NULL"));
 		return status;
 	}
-	lim_log(mac_ctx, LOG1, FL("LFR3:Received WMA_ROAM_OFFLOAD_SYNCH_IND"));
-	lim_log(mac_ctx, QDF_TRACE_LEVEL_DEBUG, FL("LFR3:auth=%d, vdevId=%d"),
+	lim_log(mac_ctx, QDF_TRACE_LEVEL_DEBUG,
+		FL("LFR3:Received WMA_ROAM_OFFLOAD_SYNCH_IND LFR3:auth=%d, vdevId=%d"),
 		roam_sync_ind_ptr->authStatus, roam_sync_ind_ptr->roamedVdevId);
 	lim_print_mac_addr(mac_ctx, roam_sync_ind_ptr->bssid.bytes,
 			QDF_TRACE_LEVEL_DEBUG);
@@ -2169,8 +2150,8 @@ QDF_STATUS pe_roam_synch_callback(tpAniSirGlobal mac_ctx,
 			roam_sync_ind_ptr->reassocRespOffset,
 			mac_ctx->roam.reassocRespLen);
 
-	lim_log(mac_ctx, LOG1, FL("LFR3:the reassoc resp frame data:"));
-	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_INFO,
+	lim_log(mac_ctx, LOGD, FL("LFR3:the reassoc resp frame data:"));
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_DEBUG,
 			mac_ctx->roam.pReassocResp,
 			mac_ctx->roam.reassocRespLen);
 	ft_session_ptr->bRoamSynchInProgress = true;
@@ -2187,7 +2168,7 @@ QDF_STATUS pe_roam_synch_callback(tpAniSirGlobal mac_ctx,
 
 #ifdef FEATURE_WLAN_ESE
 	join_rsp_len += ft_session_ptr->tspecLen;
-	lim_log(mac_ctx, LOG1, FL("tspecLen = %d"),
+	lim_log(mac_ctx, LOGD, FL("tspecLen = %d"),
 			ft_session_ptr->tspecLen);
 #endif
 
@@ -2201,7 +2182,7 @@ QDF_STATUS pe_roam_synch_callback(tpAniSirGlobal mac_ctx,
 		return QDF_STATUS_E_NOMEM;
 	}
 
-	lim_log(mac_ctx, LOG1, FL("Session RicLength = %d"),
+	lim_log(mac_ctx, LOGD, FL("Session RicLength = %d"),
 			ft_session_ptr->RICDataLen);
 	if (ft_session_ptr->ricData != NULL) {
 		roam_sync_ind_ptr->join_rsp->parsedRicRspLen =

+ 14 - 16
core/mac/src/pe/lim/lim_send_frames_host_roam.c

@@ -99,7 +99,7 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 
 	add_ie_len = pe_session->pLimReAssocReq->addIEAssoc.length;
 	add_ie = pe_session->pLimReAssocReq->addIEAssoc.addIEdata;
-	lim_log(mac_ctx, LOG1,
+	lim_log(mac_ctx, LOGD,
 		FL("called in state (%d)."), pe_session->limMlmState);
 
 	qdf_mem_set((uint8_t *) &frm, sizeof(frm), 0);
@@ -294,7 +294,7 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 	}
 	if (pe_session->vhtCapability &&
 	    pe_session->vhtCapabilityPresentInBeacon) {
-		lim_log(mac_ctx, LOG1,
+		lim_log(mac_ctx, LOGD,
 			FL("Populate VHT IEs in Re-Assoc Request"));
 		populate_dot11f_vht_caps(mac_ctx, pe_session, &frm.VHTCaps);
 		vht_enabled = true;
@@ -303,7 +303,7 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 	}
 	if (!vht_enabled &&
 			pe_session->is_vendor_specific_vhtcaps) {
-		lim_log(mac_ctx, LOG1,
+		lim_log(mac_ctx, LOGD,
 			FL("Populate Vendor VHT IEs in Re-Assoc Request"));
 		frm.vendor_vht_ie.present = 1;
 		frm.vendor_vht_ie.type =
@@ -325,7 +325,7 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 	status = dot11f_get_packed_re_assoc_request_size(mac_ctx, &frm,
 			&payload);
 	if (DOT11F_FAILED(status)) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 			FL("Failure in size calculation (0x%08x)."), status);
 		/* We'll fall back on the worst case scenario: */
 		payload = sizeof(tDot11fReAssocRequest);
@@ -336,7 +336,7 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 
 	bytes = payload + sizeof(tSirMacMgmtHdr) + add_ie_len;
 
-	lim_log(mac_ctx, LOG1, FL("FT IE Reassoc Req (%d)."),
+	lim_log(mac_ctx, LOGD, FL("FT IE Reassoc Req (%d)."),
 		ft_sme_context->reassoc_ft_ies_length);
 
 	if (pe_session->is11Rconnection)
@@ -349,13 +349,13 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 		MTRACE(qdf_trace(QDF_MODULE_ID_PE, TRACE_CODE_MLM_STATE,
 				 pe_session->peSessionId,
 				 pe_session->limMlmState));
-		lim_log(mac_ctx, LOGP, FL("Failed to alloc memory %d"), bytes);
+		lim_log(mac_ctx, LOGE, FL("Failed to alloc memory %d"), bytes);
 		goto end;
 	}
 	/* Paranoia: */
 	qdf_mem_set(frame, bytes + ft_ies_length, 0);
 
-	lim_print_mac_addr(mac_ctx, pe_session->limReAssocbssId, LOG1);
+	lim_print_mac_addr(mac_ctx, pe_session->limReAssocbssId, LOGD);
 	/* Next, we fill out the buffer descriptor: */
 	lim_populate_mac_header(mac_ctx, frame, SIR_MAC_MGMT_FRAME,
 		SIR_MAC_MGMT_REASSOC_REQ, pe_session->limReAssocbssId,
@@ -374,7 +374,7 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 			status);
 	}
 
-	lim_log(mac_ctx, LOG3,
+	lim_log(mac_ctx, LOGD,
 		       FL("*** Sending Re-Assoc Request length %d %d to "),
 		       bytes, payload);
 
@@ -447,7 +447,7 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 			pe_session->assocReqLen = ft_ies_length;
 		}
 	} else {
-		lim_log(mac_ctx, LOG1, FL("FT IEs not present"));
+		lim_log(mac_ctx, LOGD, FL("FT IEs not present"));
 		pe_session->assocReqLen = 0;
 	}
 
@@ -509,7 +509,7 @@ void lim_send_retry_reassoc_req_frame(tpAniSirGlobal pMac,
 	    != TX_SUCCESS) {
 		/* Could not start reassoc failure timer. */
 		/* Log error */
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("could not start Reassociation failure timer"));
 		/* Return Reassoc confirm with */
 		/* Resources Unavailable */
@@ -706,7 +706,7 @@ void lim_send_reassoc_req_mgmt_frame(tpAniSirGlobal pMac,
 	nStatus =
 		dot11f_get_packed_re_assoc_request_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Fail to get size:ReassocReq(0x%08x)"),
+		lim_log(pMac, LOGE, FL("Fail to get size:ReassocReq(0x%08x)"),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
 		nPayload = sizeof(tDot11fReAssocRequest);
@@ -724,7 +724,7 @@ void lim_send_reassoc_req_mgmt_frame(tpAniSirGlobal pMac,
 		MTRACE(qdf_trace(QDF_MODULE_ID_PE, TRACE_CODE_MLM_STATE,
 				 psessionEntry->peSessionId,
 				 psessionEntry->limMlmState));
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Failed to alloc %d bytes for a ReAssociation Req."),
 			nBytes);
 		goto end;
@@ -752,11 +752,9 @@ void lim_send_reassoc_req_mgmt_frame(tpAniSirGlobal pMac,
 			nStatus);
 	}
 
-	PELOG1(lim_log
-		       (pMac, LOG1,
-		       FL("*** Sending Re-Association Request length %d" "to "),
+	lim_log(pMac, LOGD,
+		FL("*** Sending Re-Association Request length %d" "to "),
 		       nBytes);
-	       )
 
 	if (psessionEntry->assocReq != NULL) {
 		qdf_mem_free(psessionEntry->assocReq);