Btrfs: run the backing dev more often in the submit_bio helper
authorChris Mason <chris.mason@oracle.com>
Wed, 10 Mar 2010 20:33:32 +0000 (15:33 -0500)
committerChris Mason <chris.mason@oracle.com>
Mon, 15 Mar 2010 15:00:09 +0000 (11:00 -0400)
The submit_bio helper thread can decide to loop back around to
service more bios.  This commit forces it to unplug first, which helps
reduce the latency seen by submitters.

Signed-off-by: Chris Mason <chris.mason@oracle.com>
fs/btrfs/volumes.c

index 41ecbb2347f2d3171f7645782ab4e5a39621c84d..ace2e8d7bbcd0293f2f0e289ee43894e64d6f085 100644 (file)
@@ -325,16 +325,6 @@ loop_lock:
                num_sync_run = 0;
                blk_run_backing_dev(bdi, NULL);
        }
-
-       cond_resched();
-       if (again)
-               goto loop;
-
-       spin_lock(&device->io_lock);
-       if (device->pending_bios.head || device->pending_sync_bios.head)
-               goto loop_lock;
-       spin_unlock(&device->io_lock);
-
        /*
         * IO has already been through a long path to get here.  Checksumming,
         * async helper threads, perhaps compression.  We've done a pretty
@@ -346,6 +336,16 @@ loop_lock:
         * cared about found its way down here.
         */
        blk_run_backing_dev(bdi, NULL);
+
+       cond_resched();
+       if (again)
+               goto loop;
+
+       spin_lock(&device->io_lock);
+       if (device->pending_bios.head || device->pending_sync_bios.head)
+               goto loop_lock;
+       spin_unlock(&device->io_lock);
+
 done:
        return 0;
 }