18 Feb
2025
18 Feb
'25
10:13 a.m.
-----Original Message----- From: Wei Fang wei.fang@nxp.com Sent: Tuesday, February 18, 2025 11:18 AM
[...]
enetc_free_tx_frame() call on dma_err path is still needed for count 0 because it needs to free the skb.
First, the tx_swbd is not set when count is 0, so tx_swbd->skb is NULL when the error occurs.
Second, the skb is freed in enetc_start_xmit() not enetc_free_tx_frame().
Yeah, noticed too finally. 'do {} while()' is not ok on error path here.