linux.git/block/blk-timeout.c, branch v2.6.29

block: fix deadlock in blk_abort_queue() for drivers that readd to timeout list

2009-02-18T09:34:16+00:00

blk_abort_queue() iterates the timeout list and aborts each request on the
list, but if the driver error handling readds a request to the timeout list
during this processing, we could be looping forever. Fix this by splicing
current entries to a local list and run over that list instead.

Signed-off-by: Jens Axboe

block: leave the request timeout timer running even on an empty list

2008-12-29T07:28:42+00:00

For sync IO, we'll often do them serialized. This means we'll be touching
the queue timer for every IO, as opposed to only occasionally like we
do for queued IO. Instead of deleting the timer when the last request
is removed, just let continue running. If a new request comes up soon
we then don't have to readd the timer again. If no new requests arrive,
the timer will expire without side effect later.

This improves high iops sync IO by ~1%.

Signed-off-by: Jens Axboe

block: add comment in blk_rq_timed_out() about why next can not be 0

2008-12-29T07:28:42+00:00

Signed-off-by: Jens Axboe

block: optimizations in blk_rq_timed_out_timer()

2008-12-29T07:28:42+00:00

Now the rq->deadline can't be zero if the request is in the
timeout_list, so there is no need to have next_set. There is no need to
access a request's deadline field if blk_rq_timed_out is called on it.

Signed-off-by: Malahal Naineni 
Signed-off-by: Jens Axboe

Block: use round_jiffies_up()

2008-11-06T07:42:49+00:00

This patch (as1159b) changes the timeout routines in the block core to
use round_jiffies_up().  There's no point in rounding the timer
deadline down, since if it expires too early we will have to restart
it.

The patch also removes some unnecessary tests when a request is
removed from the queue's timer list.

Signed-off-by: Alan Stern 
Signed-off-by: Jens Axboe

block: use rq complete marking in blk_abort_request()

2008-10-09T06:56:17+00:00

We cannot abort a request if we raced with the timeout handler already,
or with the IO completion. So make blk_abort_request() mark the request
as complete, and only continue if we succeeded.

Found and suggested by Mike Anderson 

Signed-off-by: Jens Axboe

block: add fault injection mechanism for faking request timeouts

2008-10-09T06:56:17+00:00

Only works for the generic request timer handling. Allows one to
sporadically ignore request completions, thus exercising the timeout
handling.

Signed-off-by: Jens Axboe

block: Add interface to abort queued requests

2008-10-09T06:56:13+00:00

Signed-off-by: Mike Anderson 
Signed-off-by: Jens Axboe

block: unify request timeout handling

2008-10-09T06:56:13+00:00

Right now SCSI and others do their own command timeout handling.
Move those bits to the block layer.

Instead of having a timer per command, we try to be a bit more clever
and simply have one per-queue. This avoids the overhead of having to
tear down and setup a timer for each command, so it will result in a lot
less timer fiddling.

Signed-off-by: Mike Anderson 
Signed-off-by: Jens Axboe