Bläddra i källkod

qcacld-3.0: Fix logging levels - Part 2

Move the log messages to appropriate log levels so that
the console does not get a huge spam of unnecessary logs

Change-Id: Ie58b45078ad8b4732d7b84bfb77261396bd5862a
CRs-Fixed: 2043741
Varun Reddy Yeturu 8 år sedan
förälder
incheckning
dd51e8d67b

+ 5 - 7
core/dp/txrx/ol_txrx.c

@@ -3141,12 +3141,6 @@ int ol_txrx_peer_unref_delete(ol_txrx_peer_handle peer)
 	if (qdf_atomic_dec_and_test(&peer->ref_cnt)) {
 		u_int16_t peer_id;
 
-		QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_INFO,
-			   "Deleting peer %p (%pM) ref_cnt %d\n",
-			   peer,
-			   peer->mac_addr.raw,
-			   qdf_atomic_read(&peer->ref_cnt));
-
 		peer_id = peer->local_id;
 		/* remove the reference to the peer from the hash table */
 		ol_txrx_peer_find_hash_remove(pdev, peer);
@@ -3221,6 +3215,10 @@ int ol_txrx_peer_unref_delete(ol_txrx_peer_handle peer)
 			qdf_spin_unlock_bh(&pdev->peer_ref_mutex);
 		}
 
+		QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_DEBUG,
+			  "%s: Deleting peer %p (%pM) ref_cnt = %d",
+			  __func__, peer, peer->mac_addr.raw,
+			  qdf_atomic_read(&peer->ref_cnt));
 		ol_txrx_peer_tx_queue_free(pdev, peer);
 
 		/*
@@ -3246,7 +3244,7 @@ int ol_txrx_peer_unref_delete(ol_txrx_peer_handle peer)
 		qdf_mem_free(peer);
 	} else {
 		qdf_spin_unlock_bh(&pdev->peer_ref_mutex);
-		QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_INFO_HIGH,
+		QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_DEBUG,
 			  "%s: peer %p peer->ref_cnt = %d",
 			  __func__, peer, rc);
 	}

+ 1 - 1
core/dp/txrx/ol_txrx_peer_find.c

@@ -593,7 +593,7 @@ void ol_rx_peer_unmap_handler(ol_txrx_pdev_handle pdev, uint16_t peer_id)
 	 */
 	ol_txrx_peer_unref_delete(peer);
 
-	QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_INFO,
+	QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_DEBUG,
 	   "%s: Remove the ID %d reference to peer %p peer_id_ref_cnt %d",
 	   __func__, peer_id, peer, ref_cnt);
 }

+ 1 - 1
core/hdd/src/wlan_hdd_assoc.c

@@ -1937,7 +1937,7 @@ QDF_STATUS hdd_roam_register_sta(hdd_adapter_t *pAdapter,
 		staDesc.is_qos_enabled = 0;
 
 #ifdef FEATURE_WLAN_WAPI
-	hdd_notice("WAPI STA Registered: %d",
+	hdd_debug("WAPI STA Registered: %d",
 		   pAdapter->wapi_info.fIsWapiSta);
 	if (pAdapter->wapi_info.fIsWapiSta)
 		staDesc.is_wapi_supported = 1;

+ 1 - 1
core/hdd/src/wlan_hdd_p2p.c

@@ -1253,7 +1253,7 @@ __wlan_hdd_cfg80211_cancel_remain_on_channel(struct wiphy *wiphy,
 	pRemainChanCtx = cfgState->remain_on_chan_ctx;
 
 	if (pRemainChanCtx) {
-		hdd_notice("action_cookie = %08llx, roc cookie = %08llx, cookie = %08llx",
+		hdd_debug("action_cookie = %08llx, roc cookie = %08llx, cookie = %08llx",
 				cfgState->action_cookie, pRemainChanCtx->cookie,
 				cookie);
 

+ 5 - 5
core/hdd/src/wlan_hdd_tdls.c

@@ -239,7 +239,7 @@ void wlan_hdd_tdls_disable_offchan_and_teardown_links(hdd_context_t *hddctx)
 	connected_tdls_peers = wlan_hdd_tdls_connected_peers(adapter);
 
 	if (!connected_tdls_peers) {
-		hdd_notice("No TDLS connected peers to delete");
+		hdd_debug("No TDLS connected peers to delete");
 		return;
 	}
 
@@ -1696,7 +1696,7 @@ static void wlan_hdd_tdls_set_mode(hdd_context_t *pHddCtx,
 
 	if (pHddCtx->tdls_mode == tdls_mode) {
 		mutex_unlock(&pHddCtx->tdls_lock);
-		hdd_notice("already in mode %d", (int)tdls_mode);
+		hdd_debug("already in mode %d", (int)tdls_mode);
 
 		switch (tdls_mode) {
 		/* TDLS is already enabled hence clear source mask, return */
@@ -1705,13 +1705,13 @@ static void wlan_hdd_tdls_set_mode(hdd_context_t *pHddCtx,
 		case eTDLS_SUPPORT_EXTERNAL_CONTROL:
 			clear_bit((unsigned long)source,
 				  &pHddCtx->tdls_source_bitmap);
-			hdd_notice("clear source mask:%d", source);
+			hdd_debug("clear source mask:%d", source);
 			return;
 		/* TDLS is already disabled hence set source mask, return */
 		case eTDLS_SUPPORT_DISABLED:
 			set_bit((unsigned long)source,
 				&pHddCtx->tdls_source_bitmap);
-			hdd_notice("set source mask:%d", source);
+			hdd_debug("set source mask:%d", source);
 			return;
 		default:
 			return;
@@ -1736,7 +1736,7 @@ static void wlan_hdd_tdls_set_mode(hdd_context_t *pHddCtx,
 				 */
 				if (pHddCtx->tdls_source_bitmap) {
 					mutex_unlock(&pHddCtx->tdls_lock);
-					hdd_notice("Don't enable TDLS, source"
+					hdd_debug("Don't enable TDLS, source"
 						"bitmap: %lu",
 						pHddCtx->tdls_source_bitmap);
 					return;

+ 1 - 1
core/hdd/src/wlan_hdd_tx_rx.c

@@ -440,7 +440,7 @@ static void hdd_get_transmit_sta_id(hdd_adapter_t *adapter,
 	if (QDF_IS_STATUS_ERROR(status)) {
 		if (QDF_NBUF_CB_GET_IS_BCAST(skb) ||
 				QDF_NBUF_CB_GET_IS_MCAST(skb)) {
-			hdd_info("Received MC/BC packet for transmission");
+			hdd_debug("Received MC/BC packet for transmission");
 			mcbc_addr = true;
 		}
 	}

+ 10 - 10
core/hdd/src/wlan_hdd_wmm.c

@@ -1280,7 +1280,7 @@ QDF_STATUS hdd_wmm_init(hdd_adapter_t *pAdapter)
 	sme_QosWmmUpType *hddWmmDscpToUpMap = pAdapter->hddWmmDscpToUpMap;
 	uint8_t dscp;
 
-	hdd_info("Entered");
+	hdd_debug("Entered");
 
 	/* DSCP to User Priority Lookup Table
 	 * By default use the 3 Precedence bits of DSCP as the User Priority
@@ -1809,7 +1809,7 @@ QDF_STATUS hdd_wmm_assoc(hdd_adapter_t *pAdapter,
 	 * enable UAPSD for any access categories
 	 */
 
-	hdd_info("Entered");
+	hdd_debug("Entered");
 
 	if (pRoamInfo->fReassocReq) {
 		/* when we reassociate we should continue to use
@@ -1821,7 +1821,7 @@ QDF_STATUS hdd_wmm_assoc(hdd_adapter_t *pAdapter,
 		 * parameters will be updated there
 		 */
 
-		hdd_info("Reassoc so no work, Exiting");
+		hdd_debug("Reassoc so no work, Exiting");
 
 		return QDF_STATUS_SUCCESS;
 	}
@@ -1829,7 +1829,7 @@ QDF_STATUS hdd_wmm_assoc(hdd_adapter_t *pAdapter,
 	uapsdMask =
 		pRoamInfo->u.pConnectedProfile->modifyProfileFields.uapsd_mask;
 
-	hdd_info("U-APSD mask is 0x%02x", (int)uapsdMask);
+	hdd_debug("U-APSD mask is 0x%02x", (int)uapsdMask);
 
 	if (uapsdMask & HDD_AC_VO) {
 		status =
@@ -1903,7 +1903,7 @@ QDF_STATUS hdd_wmm_assoc(hdd_adapter_t *pAdapter,
 		hdd_wmm_init(pAdapter);
 	}
 
-	hdd_info("Exiting");
+	hdd_debug("Exiting");
 
 	return QDF_STATUS_SUCCESS;
 }
@@ -1933,7 +1933,7 @@ QDF_STATUS hdd_wmm_connect(hdd_adapter_t *pAdapter,
 	bool qosConnection;
 	uint8_t acmMask;
 
-	hdd_info("Entered");
+	hdd_debug("Entered");
 
 	if ((eCSR_BSS_TYPE_INFRASTRUCTURE == eBssType) &&
 	    pRoamInfo && pRoamInfo->u.pConnectedProfile) {
@@ -1946,7 +1946,7 @@ QDF_STATUS hdd_wmm_connect(hdd_adapter_t *pAdapter,
 		acmMask = 0x0;
 	}
 
-	hdd_info("qap is %d, qosConnection is %d, acmMask is 0x%x",
+	hdd_debug("qap is %d, qosConnection is %d, acmMask is 0x%x",
 		 qap, qosConnection, acmMask);
 
 	pAdapter->hddWmmStatus.wmmQap = qap;
@@ -1954,7 +1954,7 @@ QDF_STATUS hdd_wmm_connect(hdd_adapter_t *pAdapter,
 
 	for (ac = 0; ac < WLAN_MAX_AC; ac++) {
 		if (qap && qosConnection && (acmMask & acm_mask_bit[ac])) {
-			hdd_info("ac %d on", ac);
+			hdd_debug("ac %d on", ac);
 
 			/* admission is required */
 			pAdapter->hddWmmStatus.wmmAcStatus[ac].
@@ -1983,7 +1983,7 @@ QDF_STATUS hdd_wmm_connect(hdd_adapter_t *pAdapter,
 					wmmAcAccessAllowed = false;
 			}
 		} else {
-			hdd_info("ac %d off", ac);
+			hdd_debug("ac %d off", ac);
 			/* admission is not required so access is allowed */
 			pAdapter->hddWmmStatus.wmmAcStatus[ac].
 			wmmAcAccessRequired = false;
@@ -1993,7 +1993,7 @@ QDF_STATUS hdd_wmm_connect(hdd_adapter_t *pAdapter,
 
 	}
 
-	hdd_info("Exiting");
+	hdd_debug("Exiting");
 
 	return QDF_STATUS_SUCCESS;
 }

+ 1 - 1
core/mac/src/pe/lim/lim_process_message_queue.c

@@ -1924,7 +1924,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 	default:
 		qdf_mem_free((void *)msg->bodyptr);
 		msg->bodyptr = NULL;
-		pe_err("Discarding unexpected message received %X",
+		pe_debug("Discarding unexpected message received %X",
 			msg->type);
 		lim_print_msg_name(mac_ctx, LOGE, msg->type);
 		break;

+ 3 - 3
core/sap/dfs/src/dfs_init.c

@@ -245,7 +245,7 @@ int dfs_init_radar_filters(struct ieee80211com *ic,
 		return DFS_STATUS_SUCCESS;
 	}
 
-	QDF_TRACE(QDF_MODULE_ID_SAP, QDF_TRACE_LEVEL_INFO,
+	QDF_TRACE(QDF_MODULE_ID_SAP, QDF_TRACE_LEVEL_DEBUG,
 		  "%s[%d]:dfsdomain=%d, numradars=%d, numb5radars=%d",
 		  __func__, __LINE__, radar_info->dfsdomain,
 		  radar_info->numradars, radar_info->numb5radars);
@@ -471,10 +471,10 @@ int dfs_init_radar_filters(struct ieee80211com *ic,
 		dfs_round((int32_t) ((max_pulsedur * 100 / 80) * 100));
 	/* relax the max pulse duration a little bit due to inaccuracy caused by chirping. */
 	dfs->dfs_rinfo.rn_maxpulsedur = dfs->dfs_rinfo.rn_maxpulsedur + 20;
-	QDF_TRACE(QDF_MODULE_ID_SAP, QDF_TRACE_LEVEL_INFO,
+	QDF_TRACE(QDF_MODULE_ID_SAP, QDF_TRACE_LEVEL_DEBUG,
 		  "%s[%d]: DFS min filter rssiThresh = %d",
 		  __func__, __LINE__, min_rssithresh);
-	QDF_TRACE(QDF_MODULE_ID_SAP, QDF_TRACE_LEVEL_INFO,
+	QDF_TRACE(QDF_MODULE_ID_SAP, QDF_TRACE_LEVEL_DEBUG,
 		  "%s[%d]:DFS max pulse dur = %d ticks",
 		  __func__, __LINE__, dfs->dfs_rinfo.rn_maxpulsedur);
 	return DFS_STATUS_SUCCESS;