mm/slub: remove full list manipulation for non-debug slab
authorChengming Zhou <zhouchengming@bytedance.com>
Tue, 23 Jan 2024 09:33:30 +0000 (09:33 +0000)
committerVlastimil Babka <vbabka@suse.cz>
Tue, 23 Jan 2024 10:44:06 +0000 (11:44 +0100)
Since debug slab is processed by free_to_partial_list(), and only debug
slab which has SLAB_STORE_USER flag would care about the full list, we
can remove these unrelated full list manipulations from __slab_free().

Acked-by: Christoph Lameter (Ampere) <cl@linux.com>
Reviewed-by: Vlastimil Babka <vbabka@suse.cz>
Signed-off-by: Chengming Zhou <zhouchengming@bytedance.com>
Signed-off-by: Vlastimil Babka <vbabka@suse.cz>
mm/slub.c

index fda402b..5c6fbee 100644 (file)
--- a/mm/slub.c
+++ b/mm/slub.c
@@ -4188,7 +4188,6 @@ static void __slab_free(struct kmem_cache *s, struct slab *slab,
         * then add it.
         */
        if (!kmem_cache_has_cpu_partial(s) && unlikely(!prior)) {
-               remove_full(s, n, slab);
                add_partial(n, slab, DEACTIVATE_TO_TAIL);
                stat(s, FREE_ADD_PARTIAL);
        }
@@ -4202,9 +4201,6 @@ slab_empty:
                 */
                remove_partial(n, slab);
                stat(s, FREE_REMOVE_PARTIAL);
-       } else {
-               /* Slab must be on the full list */
-               remove_full(s, n, slab);
        }
 
        spin_unlock_irqrestore(&n->list_lock, flags);