Commit 9d6e778d authored Mar 19, 2021 by Manikanta Pubbisetty Committed by snandini Mar 25, 2021

qcacmn: assert on processing invalid RX ring descriptors

Currently, we break the dp_rx_process() reap loop when we encounter
a ring descriptor with invalid bit set in SW cookie. If the HW
has copied old/stale entries in the ring, then we will end up
processing the same entry every time we try to reap the ring.
Instead, assert on processing such invalid entries so that
we can obtain useful debug information.

CRs-Fixed: 2904509
Change-Id: Iddde959a573ddf4ca364e1fb7a54ed0eaf4dc169

parent 3455eeae

dp/wifi3.0/dp_rx.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -2366,7 +2366,7 @@ uint32_t dp_rx_process(struct dp_intr *int_ctx, hal_ring_handle_t hal_ring_hdl,
		status = dp_rx_cookie_check_and_invalidate(ring_desc);
		if (qdf_unlikely(QDF_IS_STATUS_ERROR(status))) {
		DP_STATS_INC(soc, rx.err.stale_cookie, 1);
		break;
		qdf_assert_always(0);
		}

		rx_desc = dp_rx_cookie_2_va_rxdma_buf(soc, rx_buf_cookie);