app/pdump: free mempool at cleanup resources

Message ID 1645750837-3700-1-git-send-email-laitianli@tom.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series app/pdump: free mempool at cleanup resources |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/iol-abi-testing success Testing PASS

Commit Message

Tianli Lai Feb. 25, 2022, 1 a.m. UTC
  the mempool should be free when cleanup resources.

Signed-off-by: Tianli Lai <laitianli@tom.com>
---
 app/pdump/main.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Stephen Hemminger Feb. 25, 2022, 2:40 a.m. UTC | #1
On Fri, 25 Feb 2022 09:00:37 +0800
Tianli Lai <laitianli@tom.com> wrote:

>  the mempool should be free when cleanup resources.
> 
> Signed-off-by: Tianli Lai <laitianli@tom.com>

Thanks for fixing this.

Acked-by: Stephen Hemminger <stephen@networkplumber.org>
  
Thomas Monjalon March 7, 2022, 11:21 p.m. UTC | #2
25/02/2022 03:40, Stephen Hemminger:
> On Fri, 25 Feb 2022 09:00:37 +0800
> Tianli Lai <laitianli@tom.com> wrote:
> 
> >  the mempool should be free when cleanup resources.
> > 
> > Signed-off-by: Tianli Lai <laitianli@tom.com>
> 
> Thanks for fixing this.
> 
> Acked-by: Stephen Hemminger <stephen@networkplumber.org>

Applied, thanks.
  

Patch

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_val {
 		/* free the rings */
 		rte_ring_free(pt->rx_ring);
 		rte_ring_free(pt->tx_ring);
+		rte_mempool_free(pt->mp);
 	}
 }