Re: [dpdk-dev] [PATCH] net/virtio: fix memory leak of interrupt handle

2021-07-27 Thread Xia, Chenbo
> -Original Message- > From: Gaoxiang Liu > Sent: Monday, July 26, 2021 10:42 PM > To: Xia, Chenbo ; dev@dpdk.org; liugaoxi...@huawei.com > Cc: Gaoxiang Liu > Subject: [PATCH] net/virtio: fix memory leak of interrupt handle > > Free memory of interrupt handle in virtio_user_dev_uninit()

Re: [dpdk-dev] [PATCH] net/virtio: fix memory leak of interrupt handle

2021-07-26 Thread Xia, Chenbo
Missed R-by of Maxime will also be added when applying. /Chenbo > -Original Message- > From: dev On Behalf Of Xia, Chenbo > Sent: Tuesday, July 27, 2021 9:53 AM > To: Gaoxiang Liu ; dev@dpdk.org; liugaoxi...@huawei.com > Cc: Maxime Coquelin > Subject: Re: [dpdk-dev]

Re: [dpdk-dev] [PATCH] net/virtio: fix memory leak of interrupt handle

2021-07-26 Thread Xia, Chenbo
Hi Gaoxiang, > -Original Message- > From: Gaoxiang Liu > Sent: Monday, July 26, 2021 10:42 PM > To: Xia, Chenbo ; dev@dpdk.org; liugaoxi...@huawei.com > Cc: Gaoxiang Liu > Subject: [PATCH] net/virtio: fix memory leak of interrupt handle > > Free memory of interrupt handle in virtio_user

[dpdk-dev] [PATCH] net/virtio: fix memory leak of interrupt handle

2021-07-26 Thread Gaoxiang Liu
Free memory of interrupt handle in virtio_user_dev_uninit() to avoid memory leak. when virtio user dev closes, memory of interrupt handle is not freed that is allocated in virtio_user_fill_intr_handle(). Fixes: 3d4fb6fd2505 <"net/virtio-user: support Rx interrupt"> Signed-off-by: Gaoxiang Liu --

Re: [dpdk-dev] [PATCH] net/virtio: fix memory leak of interrupt handle

2021-07-26 Thread Maxime Coquelin
On 7/26/21 1:59 PM, Gaoxiang Liu wrote: > Free memory of interrupt handle in virtio_user_dev_uninit() to > avoid memory leak. > when virtio user dev closes, the "intr_handle" memory is not freed > that is allocated in virtio_user_fill_intr_handle(). > > Fixes: 3d4fb6fd2505 <"net/virtio-user: su

[dpdk-dev] [PATCH] net/virtio: fix memory leak of interrupt handle

2021-07-26 Thread Gaoxiang Liu
Free memory of interrupt handle in virtio_user_dev_uninit() to avoid memory leak. when virtio user dev closes, the "intr_handle" memory is not freed that is allocated in virtio_user_fill_intr_handle(). Fixes: 3d4fb6fd2505 <"net/virtio-user: support Rx interrupt"> Signed-off-by: Gaoxiang Liu ---

Re: [dpdk-dev] [PATCH] net/virtio: fix memory leak in dev close

2021-07-26 Thread 刘高翔
>Hi, chen bo >>And I think the fix commit id is not correct. Fix commit id should be the >>first commit that introduces the bug. I think it's: >>3d4fb6fd2505 ("net/virtio-user: support Rx interrupt") > Yes, it should be the commit id. And I will fix other issues right now. > Thanks. At 2021-0

Re: [dpdk-dev] [PATCH] net/virtio: fix memory leak in dev close

2021-07-26 Thread Xia, Chenbo
Hi Gaoxiang, > -Original Message- > From: Gaoxiang Liu > Sent: Sunday, July 25, 2021 10:03 AM > To: Maxime Coquelin ; Xia, Chenbo > > Cc: dev@dpdk.org; liugaoxi...@huawei.com; Gaoxiang Liu > Subject: [PATCH] net/virtio: fix memory leak in dev close I suggest the title: fix memory leak

[dpdk-dev] [PATCH] net/virtio: fix memory leak in dev close

2021-07-24 Thread Gaoxiang Liu
Free the "intr_handle" memory in virtio_user_dev_uninit() to avoid memory leak. when virtio user dev closes, the "intr_handle" memory is not freeed that is alloced in virtio_user_fill_intr_handle(). Fixes: 7f468b2ebfad ("net/virtio: release port upon close") Signed-off-by: Gaoxiang Liu --- driv

[dpdk-dev] [PATCH] net/virtio: fix memory leak

2017-10-26 Thread Pengzhen Liu
In function eth_virtio_dev_init(), dynamic memory stored in "eth_dev->data->mac_addrs" variable and it is not freed when function return, this is a possible memory leak. Fixes: 8ced1542f7a3 ("net/virtio: eth_dev->data->mac_addrs is not freed") Cc: sta...@dpdk.org Signed-off-by: Pengzhen Liu ---

[dpdk-dev] [PATCH] net/virtio: fix memory leak

2017-10-26 Thread Pengzhen Liu
In function eth_virtio_dev_init(), dynamic memory stored in "eth_dev->data->mac_addrs" variable and it is not freed when function return, this is a possible memory leak. Fixes: 289ba0c0f52a ("net/virtio: eth_dev->data->mac_addrs is not freed") Cc: sta...@dpdk.org Signed-off-by: Pengzhen Liu ---

Re: [dpdk-dev] [PATCH] net/virtio: fix memory leak

2017-10-26 Thread Maxime Coquelin
Hi Pengzhen, On 10/26/2017 10:52 AM, Pengzhen Liu wrote: In function eth_virtio_dev_init(), dynamic memory stored in "eth_dev->data->mac_addrs" variable and it is not freed when function return, this is a possible memory leak. Fixes: 6b9ed026a870 ("net/virtio: eth_dev->data->mac_addrs is not fr

[dpdk-dev] [PATCH] net/virtio: fix memory leak

2017-10-26 Thread Pengzhen Liu
In function eth_virtio_dev_init(), dynamic memory stored in "eth_dev->data->mac_addrs" variable and it is not freed when function return, this is a possible memory leak. Fixes: 6b9ed026a870 ("net/virtio: eth_dev->data->mac_addrs is not freed") Cc: sta...@dpdk.org Signed-off-by: Pengzhen Liu ---

[dpdk-dev] [PATCH] net/virtio: fix memory leak

2017-10-26 Thread Pengzhen Liu
In function eth_virtio_dev_init(), dynamic memory stored in "eth_dev->data->mac_addrs" variable and it is not freed when function return, this is a possible memory leak. Fixes: 6b9ed026a870 ("net/virtio: eth_dev->data->mac_addrs is not freed") Cc: sta...@dpdk.org Signed-off-by: Pengzhen Liu ---

[dpdk-dev] [PATCH] net/virtio: fix memory leak

2017-10-25 Thread Pengzhen Liu
In function eth_virtio_dev_init(), dynamic memory stored in "eth_dev->data->mac_addrs" variable and it is not freed whe function return, this is a possible memory leak. Fixes: 6b9ed026a870 ("net/virtio: eth_dev->data->mac_addrs is not freed") Cc: sta...@dpdk.org Signed-off-by: Pengzhen Liu --- d