> -----Original Message-----
> From: Lu, Wenzhuo <[email protected]>
> Sent: Monday, September 4, 2023 10:18 AM
> To: Richardson, Bruce <[email protected]>; [email protected]
> Cc: Wu, Jingjing <[email protected]>; [email protected]
> Subject: RE: [PATCH v2 2/4] net/iavf: fix buffer leak on Tx queue stop
>
>
>
> > -----Original Message-----
> > From: Richardson, Bruce <[email protected]>
> > Sent: Thursday, August 31, 2023 8:34 PM
> > To: [email protected]
> > Cc: Richardson, Bruce <[email protected]>; Lu, Wenzhuo
> > <[email protected]>; Wu, Jingjing <[email protected]>;
> > [email protected]
> > Subject: [PATCH v2 2/4] net/iavf: fix buffer leak on Tx queue stop
> >
> > When a queue is stopped all buffers are meant to released from it, and
> > then a new set of buffers reallocated on start. For the iavf code when
> > using vector Tx, some buffers were left in the ring, and so those
> > buffers were leaked. The buffers were missed as the release code only
> > handled one side of a wrap-around case in the ring.
> >
> > Fix the issue by doing a single iteration of the ring freeing all
> > buffers in it, handling wraparound through a simple post-increment check.
> >
> > Fixes: 319c421f3890 ("net/avf: enable SSE Rx Tx")
> > Fixes: 9ab9514c150e ("net/iavf: enable AVX512 for Tx")
> > Cc: [email protected]
> > Cc: [email protected]
> > Cc: [email protected]
> >
> > Signed-off-by: Bruce Richardson <[email protected]>
> Acked-by: Wenzhuo Lu <[email protected]>
Applied to dpdk-next-net-intel.
Thanks
Qi
