diff options
| author | Suren Baghdasaryan <surenb@google.com> | 2025-09-15 13:09:18 -0700 |
|---|---|---|
| committer | Vlastimil Babka <vbabka@suse.cz> | 2025-09-16 14:32:51 +0200 |
| commit | f7381b9116407ba2a429977c80ff8df953ea9354 (patch) | |
| tree | bb59e093f29632f3dfbd741fe681fb16524bd9f6 | |
| parent | 4038016397da5c1cebb10e7c85a36d06123724a8 (diff) | |
slab: mark slab->obj_exts allocation failures unconditionally
alloc_slab_obj_exts() should mark failed obj_exts vector allocations
independent on whether the vector is being allocated for a new or an
existing slab. Current implementation skips doing this for existing
slabs. Fix this by marking failed allocations unconditionally.
Fixes: 09c46563ff6d ("codetag: debug: introduce OBJEXTS_ALLOC_FAIL to mark failed slab_ext allocations")
Reported-by: Shakeel Butt <shakeel.butt@linux.dev>
Closes: https://lore.kernel.org/all/avhakjldsgczmq356gkwmvfilyvf7o6temvcmtt5lqd4fhp5rk@47gp2ropyixg/
Signed-off-by: Suren Baghdasaryan <surenb@google.com>
Cc: stable@vger.kernel.org # v6.10+
Acked-by: Shakeel Butt <shakeel.butt@linux.dev>
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
| -rw-r--r-- | mm/slub.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/mm/slub.c b/mm/slub.c index 674c5036a18a..b74d65aa32c6 100644 --- a/mm/slub.c +++ b/mm/slub.c @@ -2039,8 +2039,7 @@ int alloc_slab_obj_exts(struct slab *slab, struct kmem_cache *s, slab_nid(slab)); if (!vec) { /* Mark vectors which failed to allocate */ - if (new_slab) - mark_failed_objexts_alloc(slab); + mark_failed_objexts_alloc(slab); return -ENOMEM; } |