Save one spinlock by using waitqueue_active. We hold the bufio lock at
this place, so no one can add entries to the waitqueue at this point.

Signed-off-by: Mikulas Patocka <mpato...@redhat.com>

---
 drivers/md/dm-bufio.c |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Index: linux-2.6/drivers/md/dm-bufio.c
===================================================================
--- linux-2.6.orig/drivers/md/dm-bufio.c
+++ linux-2.6/drivers/md/dm-bufio.c
@@ -1667,7 +1667,12 @@ static void __free_buffer_wake(struct dm
                c->need_reserved_buffers--;
        }
 
-       wake_up(&c->free_buffer_wait);
+       /*
+        * We hold the bufio lock here, so no one can add entries to the
+        * wait queue anyway.
+        */
+       if (unlikely(waitqueue_active(&c->free_buffer_wait)))
+               wake_up(&c->free_buffer_wait);
 }
 
 static enum evict_result cleaned(struct dm_buffer *b, void *context)
--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel

Reply via email to