Linux-mm Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tools/mm/slabinfo: fix total_objects attribute name
@ 2026-06-12  7:13 Yichong Chen
  2026-06-13  0:29 ` SeongJae Park
  0 siblings, 1 reply; 3+ messages in thread
From: Yichong Chen @ 2026-06-12  7:13 UTC (permalink / raw)
  To: akpm; +Cc: penberg, clameter, linux-mm, linux-kernel, Yichong Chen

SLUB exports the total_objects sysfs attribute, but slabinfo tries to read
objects_total. As a result, the lookup fails and the field remains zero.

Use the correct attribute name and rename the corresponding structure
member to match.

Fixes: 205ab99dd103 ("slub: Update statistics handling for variable order slabs")
Signed-off-by: Yichong Chen <chenyichong@uniontech.com>
---
 tools/mm/slabinfo.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/mm/slabinfo.c b/tools/mm/slabinfo.c
index 54c7265ab52d..f4077746d6df 100644
--- a/tools/mm/slabinfo.c
+++ b/tools/mm/slabinfo.c
@@ -33,7 +33,7 @@ struct slabinfo {
 	unsigned int hwcache_align, object_size, objs_per_slab;
 	unsigned int sanity_checks, slab_size, store_user, trace;
 	int order, poison, reclaim_account, red_zone;
-	unsigned long partial, objects, slabs, objects_partial, objects_total;
+	unsigned long partial, objects, slabs, objects_partial, total_objects;
 	unsigned long alloc_fastpath, alloc_slowpath;
 	unsigned long free_fastpath, free_slowpath;
 	unsigned long free_frozen, free_add_partial, free_remove_partial;
@@ -1263,7 +1263,7 @@ static void read_slab_dir(void)
 			slab->object_size = get_obj("object_size");
 			slab->objects = get_obj("objects");
 			slab->objects_partial = get_obj("objects_partial");
-			slab->objects_total = get_obj("objects_total");
+			slab->total_objects = get_obj("total_objects");
 			slab->objs_per_slab = get_obj("objs_per_slab");
 			slab->order = get_obj("order");
 			slab->partial = get_obj("partial");
-- 
2.51.0



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

* Re: [PATCH] tools/mm/slabinfo: fix total_objects attribute name
  2026-06-12  7:13 [PATCH] tools/mm/slabinfo: fix total_objects attribute name Yichong Chen
@ 2026-06-13  0:29 ` SeongJae Park
  2026-06-13  5:20   ` Yichong Chen
  0 siblings, 1 reply; 3+ messages in thread
From: SeongJae Park @ 2026-06-13  0:29 UTC (permalink / raw)
  To: Yichong Chen
  Cc: SeongJae Park, akpm, penberg, clameter, linux-mm, linux-kernel

On Fri, 12 Jun 2026 15:13:59 +0800 Yichong Chen <chenyichong@uniontech.com> wrote:

> SLUB exports the total_objects sysfs attribute, but slabinfo tries to read
> objects_total. As a result, the lookup fails and the field remains zero.
> 
> Use the correct attribute name and rename the corresponding structure
> member to match.

Good finding, thank you for fixing this!

> 
> Fixes: 205ab99dd103 ("slub: Update statistics handling for variable order slabs")

Should we add Cc: stable@ ?

> Signed-off-by: Yichong Chen <chenyichong@uniontech.com>

Reviewed-by: SeongJae Park <sj@kernel.org>


Thanks,
SJ

[...]


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

* Re: [PATCH] tools/mm/slabinfo: fix total_objects attribute name
  2026-06-13  0:29 ` SeongJae Park
@ 2026-06-13  5:20   ` Yichong Chen
  0 siblings, 0 replies; 3+ messages in thread
From: Yichong Chen @ 2026-06-13  5:20 UTC (permalink / raw)
  To: sj; +Cc: akpm, chenyichong, clameter, linux-kernel, linux-mm, penberg

Yes, please add Cc: stable@vger.kernel.org.

Thanks for the review.


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

end of thread, other threads:[~2026-06-13  5:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-12  7:13 [PATCH] tools/mm/slabinfo: fix total_objects attribute name Yichong Chen
2026-06-13  0:29 ` SeongJae Park
2026-06-13  5:20   ` Yichong Chen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox