From 8d4c720cd828467512c7a0734d51611dfeaf5437 Mon Sep 17 00:00:00 2001 From: Nishank Aggarwal Date: Fri, 10 Mar 2017 13:13:54 +0530 Subject: [PATCH] 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 --- core/mac/src/pe/lim/lim_api.c | 125 ++++++++---------- .../src/pe/lim/lim_send_frames_host_roam.c | 30 ++--- 2 files changed, 67 insertions(+), 88 deletions(-) diff --git a/core/mac/src/pe/lim/lim_api.c b/core/mac/src/pe/lim/lim_api.c index 66b7259342..fd3a4c2454 100644 --- a/core/mac/src/pe/lim/lim_api.c +++ b/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 = diff --git a/core/mac/src/pe/lim/lim_send_frames_host_roam.c b/core/mac/src/pe/lim/lim_send_frames_host_roam.c index 8d7354ca0c..1f62a64398 100644 --- a/core/mac/src/pe/lim/lim_send_frames_host_roam.c +++ b/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);