Re: [PATCH UPDATED block/for-linus] block: blk-throttle should bedrained regardless of q->elevator

From: Jens Axboe
Date: Wed Feb 15 2012 - 10:58:18 EST


On 2012-02-14 02:14, Tejun Heo wrote:
> Currently, blk_cleanup_queue() doesn't call elv_drain_elevator() if
> q->elevator doesn't exist; however, bio based drivers don't have
> elevator initialized but can still use blk-throttle. This patch moves
> q->elevator test inside blk_drain_queue() such that only
> elv_drain_elevator() is skipped if !q->elevator.
>
> -v2: loop can have registered queue which has NULL request_fn. Make
> sure we don't call into __blk_run_queue() in such cases.
>
> Signed-off-by: Tejun Heo <tj@xxxxxxxxxx>
> Reported-by: Vivek Goyal <vgoyal@xxxxxxxxxx>
> Acked-by: Vivek Goyal <vgoyal@xxxxxxxxxx>
> ---
> Added check against NULL q->request_fn for loop driver. Thanks.

Applied, thanks.

--
Jens Axboe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/