Re: [PATCH] app/pdump: free mempool at cleanup resources

2022-03-07 Thread Thomas Monjalon
25/02/2022 03:40, Stephen Hemminger: > On Fri, 25 Feb 2022 09:00:37 +0800 > Tianli Lai wrote: > > > the mempool should be free when cleanup resources. > > > > Signed-off-by: Tianli Lai > > Thanks for fixing this. > > Acked-by: Stephen Hemminger Applied, thanks.

Re: [PATCH] app/pdump: free mempool at cleanup resources

2022-02-24 Thread Stephen Hemminger
On Fri, 25 Feb 2022 09:00:37 +0800 Tianli Lai wrote: > the mempool should be free when cleanup resources. > > Signed-off-by: Tianli Lai Thanks for fixing this. Acked-by: Stephen Hemminger

[PATCH] app/pdump: free mempool at cleanup resources

2022-02-24 Thread Tianli Lai
the mempool should be free when cleanup resources. Signed-off-by: Tianli Lai --- app/pdump/main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/app/pdump/main.c b/app/pdump/main.c index 04a38e8..3fe17ea 100644 --- a/app/pdump/main.c +++ b/app/pdump/main.c @@ -507,6 +507,7 @@ struct parse_

Re: [PATCH] app/pdump: free mempool at cleanup resources

2022-02-24 Thread Stephen Hemminger
On Thu, 24 Feb 2022 20:14:58 +0800 Tianli Lai wrote: > + if (pt->mp) { > + rte_mempool_free(pt->mp); > + pt->mp = NULL; Check for null is unnecessary here, rte_mempool_free(NULL) is nop.

[PATCH] app/pdump: free mempool at cleanup resources

2022-02-24 Thread Tianli Lai
the mempool should be free when cleanup resources. Signed-off-by: Tianli Lai --- app/pdump/main.c | 4 1 file changed, 4 insertions(+) diff --git a/app/pdump/main.c b/app/pdump/main.c index 04a38e8..9c77fc6 100644 --- a/app/pdump/main.c +++ b/app/pdump/main.c @@ -507,6 +507,10 @@ struct pa

[PATCH] app/pdump: free mempool at cleanup resources

2022-02-24 Thread Tianli Lai
the mempool should be free when cleanup resources. Signed-off-by: Tianli Lai --- app/pdump/main.c | 4 1 file changed, 4 insertions(+) diff --git a/app/pdump/main.c b/app/pdump/main.c index 46f9d25..8101078 100644 --- a/app/pdump/main.c +++ b/app/pdump/main.c @@ -510,6 +510,10 @@ struct pa