Revert "scsi: core: run queue if SCSI device queue isn't ready and queue is idle"
[ Upstream commit b4fd63f42647110c963d4bfcd526ac48f5a5faff ] This reverts commit7e70aa789d
. Now that we have the patches ("blk-mq: In blk_mq_dispatch_rq_list() "no budget" is a reason to kick") and ("blk-mq: Rerun dispatching in the case of budget contention") we should no longer need the fix in the SCSI code. Revert it, resolving conflicts with other patches that have touched this code. With this revert (and the two new patches) I can run the script that was in commit7e70aa789d
("scsi: core: run queue if SCSI device queue isn't ready and queue is idle") in a loop with no failure. If I do this revert without the two new patches I can easily get a failure. Signed-off-by: Douglas Anderson <dianders@chromium.org> Reviewed-by: Ming Lei <ming.lei@redhat.com> Acked-by: Martin K. Petersen <martin.petersen@oracle.com> Signed-off-by: Jens Axboe <axboe@kernel.dk> Stable-dep-of: c31e76bcc379 ("blk-mq: remove stale comment for blk_mq_sched_mark_restart_hctx") Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
2d3c3aa412
commit
578c8f09c0
@ -1645,12 +1645,7 @@ static bool scsi_mq_get_budget(struct blk_mq_hw_ctx *hctx)
|
|||||||
struct request_queue *q = hctx->queue;
|
struct request_queue *q = hctx->queue;
|
||||||
struct scsi_device *sdev = q->queuedata;
|
struct scsi_device *sdev = q->queuedata;
|
||||||
|
|
||||||
if (scsi_dev_queue_ready(q, sdev))
|
return scsi_dev_queue_ready(q, sdev);
|
||||||
return true;
|
|
||||||
|
|
||||||
if (atomic_read(&sdev->device_busy) == 0 && !scsi_device_blocked(sdev))
|
|
||||||
blk_mq_delay_run_hw_queue(hctx, SCSI_QUEUE_DELAY);
|
|
||||||
return false;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static blk_status_t scsi_queue_rq(struct blk_mq_hw_ctx *hctx,
|
static blk_status_t scsi_queue_rq(struct blk_mq_hw_ctx *hctx,
|
||||||
|
Loading…
Reference in New Issue
Block a user