]> pilppa.com Git - linux-2.6-omap-h63xx.git/commit
block: internal dequeue shouldn't start timer
authorTejun Heo <tj@kernel.org>
Wed, 3 Dec 2008 11:41:26 +0000 (12:41 +0100)
committerJens Axboe <jens.axboe@oracle.com>
Wed, 3 Dec 2008 11:41:26 +0000 (12:41 +0100)
commit53a08807c01989c6847bb135d8d43f61c5dfdda5
treee2297de901dc82a0336e6af28b594864792c585d
parentbf91db18ac2852a3ff39fe25ff56c5557c0fff78
block: internal dequeue shouldn't start timer

blkdev_dequeue_request() and elv_dequeue_request() are equivalent and
both start the timeout timer.  Barrier code dequeues the original
barrier request but doesn't passes the request itself to lower level
driver, only broken down proxy requests; however, as the original
barrier code goes through the same dequeue path and timeout timer is
started on it.  If barrier sequence takes long enough, this timer
expires but the low level driver has no idea about this request and
oops follows.

Timeout timer shouldn't have been started on the original barrier
request as it never goes through actual IO.  This patch unexports
elv_dequeue_request(), which has no external user anyway, and makes it
operate on elevator proper w/o adding the timer and make
blkdev_dequeue_request() call elv_dequeue_request() and add timer.
Internal users which don't pass the request to driver - barrier code
and end_that_request_last() - are converted to use
elv_dequeue_request().

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Mike Anderson <andmike@linux.vnet.ibm.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
block/blk-barrier.c
block/blk-core.c
block/elevator.c
include/linux/blkdev.h