DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH 1/2] app/proc-info: add memory heap dump
@ 2024-07-02 13:14 Gagandeep Singh
  2024-07-02 13:14 ` [PATCH 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
                   ` (3 more replies)
  0 siblings, 4 replies; 15+ messages in thread
From: Gagandeep Singh @ 2024-07-02 13:14 UTC (permalink / raw)
  To: dev, Reshma Pattan; +Cc: hemant.agrawal

This patch add the heap dump support in proc-info
memory dump option.

Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
---
 app/proc-info/main.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/proc-info/main.c b/app/proc-info/main.c
index b672aaefbe..7137891c14 100644
--- a/app/proc-info/main.c
+++ b/app/proc-info/main.c
@@ -20,6 +20,7 @@
 #include <rte_debug.h>
 #include <rte_ethdev.h>
 #include <rte_memory.h>
+#include <rte_malloc.h>
 #include <rte_memzone.h>
 #include <rte_launch.h>
 #include <rte_tailq.h>
@@ -637,6 +638,10 @@ meminfo_display(void)
 	rte_memzone_dump(stdout);
 	printf("---------- END_MEMORY_ZONES -----------\n");
 
+	printf("------------ HEAP DUMP -------------\n");
+	rte_malloc_dump_heaps(stdout);
+	printf("---------- END_HEAP_DUMP -----------\n");
+
 	printf("------------- TAIL_QUEUES -------------\n");
 	rte_dump_tailq(stdout);
 	printf("---------- END_TAIL_QUEUES ------------\n");
-- 
2.25.1


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

* [PATCH 2/2] eal: add total memory size in memory dump APIs
  2024-07-02 13:14 [PATCH 1/2] app/proc-info: add memory heap dump Gagandeep Singh
@ 2024-07-02 13:14 ` Gagandeep Singh
  2024-07-03  8:33 ` [PATCH 1/2] app/proc-info: add memory heap dump Hemant Agrawal
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 15+ messages in thread
From: Gagandeep Singh @ 2024-07-02 13:14 UTC (permalink / raw)
  To: dev, Anatoly Burakov, Tyler Retzlaff; +Cc: hemant.agrawal

This patch add total memory size dump in memzone and
memsegments dump APIs.

Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
---
 lib/eal/common/eal_common_memory.c  |  2 ++
 lib/eal/common/eal_common_memzone.c | 18 ++++++++++++++++--
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/lib/eal/common/eal_common_memory.c b/lib/eal/common/eal_common_memory.c
index 60ddc30580..c6b9c16617 100644
--- a/lib/eal/common/eal_common_memory.c
+++ b/lib/eal/common/eal_common_memory.c
@@ -531,6 +531,8 @@ void
 rte_dump_physmem_layout(FILE *f)
 {
 	rte_memseg_walk(dump_memseg, f);
+	fprintf(f, "Total Memory Segments size = %uM\n",
+			(unsigned int) rte_eal_get_physmem_size() / (1024 * 1024));
 }
 
 static int
diff --git a/lib/eal/common/eal_common_memzone.c b/lib/eal/common/eal_common_memzone.c
index 32e6b78f87..1d12ccc443 100644
--- a/lib/eal/common/eal_common_memzone.c
+++ b/lib/eal/common/eal_common_memzone.c
@@ -58,6 +58,11 @@ rte_memzone_max_get(void)
 	return mcfg->max_memzone;
 }
 
+struct memzone_info {
+	FILE *f;
+	uint64_t t_size;
+};
+
 static inline const struct rte_memzone *
 memzone_lookup_thread_unsafe(const char *name)
 {
@@ -369,7 +374,8 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 	struct rte_memseg *ms;
 	int mz_idx, ms_idx;
 	size_t page_sz;
-	FILE *f = arg;
+	struct memzone_info *info = arg;
+	FILE *f = info->f;
 
 	mz_idx = rte_fbarray_find_idx(&mcfg->memzones, mz);
 
@@ -382,6 +388,7 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 			mz->socket_id,
 			mz->flags);
 
+	info->t_size += mz->len;
 	/* go through each page occupied by this memzone */
 	msl = rte_mem_virt2memseg_list(mz->addr);
 	if (!msl) {
@@ -414,7 +421,14 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 void
 rte_memzone_dump(FILE *f)
 {
-	rte_memzone_walk(dump_memzone, f);
+	struct memzone_info info;
+
+	memset(&info, 0, sizeof(info));
+	info.f = f;
+
+	rte_memzone_walk(dump_memzone, &info);
+	fprintf(f, "Total Memory Zones size = %uM\n", (unsigned int)info.t_size
+			/ (1024 * 1024));
 }
 
 /*
-- 
2.25.1


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

* Re: [PATCH 1/2] app/proc-info: add memory heap dump
  2024-07-02 13:14 [PATCH 1/2] app/proc-info: add memory heap dump Gagandeep Singh
  2024-07-02 13:14 ` [PATCH 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
@ 2024-07-03  8:33 ` Hemant Agrawal
  2024-07-29 17:18 ` Thomas Monjalon
  2024-07-30 11:03 ` [v2 0/2] proc-info memory dump enhancement Gagandeep Singh
  3 siblings, 0 replies; 15+ messages in thread
From: Hemant Agrawal @ 2024-07-03  8:33 UTC (permalink / raw)
  To: dev


On 02-07-2024 18:44, Gagandeep Singh wrote:
> This patch add the heap dump support in proc-info
> memory dump option.
>
> Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
> ---
>   app/proc-info/main.c | 5 +++++
>   1 file changed, 5 insertions(+)

Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>



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

* Re: [PATCH 1/2] app/proc-info: add memory heap dump
  2024-07-02 13:14 [PATCH 1/2] app/proc-info: add memory heap dump Gagandeep Singh
  2024-07-02 13:14 ` [PATCH 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
  2024-07-03  8:33 ` [PATCH 1/2] app/proc-info: add memory heap dump Hemant Agrawal
@ 2024-07-29 17:18 ` Thomas Monjalon
  2024-07-30 11:10   ` Gagandeep Singh
  2024-07-30 11:03 ` [v2 0/2] proc-info memory dump enhancement Gagandeep Singh
  3 siblings, 1 reply; 15+ messages in thread
From: Thomas Monjalon @ 2024-07-29 17:18 UTC (permalink / raw)
  To: Gagandeep Singh; +Cc: dev, Reshma Pattan, hemant.agrawal

02/07/2024 15:14, Gagandeep Singh:
> --- a/app/proc-info/main.c
> +++ b/app/proc-info/main.c
> @@ -637,6 +638,10 @@ meminfo_display(void)
>  	rte_memzone_dump(stdout);
>  	printf("---------- END_MEMORY_ZONES -----------\n");
>  
> +	printf("------------ HEAP DUMP -------------\n");

Should we add "MALLOC"?
With underscores for consistency?

> +	rte_malloc_dump_heaps(stdout);
> +	printf("---------- END_HEAP_DUMP -----------\n");
> +
>  	printf("------------- TAIL_QUEUES -------------\n");
>  	rte_dump_tailq(stdout);
>  	printf("---------- END_TAIL_QUEUES ------------\n");
> 






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

* [v2 0/2] proc-info memory dump enhancement
  2024-07-02 13:14 [PATCH 1/2] app/proc-info: add memory heap dump Gagandeep Singh
                   ` (2 preceding siblings ...)
  2024-07-29 17:18 ` Thomas Monjalon
@ 2024-07-30 11:03 ` Gagandeep Singh
  2024-07-30 11:03   ` [v2 1/2] app/proc-info: add memory heap dump Gagandeep Singh
                     ` (2 more replies)
  3 siblings, 3 replies; 15+ messages in thread
From: Gagandeep Singh @ 2024-07-30 11:03 UTC (permalink / raw)
  To: dev

v2 change:
* Handled a comment to add "MALLOC" while dumping
  malloc heaps.

Gagandeep Singh (2):
  app/proc-info: add memory heap dump
  eal: add total memory size in memory dump APIs

 app/proc-info/main.c                |  5 +++++
 lib/eal/common/eal_common_memory.c  |  2 ++
 lib/eal/common/eal_common_memzone.c | 18 ++++++++++++++++--
 3 files changed, 23 insertions(+), 2 deletions(-)

-- 
2.25.1


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

* [v2 1/2] app/proc-info: add memory heap dump
  2024-07-30 11:03 ` [v2 0/2] proc-info memory dump enhancement Gagandeep Singh
@ 2024-07-30 11:03   ` Gagandeep Singh
  2024-10-10  7:16     ` [v3 " Gagandeep Singh
  2024-07-30 11:03   ` [v2 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
  2024-10-11 13:15   ` [v2 0/2] proc-info memory dump enhancement David Marchand
  2 siblings, 1 reply; 15+ messages in thread
From: Gagandeep Singh @ 2024-07-30 11:03 UTC (permalink / raw)
  To: dev, Reshma Pattan; +Cc: Hemant Agrawal

This patch add the malloc heap dump support in proc-info
memory dump option.

Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 app/proc-info/main.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/proc-info/main.c b/app/proc-info/main.c
index b672aaefbe..9e26b5cf3b 100644
--- a/app/proc-info/main.c
+++ b/app/proc-info/main.c
@@ -20,6 +20,7 @@
 #include <rte_debug.h>
 #include <rte_ethdev.h>
 #include <rte_memory.h>
+#include <rte_malloc.h>
 #include <rte_memzone.h>
 #include <rte_launch.h>
 #include <rte_tailq.h>
@@ -637,6 +638,10 @@ meminfo_display(void)
 	rte_memzone_dump(stdout);
 	printf("---------- END_MEMORY_ZONES -----------\n");
 
+	printf("---------- MALLOC_HEAP_DUMP -----------\n");
+	rte_malloc_dump_heaps(stdout);
+	printf("-------- END_MALLOC_HEAP_DUMP ---------\n");
+
 	printf("------------- TAIL_QUEUES -------------\n");
 	rte_dump_tailq(stdout);
 	printf("---------- END_TAIL_QUEUES ------------\n");
-- 
2.25.1


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

* [v2 2/2] eal: add total memory size in memory dump APIs
  2024-07-30 11:03 ` [v2 0/2] proc-info memory dump enhancement Gagandeep Singh
  2024-07-30 11:03   ` [v2 1/2] app/proc-info: add memory heap dump Gagandeep Singh
@ 2024-07-30 11:03   ` Gagandeep Singh
  2024-10-10  0:25     ` Stephen Hemminger
  2024-10-11 13:15   ` [v2 0/2] proc-info memory dump enhancement David Marchand
  2 siblings, 1 reply; 15+ messages in thread
From: Gagandeep Singh @ 2024-07-30 11:03 UTC (permalink / raw)
  To: dev, Anatoly Burakov, Tyler Retzlaff

This patch add total memory size dump in memzone and
memsegments dump APIs.

Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
---
 lib/eal/common/eal_common_memory.c  |  2 ++
 lib/eal/common/eal_common_memzone.c | 18 ++++++++++++++++--
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/lib/eal/common/eal_common_memory.c b/lib/eal/common/eal_common_memory.c
index 60ddc30580..c6b9c16617 100644
--- a/lib/eal/common/eal_common_memory.c
+++ b/lib/eal/common/eal_common_memory.c
@@ -531,6 +531,8 @@ void
 rte_dump_physmem_layout(FILE *f)
 {
 	rte_memseg_walk(dump_memseg, f);
+	fprintf(f, "Total Memory Segments size = %uM\n",
+			(unsigned int) rte_eal_get_physmem_size() / (1024 * 1024));
 }
 
 static int
diff --git a/lib/eal/common/eal_common_memzone.c b/lib/eal/common/eal_common_memzone.c
index 2d9b6aa3e3..4cd077d8d8 100644
--- a/lib/eal/common/eal_common_memzone.c
+++ b/lib/eal/common/eal_common_memzone.c
@@ -58,6 +58,11 @@ rte_memzone_max_get(void)
 	return mcfg->max_memzone;
 }
 
+struct memzone_info {
+	FILE *f;
+	uint64_t t_size;
+};
+
 static inline const struct rte_memzone *
 memzone_lookup_thread_unsafe(const char *name)
 {
@@ -367,7 +372,8 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 	struct rte_memseg *ms;
 	int mz_idx, ms_idx;
 	size_t page_sz;
-	FILE *f = arg;
+	struct memzone_info *info = arg;
+	FILE *f = info->f;
 
 	mz_idx = rte_fbarray_find_idx(&mcfg->memzones, mz);
 
@@ -380,6 +386,7 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 			mz->socket_id,
 			mz->flags);
 
+	info->t_size += mz->len;
 	/* go through each page occupied by this memzone */
 	msl = rte_mem_virt2memseg_list(mz->addr);
 	if (!msl) {
@@ -412,7 +419,14 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 void
 rte_memzone_dump(FILE *f)
 {
-	rte_memzone_walk(dump_memzone, f);
+	struct memzone_info info;
+
+	memset(&info, 0, sizeof(info));
+	info.f = f;
+
+	rte_memzone_walk(dump_memzone, &info);
+	fprintf(f, "Total Memory Zones size = %uM\n", (unsigned int)info.t_size
+			/ (1024 * 1024));
 }
 
 /*
-- 
2.25.1


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

* RE: [PATCH 1/2] app/proc-info: add memory heap dump
  2024-07-29 17:18 ` Thomas Monjalon
@ 2024-07-30 11:10   ` Gagandeep Singh
  0 siblings, 0 replies; 15+ messages in thread
From: Gagandeep Singh @ 2024-07-30 11:10 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, Reshma Pattan, Hemant Agrawal

Hi,

> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Monday, July 29, 2024 10:48 PM
> To: Gagandeep Singh <G.Singh@nxp.com>
> Cc: dev@dpdk.org; Reshma Pattan <reshma.pattan@intel.com>; Hemant
> Agrawal <hemant.agrawal@nxp.com>
> Subject: Re: [PATCH 1/2] app/proc-info: add memory heap dump
> 
> 02/07/2024 15:14, Gagandeep Singh:
> > --- a/app/proc-info/main.c
> > +++ b/app/proc-info/main.c
> > @@ -637,6 +638,10 @@ meminfo_display(void)
> >  	rte_memzone_dump(stdout);
> >  	printf("---------- END_MEMORY_ZONES -----------\n");
> >
> > +	printf("------------ HEAP DUMP -------------\n");
> 
> Should we add "MALLOC"?
> With underscores for consistency?
Ok, submitted the V2.

> 
> > +	rte_malloc_dump_heaps(stdout);
> > +	printf("---------- END_HEAP_DUMP -----------\n");
> > +
> >  	printf("------------- TAIL_QUEUES -------------\n");
> >  	rte_dump_tailq(stdout);
> >  	printf("---------- END_TAIL_QUEUES ------------\n");
> >
> 
> 
> 
> 


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

* Re: [v2 2/2] eal: add total memory size in memory dump APIs
  2024-07-30 11:03   ` [v2 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
@ 2024-10-10  0:25     ` Stephen Hemminger
  0 siblings, 0 replies; 15+ messages in thread
From: Stephen Hemminger @ 2024-10-10  0:25 UTC (permalink / raw)
  To: Gagandeep Singh; +Cc: dev, Anatoly Burakov, Tyler Retzlaff

On Tue, 30 Jul 2024 16:33:13 +0530
Gagandeep Singh <g.singh@nxp.com> wrote:

> This patch add total memory size dump in memzone and
> memsegments dump APIs.
> 
> Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
> ---
>  lib/eal/common/eal_common_memory.c  |  2 ++
>  lib/eal/common/eal_common_memzone.c | 18 ++++++++++++++++--
>  2 files changed, 18 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/eal/common/eal_common_memory.c b/lib/eal/common/eal_common_memory.c
> index 60ddc30580..c6b9c16617 100644
> --- a/lib/eal/common/eal_common_memory.c
> +++ b/lib/eal/common/eal_common_memory.c
> @@ -531,6 +531,8 @@ void
>  rte_dump_physmem_layout(FILE *f)
>  {
>  	rte_memseg_walk(dump_memseg, f);
> +	fprintf(f, "Total Memory Segments size = %uM\n",
> +			(unsigned int) rte_eal_get_physmem_size() / (1024 * 1024));
>  }

You are going to get truncated result here because rte_eal_get_physmem_size() is uint64_t
and unsigned int is 32 bit. The cast happens before the division which leads to
truncation with 4G. Simplest fix would be

	fprintf(f, "Total Memory Segments size = %"PRIu64"M\n",
		rte_eal_get_physmem_size() / (1024 * 1024));

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

* [v3 1/2] app/proc-info: add memory heap dump
  2024-07-30 11:03   ` [v2 1/2] app/proc-info: add memory heap dump Gagandeep Singh
@ 2024-10-10  7:16     ` Gagandeep Singh
  2024-10-10  7:16       ` [v3 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
  2024-10-10 14:53       ` [v3 1/2] app/proc-info: add memory heap dump Stephen Hemminger
  0 siblings, 2 replies; 15+ messages in thread
From: Gagandeep Singh @ 2024-10-10  7:16 UTC (permalink / raw)
  To: dev, Reshma Pattan; +Cc: Hemant Agrawal

This patch add the malloc heap dump support in proc-info
memory dump option.

Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 app/proc-info/main.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/proc-info/main.c b/app/proc-info/main.c
index b672aaefbe..9e26b5cf3b 100644
--- a/app/proc-info/main.c
+++ b/app/proc-info/main.c
@@ -20,6 +20,7 @@
 #include <rte_debug.h>
 #include <rte_ethdev.h>
 #include <rte_memory.h>
+#include <rte_malloc.h>
 #include <rte_memzone.h>
 #include <rte_launch.h>
 #include <rte_tailq.h>
@@ -637,6 +638,10 @@ meminfo_display(void)
 	rte_memzone_dump(stdout);
 	printf("---------- END_MEMORY_ZONES -----------\n");
 
+	printf("---------- MALLOC_HEAP_DUMP -----------\n");
+	rte_malloc_dump_heaps(stdout);
+	printf("-------- END_MALLOC_HEAP_DUMP ---------\n");
+
 	printf("------------- TAIL_QUEUES -------------\n");
 	rte_dump_tailq(stdout);
 	printf("---------- END_TAIL_QUEUES ------------\n");
-- 
2.25.1


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

* [v3 2/2] eal: add total memory size in memory dump APIs
  2024-10-10  7:16     ` [v3 " Gagandeep Singh
@ 2024-10-10  7:16       ` Gagandeep Singh
  2024-10-10 14:52         ` Stephen Hemminger
  2024-10-10 14:53         ` Stephen Hemminger
  2024-10-10 14:53       ` [v3 1/2] app/proc-info: add memory heap dump Stephen Hemminger
  1 sibling, 2 replies; 15+ messages in thread
From: Gagandeep Singh @ 2024-10-10  7:16 UTC (permalink / raw)
  To: dev, Anatoly Burakov, Tyler Retzlaff

This patch add total memory size dump in memzone and
memsegments dump APIs.

Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
---
 lib/eal/common/eal_common_memory.c  |  2 ++
 lib/eal/common/eal_common_memzone.c | 18 ++++++++++++++++--
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/lib/eal/common/eal_common_memory.c b/lib/eal/common/eal_common_memory.c
index 60ddc30580..7b4fb4c60d 100644
--- a/lib/eal/common/eal_common_memory.c
+++ b/lib/eal/common/eal_common_memory.c
@@ -531,6 +531,8 @@ void
 rte_dump_physmem_layout(FILE *f)
 {
 	rte_memseg_walk(dump_memseg, f);
+	fprintf(f, "Total Memory Segments size = %"PRIu64"M\n",
+			rte_eal_get_physmem_size() / (1024 * 1024));
 }
 
 static int
diff --git a/lib/eal/common/eal_common_memzone.c b/lib/eal/common/eal_common_memzone.c
index 2d9b6aa3e3..5427cbd0cb 100644
--- a/lib/eal/common/eal_common_memzone.c
+++ b/lib/eal/common/eal_common_memzone.c
@@ -58,6 +58,11 @@ rte_memzone_max_get(void)
 	return mcfg->max_memzone;
 }
 
+struct memzone_info {
+	FILE *f;
+	uint64_t t_size;
+};
+
 static inline const struct rte_memzone *
 memzone_lookup_thread_unsafe(const char *name)
 {
@@ -367,7 +372,8 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 	struct rte_memseg *ms;
 	int mz_idx, ms_idx;
 	size_t page_sz;
-	FILE *f = arg;
+	struct memzone_info *info = arg;
+	FILE *f = info->f;
 
 	mz_idx = rte_fbarray_find_idx(&mcfg->memzones, mz);
 
@@ -380,6 +386,7 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 			mz->socket_id,
 			mz->flags);
 
+	info->t_size += mz->len;
 	/* go through each page occupied by this memzone */
 	msl = rte_mem_virt2memseg_list(mz->addr);
 	if (!msl) {
@@ -412,7 +419,14 @@ dump_memzone(const struct rte_memzone *mz, void *arg)
 void
 rte_memzone_dump(FILE *f)
 {
-	rte_memzone_walk(dump_memzone, f);
+	struct memzone_info info;
+
+	memset(&info, 0, sizeof(info));
+	info.f = f;
+
+	rte_memzone_walk(dump_memzone, &info);
+	fprintf(f, "Total Memory Zones size = %"PRIu64"M\n", info.t_size
+			/ (1024 * 1024));
 }
 
 /*
-- 
2.25.1


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

* Re: [v3 2/2] eal: add total memory size in memory dump APIs
  2024-10-10  7:16       ` [v3 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
@ 2024-10-10 14:52         ` Stephen Hemminger
  2024-10-10 14:53         ` Stephen Hemminger
  1 sibling, 0 replies; 15+ messages in thread
From: Stephen Hemminger @ 2024-10-10 14:52 UTC (permalink / raw)
  To: Gagandeep Singh; +Cc: dev, Anatoly Burakov, Tyler Retzlaff

On Thu, 10 Oct 2024 12:46:21 +0530
Gagandeep Singh <g.singh@nxp.com> wrote:

> This patch add total memory size dump in memzone and
> memsegments dump APIs.
> 
> Signed-off-by: Gagandeep Singh <g.singh@nxp.com>

Looks good, and acking it as is.

But you could simplify this:
+	struct memzone_info info;
+
+	memset(&info, 0, sizeof(info));
+	info.f = f;

As:
	struct memzone_info info = { .f = f };

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

* Re: [v3 1/2] app/proc-info: add memory heap dump
  2024-10-10  7:16     ` [v3 " Gagandeep Singh
  2024-10-10  7:16       ` [v3 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
@ 2024-10-10 14:53       ` Stephen Hemminger
  1 sibling, 0 replies; 15+ messages in thread
From: Stephen Hemminger @ 2024-10-10 14:53 UTC (permalink / raw)
  To: Gagandeep Singh; +Cc: dev, Reshma Pattan, Hemant Agrawal

On Thu, 10 Oct 2024 12:46:20 +0530
Gagandeep Singh <g.singh@nxp.com> wrote:

> This patch add the malloc heap dump support in proc-info
> memory dump option.
> 
> Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
> Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

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

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

* Re: [v3 2/2] eal: add total memory size in memory dump APIs
  2024-10-10  7:16       ` [v3 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
  2024-10-10 14:52         ` Stephen Hemminger
@ 2024-10-10 14:53         ` Stephen Hemminger
  1 sibling, 0 replies; 15+ messages in thread
From: Stephen Hemminger @ 2024-10-10 14:53 UTC (permalink / raw)
  To: Gagandeep Singh; +Cc: dev, Anatoly Burakov, Tyler Retzlaff

On Thu, 10 Oct 2024 12:46:21 +0530
Gagandeep Singh <g.singh@nxp.com> wrote:

> This patch add total memory size dump in memzone and
> memsegments dump APIs.
> 
> Signed-off-by: Gagandeep Singh <g.singh@nxp.com>

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

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

* Re: [v2 0/2] proc-info memory dump enhancement
  2024-07-30 11:03 ` [v2 0/2] proc-info memory dump enhancement Gagandeep Singh
  2024-07-30 11:03   ` [v2 1/2] app/proc-info: add memory heap dump Gagandeep Singh
  2024-07-30 11:03   ` [v2 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
@ 2024-10-11 13:15   ` David Marchand
  2 siblings, 0 replies; 15+ messages in thread
From: David Marchand @ 2024-10-11 13:15 UTC (permalink / raw)
  To: Gagandeep Singh; +Cc: dev

On Tue, Jul 30, 2024 at 2:04 PM Gagandeep Singh <g.singh@nxp.com> wrote:
>
> v2 change:
> * Handled a comment to add "MALLOC" while dumping
>   malloc heaps.
>
> Gagandeep Singh (2):
>   app/proc-info: add memory heap dump
>   eal: add total memory size in memory dump APIs

Series applied, thanks.


-- 
David Marchand


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

end of thread, other threads:[~2024-10-11 13:15 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-07-02 13:14 [PATCH 1/2] app/proc-info: add memory heap dump Gagandeep Singh
2024-07-02 13:14 ` [PATCH 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
2024-07-03  8:33 ` [PATCH 1/2] app/proc-info: add memory heap dump Hemant Agrawal
2024-07-29 17:18 ` Thomas Monjalon
2024-07-30 11:10   ` Gagandeep Singh
2024-07-30 11:03 ` [v2 0/2] proc-info memory dump enhancement Gagandeep Singh
2024-07-30 11:03   ` [v2 1/2] app/proc-info: add memory heap dump Gagandeep Singh
2024-10-10  7:16     ` [v3 " Gagandeep Singh
2024-10-10  7:16       ` [v3 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
2024-10-10 14:52         ` Stephen Hemminger
2024-10-10 14:53         ` Stephen Hemminger
2024-10-10 14:53       ` [v3 1/2] app/proc-info: add memory heap dump Stephen Hemminger
2024-07-30 11:03   ` [v2 2/2] eal: add total memory size in memory dump APIs Gagandeep Singh
2024-10-10  0:25     ` Stephen Hemminger
2024-10-11 13:15   ` [v2 0/2] proc-info memory dump enhancement David Marchand

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).