debugobjects: Delete a piece of redundant code
authorZhen Lei <thunder.leizhen@huawei.com>
Mon, 7 Oct 2024 16:49:52 +0000 (18:49 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Tue, 15 Oct 2024 15:30:30 +0000 (17:30 +0200)
The statically allocated objects are all located in obj_static_pool[],
the whole memory of obj_static_pool[] will be reclaimed later. Therefore,
there is no need to split the remaining statically nodes in list obj_pool
into isolated ones, no one will use them anymore. Just write
INIT_HLIST_HEAD(&obj_pool) is enough. Since hlist_move_list() directly
discards the old list, even this can be omitted.

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/all/20240911083521.2257-2-thunder.leizhen@huawei.com
Link: https://lore.kernel.org/all/20241007164913.009849239@linutronix.de
lib/debugobjects.c

index 5ce473a..df48acc 100644 (file)
@@ -1325,10 +1325,10 @@ static int __init debug_objects_replace_static_objects(void)
         * active object references.
         */
 
-       /* Remove the statically allocated objects from the pool */
-       hlist_for_each_entry_safe(obj, tmp, &obj_pool, node)
-               hlist_del(&obj->node);
-       /* Move the allocated objects to the pool */
+       /*
+        * Replace the statically allocated objects list with the allocated
+        * objects list.
+        */
        hlist_move_list(&objects, &obj_pool);
 
        /* Replace the active object references */