qcacmn: Add check for duplicate entry

Add changes to check if current descriptor which
is being process is not already reaped. In case
if the descriptor is already process and yet to
be replenished drop the descriptor and continue.

Change-Id: I2c46b7f4e50a27e2566f0ce2d9708cecb26c38e4
CRs-Fixed: 3177257

dp/wifi3.0/dp_rx_err.c

@@ -1491,6 +1491,7 @@ dp_rx_reo_err_entry_process(struct dp_soc *soc,
 	uint8_t rx_desc_pool_id;
 	struct dp_txrx_peer *txrx_peer = NULL;
 	dp_txrx_ref_handle txrx_ref_handle = NULL;
+	hal_ring_handle_t hal_ring_hdl = soc->reo_exception_ring.hal_srng;
 
 	peer_id = dp_rx_peer_metadata_peer_id_get(soc,
 					mpdu_desc_info->peer_meta_data);
@@ -1506,12 +1507,26 @@ more_msdu_link_desc:
 						msdu_list.sw_cookie[i]);
 
 		qdf_assert_always(rx_desc);
+		nbuf = rx_desc->nbuf;
 
-		rx_desc_pool_id = rx_desc->pool_id;
-		/* all buffers from a MSDU link belong to same pdev */
-		pdev = dp_get_pdev_for_lmac_id(soc, rx_desc_pool_id);
+		/*
+		 * this is a unlikely scenario where the host is reaping
+		 * a descriptor which it already reaped just a while ago
+		 * but is yet to replenish it back to HW.
+		 * In this case host will dump the last 128 descriptors
+		 * including the software descriptor rx_desc and assert.
+		 */
+		if (qdf_unlikely(!rx_desc->in_use) ||
+		    qdf_unlikely(!nbuf)) {
+			DP_STATS_INC(soc, rx.err.hal_reo_dest_dup, 1);
+			dp_info_rl("Reaping rx_desc not in use!");
+			dp_rx_dump_info_and_assert(soc, hal_ring_hdl,
+						   ring_desc, rx_desc);
+			/* ignore duplicate RX desc and continue to process */
+			/* Pop out the descriptor */
+			continue;
+		}
 
-		nbuf = rx_desc->nbuf;
 		ret = dp_rx_desc_paddr_sanity_check(rx_desc,
 						    msdu_list.paddr[i]);
 		if (!ret) {
@@ -1520,6 +1535,10 @@ more_msdu_link_desc:
 			continue;
 		}
 
+		rx_desc_pool_id = rx_desc->pool_id;
+		/* all buffers from a MSDU link belong to same pdev */
+		pdev = dp_get_pdev_for_lmac_id(soc, rx_desc_pool_id);
+
 		rx_desc_pool = &soc->rx_desc_buf[rx_desc_pool_id];
 		dp_ipa_rx_buf_smmu_mapping_lock(soc);
 		dp_rx_nbuf_unmap_pool(soc, rx_desc_pool, nbuf);