DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] pdump: fix uninit not freeing statistics memzone
@ 2021-10-26 11:53 Konstantin Ananyev
  2021-10-26 16:54 ` Pattan, Reshma
  2021-10-26 19:57 ` Stephen Hemminger
  0 siblings, 2 replies; 4+ messages in thread
From: Konstantin Ananyev @ 2021-10-26 11:53 UTC (permalink / raw)
  To: dev; +Cc: stephen, reshma.pattan, Konstantin Ananyev

rte_pdump_init() always allocates new memzone for pdump_stats.
Though rte_pdump_uninit() never frees it.
So the following combination will always fail:
rte_pdump_init(); rte_pdump_uninit(); rte_pdump_init();
The issue was caught by pdump_autotest UT.
While first test run successful, any consecutive runs
of this test-case will fail.
Fix the issue by calling rte_memzone_free() for statistics memzone.

Fixes: 10f726efe26c ("pdump: support pcapng and filtering")

Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
---
 lib/pdump/rte_pdump.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/lib/pdump/rte_pdump.c b/lib/pdump/rte_pdump.c
index 71602685d5..a708935861 100644
--- a/lib/pdump/rte_pdump.c
+++ b/lib/pdump/rte_pdump.c
@@ -74,6 +74,7 @@ static const char MZ_RTE_PDUMP_STATS[] = "rte_pdump_stats";
 static struct {
 	struct rte_pdump_stats rx[RTE_MAX_ETHPORTS][RTE_MAX_QUEUES_PER_PORT];
 	struct rte_pdump_stats tx[RTE_MAX_ETHPORTS][RTE_MAX_QUEUES_PER_PORT];
+	const struct rte_memzone *mz;
 } *pdump_stats;
 
 /* Create a clone of mbuf to be placed into ring. */
@@ -429,6 +430,7 @@ rte_pdump_init(void)
 		return -1;
 	}
 	pdump_stats = mz->addr;
+	pdump_stats->mz = mz;
 
 	ret = rte_mp_action_register(PDUMP_MP, pdump_server);
 	if (ret && rte_errno != ENOTSUP)
@@ -441,6 +443,11 @@ rte_pdump_uninit(void)
 {
 	rte_mp_action_unregister(PDUMP_MP);
 
+	if (pdump_stats != NULL) {
+		rte_memzone_free(pdump_stats->mz);
+		pdump_stats = NULL;
+	}
+
 	return 0;
 }
 
-- 
2.26.3


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] pdump: fix uninit not freeing statistics memzone
  2021-10-26 11:53 [dpdk-dev] [PATCH] pdump: fix uninit not freeing statistics memzone Konstantin Ananyev
@ 2021-10-26 16:54 ` Pattan, Reshma
  2021-10-26 19:57 ` Stephen Hemminger
  1 sibling, 0 replies; 4+ messages in thread
From: Pattan, Reshma @ 2021-10-26 16:54 UTC (permalink / raw)
  To: Ananyev, Konstantin, dev; +Cc: stephen



> -----Original Message-----
> From: Ananyev, Konstantin <konstantin.ananyev@intel.com>
.
> 
> Fixes: 10f726efe26c ("pdump: support pcapng and filtering")

The changes looks ok. In the commit message we should also add CC: with author name and email id below the fixes line.

> 
> Signed-off-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
Acked-by:  Reshma Pattan <reshma.pattan@intel.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] pdump: fix uninit not freeing statistics memzone
  2021-10-26 11:53 [dpdk-dev] [PATCH] pdump: fix uninit not freeing statistics memzone Konstantin Ananyev
  2021-10-26 16:54 ` Pattan, Reshma
@ 2021-10-26 19:57 ` Stephen Hemminger
  2021-11-03 11:54   ` Thomas Monjalon
  1 sibling, 1 reply; 4+ messages in thread
From: Stephen Hemminger @ 2021-10-26 19:57 UTC (permalink / raw)
  To: Konstantin Ananyev; +Cc: dev, reshma.pattan

On Tue, 26 Oct 2021 12:53:01 +0100
Konstantin Ananyev <konstantin.ananyev@intel.com> wrote:

> diff --git a/lib/pdump/rte_pdump.c b/lib/pdump/rte_pdump.c
> index 71602685d5..a708935861 100644
> --- a/lib/pdump/rte_pdump.c
> +++ b/lib/pdump/rte_pdump.c
> @@ -74,6 +74,7 @@ static const char MZ_RTE_PDUMP_STATS[] = "rte_pdump_stats";
>  static struct {
>  	struct rte_pdump_stats rx[RTE_MAX_ETHPORTS][RTE_MAX_QUEUES_PER_PORT];
>  	struct rte_pdump_stats tx[RTE_MAX_ETHPORTS][RTE_MAX_QUEUES_PER_PORT];
> +	const struct rte_memzone *mz;
>  } *pdump_stats;

Thanks for fixing.

Acked-by: Stephen Hemminger <stephen@networkplumber.org>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [dpdk-dev] [PATCH] pdump: fix uninit not freeing statistics memzone
  2021-10-26 19:57 ` Stephen Hemminger
@ 2021-11-03 11:54   ` Thomas Monjalon
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2021-11-03 11:54 UTC (permalink / raw)
  To: Konstantin Ananyev; +Cc: dev, reshma.pattan, Stephen Hemminger

26/10/2021 21:57, Stephen Hemminger:
> On Tue, 26 Oct 2021 12:53:01 +0100
> Konstantin Ananyev <konstantin.ananyev@intel.com> wrote:
> 
> > diff --git a/lib/pdump/rte_pdump.c b/lib/pdump/rte_pdump.c
> > index 71602685d5..a708935861 100644
> > --- a/lib/pdump/rte_pdump.c
> > +++ b/lib/pdump/rte_pdump.c
> > @@ -74,6 +74,7 @@ static const char MZ_RTE_PDUMP_STATS[] = "rte_pdump_stats";
> >  static struct {
> >  	struct rte_pdump_stats rx[RTE_MAX_ETHPORTS][RTE_MAX_QUEUES_PER_PORT];
> >  	struct rte_pdump_stats tx[RTE_MAX_ETHPORTS][RTE_MAX_QUEUES_PER_PORT];
> > +	const struct rte_memzone *mz;
> >  } *pdump_stats;
> 
> Thanks for fixing.
> 
> Acked-by: Stephen Hemminger <stephen@networkplumber.org>

Applied, thanks.




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-11-03 11:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-26 11:53 [dpdk-dev] [PATCH] pdump: fix uninit not freeing statistics memzone Konstantin Ananyev
2021-10-26 16:54 ` Pattan, Reshma
2021-10-26 19:57 ` Stephen Hemminger
2021-11-03 11:54   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).