qcacmn: Object Manager debug print changes

1) Rate limit object manager logical delete state prints

   On object search/requesting reference in tasklet leads to many prints
   for logically deleted objects causing timing related issues.

   As these prints needed for debug, This change would rate limit the prints

2) Print all objects ref counts

3) Fixed style issues got introduced in other gerrits

Change-Id: I929976875f8da3ce2114d540735515d792836762
CRs-Fixed: 2068364

umac/cmn_services/obj_mgr/inc/wlan_objmgr_cmn.h

@@ -285,4 +285,5 @@ static inline char *string_from_dbgid(wlan_objmgr_ref_dbgid id)
 #else
 #define WLAN_OBJMGR_BUG(val)
 #endif
+#define WLAN_OBJMGR_RATELIMIT_THRESH 2
 #endif /* _WLAN_OBJMGR_CMN_H_*/

umac/cmn_services/obj_mgr/inc/wlan_objmgr_pdev_obj.h

@@ -147,6 +147,7 @@ struct wlan_objmgr_pdev_mlme {
  * @wlan_psoc:         back pointer to PSOC, its attached to
  * @ref_cnt:           Ref count
  * @ref_id_dbg:        Array to track Ref count
+ * @print_cnt:         Count to throttle Logical delete prints
  */
 struct wlan_objmgr_pdev_objmgr {
 	uint8_t wlan_pdev_id;
@@ -156,6 +157,7 @@ struct wlan_objmgr_pdev_objmgr {
 	struct wlan_objmgr_psoc *wlan_psoc;
 	qdf_atomic_t ref_cnt;
 	qdf_atomic_t ref_id_dbg[WLAN_REF_ID_MAX];
+	uint8_t print_cnt;
 };
 
 /**

umac/cmn_services/obj_mgr/inc/wlan_objmgr_peer_obj.h

@@ -141,11 +141,13 @@ struct wlan_objmgr_peer_mlme {
  * @vdev:              VDEV pointer to which it is associated
  * @ref_cnt:           Ref count
  * @ref_id_dbg:        Array to track Ref count
+ * @print_cnt:         Count to throttle Logical delete prints
  */
 struct wlan_objmgr_peer_objmgr {
 	struct wlan_objmgr_vdev *vdev;
 	qdf_atomic_t ref_cnt;
 	qdf_atomic_t ref_id_dbg[WLAN_REF_ID_MAX];
+	uint8_t print_cnt;
 };
 
 /**

umac/cmn_services/obj_mgr/inc/wlan_objmgr_psoc_obj.h

@@ -222,6 +222,7 @@ struct wlan_objmgr_psoc_nif {
  * @peer_list:            Peer list
  * @ref_cnt:              Ref count
  * @ref_id_dbg:           Array to track Ref count
+ * @print_cnt:            Count to throttle Logical delete prints
  * @qdf_dev:              QDF Device
  */
 struct wlan_objmgr_psoc_objmgr {
@@ -236,6 +237,7 @@ struct wlan_objmgr_psoc_objmgr {
 	struct wlan_peer_list peer_list;
 	qdf_atomic_t ref_cnt;
 	qdf_atomic_t ref_id_dbg[WLAN_REF_ID_MAX];
+	uint8_t print_cnt;
 	qdf_device_t qdf_dev;
 };
 

umac/cmn_services/obj_mgr/inc/wlan_objmgr_vdev_obj.h

@@ -324,6 +324,7 @@ struct wlan_objmgr_vdev_nif {
  *  @c_flags:           creation specific flags
  *  @ref_cnt:           Ref count
  *  @ref_id_dbg:        Array to track Ref count
+ *  @print_cnt:         Count to throttle Logical delete prints
  */
 struct wlan_objmgr_vdev_objmgr {
 	uint8_t vdev_id;
@@ -336,6 +337,7 @@ struct wlan_objmgr_vdev_objmgr {
 	uint32_t c_flags;
 	qdf_atomic_t ref_cnt;
 	qdf_atomic_t ref_id_dbg[WLAN_REF_ID_MAX];
+	uint8_t print_cnt;
 };
 
 /**

umac/cmn_services/obj_mgr/src/wlan_objmgr_global_obj.c

@@ -784,7 +784,9 @@ void wlan_objmgr_print_ref_ids(qdf_atomic_t *id)
 	for (i = 0; i < WLAN_REF_ID_MAX; i++) {
 		pending_ref = qdf_atomic_read(&id[i]);
 		if (pending_ref)
-		obj_mgr_info("%s -- %d", string_from_dbgid(i), pending_ref);
+			obj_mgr_info("%s -- %d",
+				string_from_dbgid(i), pending_ref);
 	}
+
 	return;
 }

umac/cmn_services/obj_mgr/src/wlan_objmgr_pdev_obj.c

@@ -70,9 +70,9 @@ static QDF_STATUS wlan_objmgr_pdev_obj_free(struct wlan_objmgr_pdev *pdev)
 	uint8_t pdev_id;
 
 	if (pdev == NULL) {
-	obj_mgr_err("pdev obj is NULL");
-	QDF_ASSERT(0);
-	return QDF_STATUS_E_FAILURE;
+		obj_mgr_err("pdev obj is NULL");
+		QDF_ASSERT(0);
+		return QDF_STATUS_E_FAILURE;
 	}
 
 	pdev_id = wlan_objmgr_pdev_get_pdev_id(pdev);
@@ -129,6 +129,7 @@ struct wlan_objmgr_pdev *wlan_objmgr_pdev_obj_create(
 	/* Save HDD/OSIF pointer */
 	pdev->pdev_nif.pdev_ospriv = osdev_priv;
 	qdf_atomic_init(&pdev->pdev_objmgr.ref_cnt);
+	pdev->pdev_objmgr.print_cnt = 0;
 	wlan_objmgr_pdev_get_ref(pdev, WLAN_OBJMGR_ID);
 	/* Invoke registered create handlers */
 	for (id = 0; id < WLAN_UMAC_MAX_COMPONENTS; id++) {
@@ -185,7 +186,8 @@ static QDF_STATUS wlan_objmgr_pdev_obj_destroy(struct wlan_objmgr_pdev *pdev)
 	pdev_id = wlan_objmgr_pdev_get_pdev_id(pdev);
 
 	if (pdev->obj_state != WLAN_OBJ_STATE_LOGICALLY_DELETED) {
-		obj_mgr_err("pdev object delete is not invoked: pdev-id:%d", pdev_id);
+		obj_mgr_err("pdev object delete is not invoked: pdev-id:%d",
+			pdev_id);
 		WLAN_OBJMGR_BUG(0);
 	}
 
@@ -822,8 +824,11 @@ QDF_STATUS wlan_objmgr_pdev_try_get_ref(struct wlan_objmgr_pdev *pdev,
 	pdev_id = wlan_objmgr_pdev_get_pdev_id(pdev);
 	if (pdev->obj_state == WLAN_OBJ_STATE_LOGICALLY_DELETED) {
 		wlan_pdev_obj_unlock(pdev);
-		obj_mgr_err("pdev obj is in Deletion Progress state: pdev-id: %d",
-			 pdev_id) ;
+		if (pdev->pdev_objmgr.print_cnt++ <=
+				WLAN_OBJMGR_RATELIMIT_THRESH)
+			obj_mgr_err("[Ref id: %d] pdev [%d] is in L-Del state",
+					id, pdev_id);
+
 		return QDF_STATUS_E_RESOURCES;
 	}
 
@@ -848,7 +853,8 @@ void wlan_objmgr_pdev_release_ref(struct wlan_objmgr_pdev *pdev,
 	pdev_id = wlan_objmgr_pdev_get_pdev_id(pdev);
 
 	if (!qdf_atomic_read(&pdev->pdev_objmgr.ref_id_dbg[id])) {
-		obj_mgr_err("pdev (id:%d)ref cnt was not taken by %d", pdev_id, id);
+		obj_mgr_err("pdev (id:%d)ref cnt was not taken by %d",
+				pdev_id, id);
 		wlan_objmgr_print_ref_ids(pdev->pdev_objmgr.ref_id_dbg);
 		WLAN_OBJMGR_BUG(0);
 	}

umac/cmn_services/obj_mgr/src/wlan_objmgr_peer_obj.c

@@ -81,7 +81,8 @@ static QDF_STATUS wlan_objmgr_peer_obj_free(struct wlan_objmgr_peer *peer)
 
 	vdev = wlan_peer_get_vdev(peer);
 	if (vdev == NULL) {
-		obj_mgr_err("VDEV is NULL for peer(%02x:%02x:%02x:%02x:%02x:%02x)",
+		obj_mgr_err(
+			"VDEV is NULL for peer(%02x:%02x:%02x:%02x:%02x:%02x)",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		return QDF_STATUS_E_FAILURE;
@@ -92,7 +93,8 @@ static QDF_STATUS wlan_objmgr_peer_obj_free(struct wlan_objmgr_peer *peer)
 	/* get PSOC from VDEV, if it is NULL, return */
 	psoc = wlan_vdev_get_psoc(vdev);
 	if (psoc == NULL) {
-		obj_mgr_err("PSOC is NULL for peer(%02x:%02x:%02x:%02x:%02x:%02x)",
+		obj_mgr_err(
+			"PSOC is NULL for peer(%02x:%02x:%02x:%02x:%02x:%02x)",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		return QDF_STATUS_E_FAILURE;
@@ -106,15 +108,16 @@ static QDF_STATUS wlan_objmgr_peer_obj_free(struct wlan_objmgr_peer *peer)
 
 	/* Detach peer from VDEV's peer list */
 	if (wlan_objmgr_vdev_peer_detach(vdev, peer) == QDF_STATUS_E_FAILURE) {
-		obj_mgr_err("Peer(%02x:%02x:%02x:%02x:%02x:%02x) VDEV detach failure \
-			for vdev-id: %d",
+		obj_mgr_err(
+		"Peer(%02x:%02x:%02x:%02x:%02x:%02x) VDEV detach fail, vdev id: %d",
 			macaddr[0], macaddr[1], macaddr[2],
 			macaddr[3], macaddr[4], macaddr[5], vdev_id);
 		return QDF_STATUS_E_FAILURE;
 	}
 	/* Detach peer from PSOC's peer list */
 	if (wlan_objmgr_psoc_peer_detach(psoc, peer) == QDF_STATUS_E_FAILURE) {
-		obj_mgr_err("Peer(%02x:%02x:%02x:%02x:%02x:%02x) PSOC detach failure",
+		obj_mgr_err(
+		"Peer(%02x:%02x:%02x:%02x:%02x:%02x) PSOC detach failure",
 			macaddr[0], macaddr[1], macaddr[2],
 			macaddr[3], macaddr[4], macaddr[5]);
 		return QDF_STATUS_E_FAILURE;
@@ -140,7 +143,8 @@ struct wlan_objmgr_peer *wlan_objmgr_peer_obj_create(
 	uint8_t id;
 
 	if (vdev == NULL) {
-		obj_mgr_err("VDEV is NULL for peer (%02x:%02x:%02x:%02x:%02x:%02x)",
+		obj_mgr_err(
+			"VDEV is NULL for peer (%02x:%02x:%02x:%02x:%02x:%02x)",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		return NULL;
@@ -148,7 +152,8 @@ struct wlan_objmgr_peer *wlan_objmgr_peer_obj_create(
 	/* Get psoc, if psoc is NULL, return */
 	psoc = wlan_vdev_get_psoc(vdev);
 	if (psoc == NULL) {
-		obj_mgr_err("PSOC is NULL for peer (%02x:%02x:%02x:%02x:%02x:%02x)",
+		obj_mgr_err(
+			"PSOC is NULL for peer (%02x:%02x:%02x:%02x:%02x:%02x)",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		return NULL;
@@ -156,7 +161,8 @@ struct wlan_objmgr_peer *wlan_objmgr_peer_obj_create(
 	/* Allocate memory for peer object */
 	peer = qdf_mem_malloc(sizeof(*peer));
 	if (peer == NULL) {
-		obj_mgr_err("Peer(%02x:%02x:%02x:%02x:%02x:%02x) allocation failure",
+		obj_mgr_err(
+		"Peer(%02x:%02x:%02x:%02x:%02x:%02x) allocation failure",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		return NULL;
@@ -171,11 +177,12 @@ struct wlan_objmgr_peer *wlan_objmgr_peer_obj_create(
 	wlan_peer_mlme_set_state(peer, WLAN_INIT_STATE);
 	wlan_peer_mlme_reset_seq_num(peer);
 	qdf_atomic_init(&peer->peer_objmgr.ref_cnt);
+	peer->peer_objmgr.print_cnt = 0;
 	/* Attach peer to psoc, psoc maintains the node table for the device */
 	if (wlan_objmgr_psoc_peer_attach(psoc, peer) !=
 					QDF_STATUS_SUCCESS) {
-		obj_mgr_err("Peer(%02x:%02x:%02x:%02x:%02x:%02x) \
-				PSOC attach failure",
+		obj_mgr_err(
+		"Peer(%02x:%02x:%02x:%02x:%02x:%02x) PSOC attach failure",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		qdf_mem_free(peer);
@@ -184,8 +191,8 @@ struct wlan_objmgr_peer *wlan_objmgr_peer_obj_create(
 	/* Attach peer to vdev peer table */
 	if (wlan_objmgr_vdev_peer_attach(vdev, peer) !=
 					QDF_STATUS_SUCCESS) {
-		obj_mgr_err("Peer(%02x:%02x:%02x:%02x:%02x:%02x) \
-				VDEV attach failure",
+		obj_mgr_err(
+		"Peer(%02x:%02x:%02x:%02x:%02x:%02x) VDEV attach failure",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		/* if attach fails, detach from psoc table before free */
@@ -228,8 +235,8 @@ struct wlan_objmgr_peer *wlan_objmgr_peer_obj_create(
 		peer->obj_state = WLAN_OBJ_STATE_PARTIALLY_CREATED;
 	} else if (obj_status == QDF_STATUS_E_FAILURE) {
 		/* Clean up the peer */
-		obj_mgr_err("Peer(%02x:%02x:%02x:%02x:%02x:%02x) \
-				component object alloc failure",
+		obj_mgr_err(
+		"Peer(%02x:%02x:%02x:%02x:%02x:%02x) comp object alloc fail",
 				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
 		wlan_objmgr_peer_obj_delete(peer);
@@ -254,8 +261,8 @@ static QDF_STATUS wlan_objmgr_peer_obj_destroy(struct wlan_objmgr_peer *peer)
 	macaddr = wlan_peer_get_macaddr(peer);
 
 	if (peer->obj_state != WLAN_OBJ_STATE_LOGICALLY_DELETED) {
-		obj_mgr_err("peer(%02x:%02x:%02x:%02x:%02x:%02x) \
-			object delete is not invoked",
+		obj_mgr_err(
+		"peer(%02x:%02x:%02x:%02x:%02x:%02x) object del is not invoked",
 			macaddr[0], macaddr[1], macaddr[2],
 			macaddr[3], macaddr[4], macaddr[5]);
 		WLAN_OBJMGR_BUG(0);
@@ -305,8 +312,8 @@ QDF_STATUS wlan_objmgr_peer_obj_delete(struct wlan_objmgr_peer *peer)
 	print_idx = qdf_get_pidx();
 	if (qdf_print_is_verbose_enabled(print_idx, QDF_MODULE_ID_OBJ_MGR,
 		QDF_TRACE_LEVEL_DEBUG)) {
-		obj_mgr_debug("Logically deleting the peer"
-					"(%02x:%02x:%02x:%02x:%02x:%02x)",
+		obj_mgr_debug(
+		"Logically deleting the peer (%02x:%02x:%02x:%02x:%02x:%02x)",
 					macaddr[0], macaddr[1], macaddr[2],
 					macaddr[3], macaddr[4], macaddr[5]);
 		wlan_objmgr_print_ref_ids(peer->peer_objmgr.ref_id_dbg);
@@ -585,10 +592,12 @@ QDF_STATUS wlan_objmgr_peer_try_get_ref(struct wlan_objmgr_peer *peer,
 	macaddr = wlan_peer_get_macaddr(peer);
 	if (peer->obj_state == WLAN_OBJ_STATE_LOGICALLY_DELETED) {
 		wlan_peer_obj_unlock(peer);
-		obj_mgr_err("peer(%02x:%02x:%02x:%02x:%02x:%02x) obj \
-			is in Deletion Progress state",
-			macaddr[0], macaddr[1], macaddr[2],
+		if (peer->peer_objmgr.print_cnt++ <=
+				WLAN_OBJMGR_RATELIMIT_THRESH)
+			obj_mgr_err("peer(%02x:%02x:%02x:%02x:%02x:%02x) L-Del",
+				macaddr[0], macaddr[1], macaddr[2],
 				macaddr[3], macaddr[4], macaddr[5]);
+
 		return QDF_STATUS_E_RESOURCES;
 	}
 
@@ -614,8 +623,8 @@ void wlan_objmgr_peer_release_ref(struct wlan_objmgr_peer *peer,
 	macaddr = wlan_peer_get_macaddr(peer);
 
 	if (!qdf_atomic_read(&peer->peer_objmgr.ref_id_dbg[id])) {
-		obj_mgr_err("peer(%02x:%02x:%02x:%02x:%02x:%02x) \
-			ref cnt was not taken by %d",
+		obj_mgr_err(
+		"peer(%02x:%02x:%02x:%02x:%02x:%02x) ref was not taken by %d",
 			macaddr[0], macaddr[1], macaddr[2],
 			macaddr[3], macaddr[4], macaddr[5], id);
 		wlan_objmgr_print_ref_ids(peer->peer_objmgr.ref_id_dbg);

umac/cmn_services/obj_mgr/src/wlan_objmgr_psoc_obj.c

@@ -131,6 +131,7 @@ struct wlan_objmgr_psoc *wlan_objmgr_psoc_obj_create(uint32_t phy_version,
 	objmgr->max_vdev_count = WLAN_UMAC_PSOC_MAX_VDEVS;
 	objmgr->wlan_peer_count = 0;
 	qdf_atomic_init(&objmgr->ref_cnt);
+	objmgr->print_cnt = 0;
 	/* set phy version, dev_type in psoc */
 	wlan_psoc_set_nif_phy_version(psoc, phy_version);
 	wlan_psoc_set_dev_type(psoc, dev_type);
@@ -1166,7 +1167,7 @@ static struct wlan_objmgr_peer *wlan_obj_psoc_peerlist_get_peer_no_state(
 }
 
 /**
- * wlan_obj_psoc_populate_logically_deleted_peerlist_by_mac_n_bssid() - get peer
+ * wlan_obj_psoc_populate_logically_del_peerlist_by_mac_n_bssid() - get peer
  *                                           from psoc peer list using
  *                                           mac and vdev self mac
  * @obj_list: peer object list
@@ -1184,7 +1185,7 @@ static struct wlan_objmgr_peer *wlan_obj_psoc_peerlist_get_peer_no_state(
  *         NULL on FAILURE
  */
 static qdf_list_t
-	*wlan_obj_psoc_populate_logically_deleted_peerlist_by_mac_n_bssid(
+	*wlan_obj_psoc_populate_logically_del_peerlist_by_mac_n_bssid(
 				qdf_list_t *obj_list, uint8_t *macaddr,
 				uint8_t *bssid,
 				wlan_objmgr_ref_dbgid dbg_id)
@@ -1220,13 +1221,22 @@ static qdf_list_t
 				/* Return peer in logically deleted state */
 				if ((peer->obj_state ==
 					WLAN_OBJ_STATE_LOGICALLY_DELETED) &&
-					qdf_atomic_read(&peer->peer_objmgr.ref_cnt)) {
+				     qdf_atomic_read(
+						&peer->peer_objmgr.ref_cnt)) {
+
 					wlan_objmgr_peer_get_ref(peer, dbg_id);
 					wlan_peer_obj_unlock(peer);
 					lock_released = true;
-					peer_list = qdf_mem_malloc(sizeof(struct wlan_logically_del_peer));
+
+					peer_list =
+					qdf_mem_malloc(
+					sizeof(struct wlan_logically_del_peer));
+
 					peer_list->peer = peer;
-					qdf_list_insert_front(logical_del_peer_list, &peer_list->list);
+
+					qdf_list_insert_front(
+						logical_del_peer_list,
+							&peer_list->list);
 				}
 			}
 		}
@@ -1592,9 +1602,13 @@ qdf_list_t *wlan_objmgr_populate_logically_deleted_peerlist_by_mac_n_vdev(
 	hash_index = WLAN_PEER_HASH(macaddr);
 	peer_list = &objmgr->peer_list;
 	qdf_spin_lock_bh(&peer_list->peer_list_lock);
+
 	/* Iterate through peer list, get peer */
-	logical_del_peer_list = wlan_obj_psoc_populate_logically_deleted_peerlist_by_mac_n_bssid(
-		&peer_list->peer_hash[hash_index], macaddr, bssid, dbg_id);
+	logical_del_peer_list =
+		wlan_obj_psoc_populate_logically_del_peerlist_by_mac_n_bssid(
+			&peer_list->peer_hash[hash_index], macaddr,
+			bssid, dbg_id);
+
 	qdf_spin_unlock_bh(&peer_list->peer_list_lock);
 	wlan_psoc_obj_unlock(psoc);
 
@@ -1682,7 +1696,10 @@ QDF_STATUS wlan_objmgr_psoc_try_get_ref(struct wlan_objmgr_psoc *psoc,
 	wlan_psoc_obj_lock(psoc);
 	if (psoc->obj_state == WLAN_OBJ_STATE_LOGICALLY_DELETED) {
 		wlan_psoc_obj_unlock(psoc);
-		obj_mgr_err("Called by %d, psoc obj is in Deletion Progress state", id);
+		if (psoc->soc_objmgr.print_cnt++ <=
+				WLAN_OBJMGR_RATELIMIT_THRESH)
+			obj_mgr_err("[Ref id: %d] psoc is in L-Del state", id);
+
 		return QDF_STATUS_E_RESOURCES;
 	}
 
@@ -1772,15 +1789,15 @@ QDF_STATUS wlan_objmgr_print_ref_all_objects_per_psoc(
 		struct wlan_objmgr_psoc *psoc)
 {
 	obj_mgr_info("Ref counts of PEER");
-	wlan_objmgr_iterate_obj_list(psoc, WLAN_PEER_OP,
+	wlan_objmgr_iterate_obj_list_all(psoc, WLAN_PEER_OP,
 				wlan_objmgr_psoc_peer_ref_print, NULL, 1,
 				WLAN_OBJMGR_ID);
 	obj_mgr_info("Ref counts of VDEV");
-	wlan_objmgr_iterate_obj_list(psoc, WLAN_VDEV_OP,
+	wlan_objmgr_iterate_obj_list_all(psoc, WLAN_VDEV_OP,
 				wlan_objmgr_psoc_vdev_ref_print, NULL, 1,
 				WLAN_OBJMGR_ID);
 	obj_mgr_info("Ref counts of PDEV");
-	wlan_objmgr_iterate_obj_list(psoc, WLAN_PDEV_OP,
+	wlan_objmgr_iterate_obj_list_all(psoc, WLAN_PDEV_OP,
 				wlan_objmgr_psoc_pdev_ref_print, NULL, 1,
 				WLAN_OBJMGR_ID);
 

umac/cmn_services/obj_mgr/src/wlan_objmgr_vdev_obj.c

@@ -168,6 +168,7 @@ struct wlan_objmgr_vdev *wlan_objmgr_vdev_obj_create(
 	/* peer count to 0 */
 	vdev->vdev_objmgr.wlan_peer_count = 0;
 	qdf_atomic_init(&vdev->vdev_objmgr.ref_cnt);
+	vdev->vdev_objmgr.print_cnt = 0;
 	wlan_objmgr_vdev_get_ref(vdev, WLAN_OBJMGR_ID);
 	/* Initialize max peer count based on opmode type */
 	if (wlan_vdev_mlme_get_opmode(vdev) == QDF_STA_MODE)
@@ -215,7 +216,7 @@ struct wlan_objmgr_vdev *wlan_objmgr_vdev_obj_create(
 	} else if (obj_status == QDF_STATUS_E_FAILURE) {
 		/* Clean up the psoc */
 		wlan_objmgr_vdev_obj_delete(vdev);
-		obj_mgr_err("VDEV component objects creation failed for vdev-id:%d",
+		obj_mgr_err("VDEV comp objects creation failed for vdev-id:%d",
 			vdev->vdev_objmgr.vdev_id);
 		return NULL;
 	}
@@ -257,7 +258,8 @@ static QDF_STATUS wlan_objmgr_vdev_obj_destroy(struct wlan_objmgr_vdev *vdev)
 	obj_status = wlan_objmgr_vdev_object_status(vdev);
 
 	if (obj_status == QDF_STATUS_E_FAILURE) {
-		obj_mgr_err("VDEV object deletion failed: vdev-id: %d", vdev_id);
+		obj_mgr_err("VDEV object deletion failed: vdev-id: %d",
+				vdev_id);
 		/* Ideally should not happen */
 		/* This leads to memleak ??? how to handle */
 		QDF_BUG(0);
@@ -735,8 +737,11 @@ QDF_STATUS wlan_objmgr_vdev_try_get_ref(struct wlan_objmgr_vdev *vdev,
 	vdev_id = wlan_vdev_get_id(vdev);
 	if (vdev->obj_state == WLAN_OBJ_STATE_LOGICALLY_DELETED) {
 		wlan_vdev_obj_unlock(vdev);
-		obj_mgr_err("called by %d, vdev obj is in Deletion Progress state: \
-				vdev-id:%d", id, vdev_id);
+		if (vdev->vdev_objmgr.print_cnt++ <=
+				WLAN_OBJMGR_RATELIMIT_THRESH)
+			obj_mgr_err("[Ref id: %d] vdev(%d) is in Log Del",
+				id, vdev_id);
+
 		return QDF_STATUS_E_RESOURCES;
 	}
 
@@ -762,7 +767,8 @@ void wlan_objmgr_vdev_release_ref(struct wlan_objmgr_vdev *vdev,
 	vdev_id = wlan_vdev_get_id(vdev);
 
 	if (!qdf_atomic_read(&vdev->vdev_objmgr.ref_id_dbg[id])) {
-		obj_mgr_err("vdev (id:%d)ref cnt was not taken by %d", vdev_id, id);
+		obj_mgr_err("vdev (id:%d)ref cnt was not taken by %d",
+				vdev_id, id);
 		wlan_objmgr_print_ref_ids(vdev->vdev_objmgr.ref_id_dbg);
 		WLAN_OBJMGR_BUG(0);
 	}