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
This commit is contained in:
Manikanta Pubbisetty
2021-03-19 16:10:42 +05:30
committed by snandini
parent 3d7e6fb7c1
commit 86e32fb122

View File

@@ -2427,7 +2427,7 @@ more_data:
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);