Btrfs: keep processing bios for a given bdev if our proc is batching
authorChris Mason <chris.mason@oracle.com>
Fri, 3 Apr 2009 14:27:10 +0000 (10:27 -0400)
committerChris Mason <chris.mason@oracle.com>
Fri, 3 Apr 2009 14:27:10 +0000 (10:27 -0400)
commitb765ead57da62cccf7fa21e00e6eed65e9df62b0
tree66541fd018482a8d0db0021c3a3f3e8611ddb6fe
parentd57e62b89796f751c9422801cbcd407a9f8dcdc4
Btrfs: keep processing bios for a given bdev if our proc is batching

Btrfs uses async helper threads to submit write bios so the checksumming
helper threads don't block on the disk.

The submit bio threads may process bios for more than one block device,
so when they find one device congested they try to move on to other
devices instead of blocking in get_request_wait for one device.

This does a pretty good job of keeping multiple devices busy, but the
congested flag has a number of problems.  A congested device may still
give you a request, and other procs that aren't backing off the congested
device may starve you out.

This commit uses the io_context stored in current to decide if our process
has been made a batching process by the block layer.  If so, it keeps
sending IO down for at least one batch.  This helps make sure we do
a good amount of work each time we visit a bdev, and avoids large IO
stalls in multi-device workloads.

It's also very ugly.  A better solution is in the works with Jens Axboe.

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