| From bc238c311242e29681de4d997921c017d8864d1a Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Wed, 20 Oct 2021 08:21:40 -0600 |
| Subject: block: remove inaccurate requeue check |
| |
| From: Jens Axboe <axboe@kernel.dk> |
| |
| [ Upstream commit 037057a5a979c7eeb2ee5d12cf4c24b805192c75 ] |
| |
| This check is meant to catch cases where a requeue is attempted on a |
| request that is still inserted. It's never really been useful to catch any |
| misuse, and now it's actively wrong. Outside of that, this should not be a |
| BUG_ON() to begin with. |
| |
| Remove the check as it's now causing active harm, as requeue off the plug |
| path will trigger it even though the request state is just fine. |
| |
| Reported-by: Yi Zhang <yi.zhang@redhat.com> |
| Link: https://lore.kernel.org/linux-block/CAHj4cs80zAUc2grnCZ015-2Rvd-=gXRfB_dFKy=RTm+wRo09HQ@mail.gmail.com/ |
| Signed-off-by: Jens Axboe <axboe@kernel.dk> |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| block/blk-mq.c | 1 - |
| 1 file changed, 1 deletion(-) |
| |
| diff --git a/block/blk-mq.c b/block/blk-mq.c |
| index 0674f53c60528..84798d09ca464 100644 |
| --- a/block/blk-mq.c |
| +++ b/block/blk-mq.c |
| @@ -733,7 +733,6 @@ void blk_mq_requeue_request(struct request *rq, bool kick_requeue_list) |
| /* this request will be re-inserted to io scheduler queue */ |
| blk_mq_sched_requeue_request(rq); |
| |
| - BUG_ON(!list_empty(&rq->queuelist)); |
| blk_mq_add_to_requeue_list(rq, true, kick_requeue_list); |
| } |
| EXPORT_SYMBOL(blk_mq_requeue_request); |
| -- |
| 2.33.0 |
| |