diff mbox series

mm: add nr_free_highatomic in show_free_areas

Message ID d92eeff74f7a4578a14ac777cfe3603a@honor.com (mailing list archive)
State New
Headers show
Series mm: add nr_free_highatomic in show_free_areas | expand

Commit Message

gaoxu April 12, 2025, 9:27 a.m. UTC
The commit c928807f6f6b6("mm/page_alloc: keep track of free highatomic")
adds a new variable nr_free_highatomic, which is useful for analyzing low
mem issues. add nr_free_highatomic in show_free_areas.

Signed-off-by: gao xu <gaoxu2@honor.com>
---
 mm/show_mem.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

David Hildenbrand April 13, 2025, 8:47 p.m. UTC | #1
On 12.04.25 11:27, gaoxu wrote:
> The commit c928807f6f6b6("mm/page_alloc: keep track of free highatomic")
> adds a new variable nr_free_highatomic, which is useful for analyzing low
> mem issues. add nr_free_highatomic in show_free_areas.
> 

Sounds helpful

Acked-by: David Hildenbrand <david@redhat.com>
Barry Song April 13, 2025, 9:52 p.m. UTC | #2
On Sat, Apr 12, 2025 at 9:27 PM gaoxu <gaoxu2@honor.com> wrote:
>
> The commit c928807f6f6b6("mm/page_alloc: keep track of free highatomic")
> adds a new variable nr_free_highatomic, which is useful for analyzing low
> mem issues. add nr_free_highatomic in show_free_areas.
>
> Signed-off-by: gao xu <gaoxu2@honor.com>

Reviewed-by: Barry Song <baohua@kernel.org>

> ---
>  mm/show_mem.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/mm/show_mem.c b/mm/show_mem.c
> index ad373b4b6..03e8d968f 100644
> --- a/mm/show_mem.c
> +++ b/mm/show_mem.c
> @@ -305,6 +305,7 @@ static void show_free_areas(unsigned int filter, nodemask_t *nodemask, int max_z
>                         " low:%lukB"
>                         " high:%lukB"
>                         " reserved_highatomic:%luKB"
> +                       " free_highatomic:%luKB"
>                         " active_anon:%lukB"
>                         " inactive_anon:%lukB"
>                         " active_file:%lukB"
> @@ -326,6 +327,7 @@ static void show_free_areas(unsigned int filter, nodemask_t *nodemask, int max_z
>                         K(low_wmark_pages(zone)),
>                         K(high_wmark_pages(zone)),
>                         K(zone->nr_reserved_highatomic),
> +                       K(zone->nr_free_highatomic),
>                         K(zone_page_state(zone, NR_ZONE_ACTIVE_ANON)),
>                         K(zone_page_state(zone, NR_ZONE_INACTIVE_ANON)),
>                         K(zone_page_state(zone, NR_ZONE_ACTIVE_FILE)),
> --
> 2.17.1
Anshuman Khandual April 15, 2025, 6:40 a.m. UTC | #3
On 4/12/25 14:57, gaoxu wrote:
> The commit c928807f6f6b6("mm/page_alloc: keep track of free highatomic")
> adds a new variable nr_free_highatomic, which is useful for analyzing low
> mem issues. add nr_free_highatomic in show_free_areas.
> 
> Signed-off-by: gao xu <gaoxu2@honor.com>
> ---
>  mm/show_mem.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/mm/show_mem.c b/mm/show_mem.c
> index ad373b4b6..03e8d968f 100644
> --- a/mm/show_mem.c
> +++ b/mm/show_mem.c
> @@ -305,6 +305,7 @@ static void show_free_areas(unsigned int filter, nodemask_t *nodemask, int max_z
>  			" low:%lukB"
>  			" high:%lukB"
>  			" reserved_highatomic:%luKB"
> +			" free_highatomic:%luKB"
>  			" active_anon:%lukB"
>  			" inactive_anon:%lukB"
>  			" active_file:%lukB"
> @@ -326,6 +327,7 @@ static void show_free_areas(unsigned int filter, nodemask_t *nodemask, int max_z
>  			K(low_wmark_pages(zone)),
>  			K(high_wmark_pages(zone)),
>  			K(zone->nr_reserved_highatomic),
> +			K(zone->nr_free_highatomic),
>  			K(zone_page_state(zone, NR_ZONE_ACTIVE_ANON)),
>  			K(zone_page_state(zone, NR_ZONE_INACTIVE_ANON)),
>  			K(zone_page_state(zone, NR_ZONE_ACTIVE_FILE)),

Reviewed-by: Anshuman Khandual <anshuman.khandual@arm.com>
diff mbox series

Patch

diff --git a/mm/show_mem.c b/mm/show_mem.c
index ad373b4b6..03e8d968f 100644
--- a/mm/show_mem.c
+++ b/mm/show_mem.c
@@ -305,6 +305,7 @@  static void show_free_areas(unsigned int filter, nodemask_t *nodemask, int max_z
 			" low:%lukB"
 			" high:%lukB"
 			" reserved_highatomic:%luKB"
+			" free_highatomic:%luKB"
 			" active_anon:%lukB"
 			" inactive_anon:%lukB"
 			" active_file:%lukB"
@@ -326,6 +327,7 @@  static void show_free_areas(unsigned int filter, nodemask_t *nodemask, int max_z
 			K(low_wmark_pages(zone)),
 			K(high_wmark_pages(zone)),
 			K(zone->nr_reserved_highatomic),
+			K(zone->nr_free_highatomic),
 			K(zone_page_state(zone, NR_ZONE_ACTIVE_ANON)),
 			K(zone_page_state(zone, NR_ZONE_INACTIVE_ANON)),
 			K(zone_page_state(zone, NR_ZONE_ACTIVE_FILE)),