[PATCH] tools/mm/slabinfo: fix total_objects attribute name

[PATCH] tools/mm/slabinfo: fix total_objects attribute name

[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

Re: [PATCH] tools/mm/slabinfo: fix total_objects attribute name

[SeongJae Park]

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

[...]

Re: [PATCH] tools/mm/slabinfo: fix total_objects attribute name

[Yichong Chen]

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

Thanks for the review.

Re: [PATCH] tools/mm/slabinfo: fix total_objects attribute name

[Vishal Moola]

On Fri, Jun 12, 2026 at 05:29:12PM -0700, SeongJae Park wrote:
> 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@ ?

This variable isn't even used... so we don't need a cc stable. That's
probably why no one spotted it until now too ;)

We should probably be printing it out in another column; I'm not sure
why the initial patch didn't do so.

> > Signed-off-by: Yichong Chen <chenyichong@uniontech.com>
> 
> Reviewed-by: SeongJae Park <sj@kernel.org>
> 
> 
> Thanks,
> SJ
> 
> [...]