[PATCH] as-iosched: fix write batch start point

2007-12-05 Thread Aaron Carroll
New write batches currently start from where the last one completed. We have no idea where the head is after switching batches, so this makes little sense. Instead, start the next batch from the request with the earliest deadline in the hope that we avoid a deadline expiry later on.

Re: [PATCH] as-iosched: fix write batch start point

2007-12-05 Thread Jens Axboe
On Wed, Dec 05 2007, Nick Piggin wrote: > On Wed, Dec 05, 2007 at 09:07:07PM +1100, Aaron Carroll wrote: > > New write batches currently start from where the last one completed. > > We have no idea where the head is after switching batches, so this > > makes little sense. Instead, start the next

Re: [PATCH] as-iosched: fix write batch start point

2007-12-05 Thread Nick Piggin
On Wed, Dec 05, 2007 at 09:07:07PM +1100, Aaron Carroll wrote: > New write batches currently start from where the last one completed. > We have no idea where the head is after switching batches, so this > makes little sense. Instead, start the next batch from the request > with the earliest

Re: [PATCH] as-iosched: fix write batch start point

2007-12-05 Thread Nick Piggin
On Wed, Dec 05, 2007 at 09:07:07PM +1100, Aaron Carroll wrote: New write batches currently start from where the last one completed. We have no idea where the head is after switching batches, so this makes little sense. Instead, start the next batch from the request with the earliest deadline

[PATCH] as-iosched: fix write batch start point

2007-12-05 Thread Aaron Carroll
New write batches currently start from where the last one completed. We have no idea where the head is after switching batches, so this makes little sense. Instead, start the next batch from the request with the earliest deadline in the hope that we avoid a deadline expiry later on.

Re: [PATCH] as-iosched: fix write batch start point

2007-12-05 Thread Jens Axboe
On Wed, Dec 05 2007, Nick Piggin wrote: On Wed, Dec 05, 2007 at 09:07:07PM +1100, Aaron Carroll wrote: New write batches currently start from where the last one completed. We have no idea where the head is after switching batches, so this makes little sense. Instead, start the next batch