dpaa2-eth: recycle the RX buffer only after all processing done
authorIoana Ciornei <ioana.ciornei@nxp.com>
Fri, 24 Nov 2023 10:28:05 +0000 (12:28 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 26 Nov 2023 15:18:57 +0000 (15:18 +0000)
commitbeb1930f966d1517921488bd5d64147f58f79abf
tree61e1673f639ca617bd6cc19ad6a3e7e6cd594047
parentf422abe3f23d483cf01f386819f26fb3fe0dbb2b
dpaa2-eth: recycle the RX buffer only after all processing done

The blamed commit added support for Rx copybreak. This meant that for
certain frame sizes, a new skb was allocated and the initial data buffer
was recycled. Instead of waiting to recycle the Rx buffer only after all
processing was done on it (like accessing the parse results or timestamp
information), the code path just went ahead and re-used the buffer right
away.

This sometimes lead to corrupted HW and SW annotation areas.
Fix this by delaying the moment when the buffer is recycled.

Fixes: 50f826999a80 ("dpaa2-eth: add rx copybreak support")
Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/dpaa2/dpaa2-eth.c