Re: [PATCH] slub: let need_slab_obj_exts() return false if SLAB_NO_OBJ_EXT is set
From: Harry Yoo
Date: Fri Feb 06 2026 - 05:01:51 EST
On Fri, Feb 06, 2026 at 10:40:54AM +0100, Vlastimil Babka wrote:
> On 2/6/26 01:53, Harry Yoo wrote:
> > On Thu, Feb 05, 2026 at 08:07:23PM +0800, Hao Li wrote:
> >> SLAB_NO_OBJ_EXT is set for boot caches, but need_slab_obj_exts() doesn't
> >> check this flag. We should return false unconditionally when
> >> SLAB_NO_OBJ_EXT is set.
> >>
> >> Signed-off-by: Hao Li <hao.li@xxxxxxxxx>
> >> ---
> >
> > Looks reasonable to me,
> > Acked-by: Harry Yoo <harry.yoo@xxxxxxxxxx>
>
> Thanks, added to slab/for-7.0/obj_metadata
> I think it doesn't change anything at the moment?
...unless you enable memory allocation profiling :)
> But a correct thing to do.
--
Cheers,
Harry / Hyeonggon