RE: [PATCH 15/15] net/dpaa: fix buffer free in slow path

2022-10-06 Thread Gagandeep Singh
> -Original Message- > From: Ferruh Yigit > Sent: Thursday, October 6, 2022 3:12 PM > To: Gagandeep Singh ; dev@dpdk.org > Cc: sta...@dpdk.org > Subject: Re: [PATCH 15/15] net/dpaa: fix buffer free in slow path > > On 10/6/2022 9:51 AM, Gagan

Re: [PATCH 15/15] net/dpaa: fix buffer free in slow path

2022-10-06 Thread Ferruh Yigit
On 10/6/2022 9:51 AM, Gagandeep Singh wrote: Hi, -Original Message- From: Ferruh Yigit Sent: Wednesday, October 5, 2022 7:52 PM To: Gagandeep Singh ; dev@dpdk.org Cc: sta...@dpdk.org Subject: Re: [PATCH 15/15] net/dpaa: fix buffer free in slow path On 9/28/2022 6:25 AM, Gagandeep

RE: [PATCH 15/15] net/dpaa: fix buffer free in slow path

2022-10-06 Thread Gagandeep Singh
Hi, > -Original Message- > From: Ferruh Yigit > Sent: Wednesday, October 5, 2022 7:52 PM > To: Gagandeep Singh ; dev@dpdk.org > Cc: sta...@dpdk.org > Subject: Re: [PATCH 15/15] net/dpaa: fix buffer free in slow path > > On 9/28/2022 6:25 AM, Gagandeep Singh wrot

Re: [PATCH 15/15] net/dpaa: fix buffer free in slow path

2022-10-05 Thread Ferruh Yigit
On 9/28/2022 6:25 AM, Gagandeep Singh wrote: Adding a check in slow path to free those buffers which are not external. Can you please explain what was the error before fix, what was happening when you try to free all mbufs? Also it seems previous logic was different, with 'prev_seg' etc, ca

[PATCH 15/15] net/dpaa: fix buffer free in slow path

2022-09-27 Thread Gagandeep Singh
Adding a check in slow path to free those buffers which are not external. Fixes: 9124e65dd3eb ("net/dpaa: enable Tx queue taildrop") Cc: sta...@dpdk.org Signed-off-by: Gagandeep Singh --- drivers/net/dpaa/dpaa_rxtx.c | 23 --- 1 file changed, 8 insertions(+), 15 deletions(-)