fsnotify: Exchange list heads instead of moving elements
authorAndreas Gruenbacher <agruen@suse.de>
Wed, 28 Jul 2010 14:18:38 +0000 (10:18 -0400)
committerEric Paris <eparis@redhat.com>
Wed, 28 Jul 2010 14:18:52 +0000 (10:18 -0400)
Instead of moving list elements from destroy_list to &private_destroy_list,
exchange the list heads.

Signed-off-by: Andreas Gruenbacher <agruen@suse.de>
Signed-off-by: Eric Paris <eparis@redhat.com>

fs/notify/mark.c

index 41f3990..236f29b 100644 (file)
@@ -349,10 +349,8 @@ static int fsnotify_mark_destroy(void *ignored)
 
        for (;;) {
                spin_lock(&destroy_lock);
-               list_for_each_entry_safe(mark, next, &destroy_list, destroy_list) {
-                       list_del(&mark->destroy_list);
-                       list_add(&mark->destroy_list, &private_destroy_list);
-               }
+               /* exchange the list head */
+               list_replace_init(&destroy_list, &private_destroy_list);
                spin_unlock(&destroy_lock);
 
                synchronize_srcu(&fsnotify_mark_srcu);