[PATCH v3 13/14] blk-mq: use switch/case to improve readability in blk_mq_try_issue_list_directly

From: Kemeng Shi
Date: Wed Jan 11 2023 - 00:04:33 EST


Use switch/case handle error as other function do to improve
readability in blk_mq_try_issue_list_directly.

Signed-off-by: Kemeng Shi <shikemeng@xxxxxxxxxxxxxxx>
---
block/blk-mq.c | 22 +++++++++++++---------
1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/block/blk-mq.c b/block/blk-mq.c
index 1b66a5169be2..baa65a15abb5 100644
--- a/block/blk-mq.c
+++ b/block/blk-mq.c
@@ -2806,18 +2806,22 @@ void blk_mq_try_issue_list_directly(struct blk_mq_hw_ctx *hctx,

list_del_init(&rq->queuelist);
ret = blk_mq_request_issue_directly(rq, list_empty(list));
- if (ret != BLK_STS_OK) {
- if (ret == BLK_STS_RESOURCE ||
- ret == BLK_STS_DEV_RESOURCE) {
- blk_mq_request_bypass_insert(rq, false,
- list_empty(list));
- break;
- }
- blk_mq_end_request(rq, ret);
- } else
+ switch (ret) {
+ case BLK_STS_OK:
queued++;
+ break;
+ case BLK_STS_RESOURCE:
+ case BLK_STS_DEV_RESOURCE:
+ blk_mq_request_bypass_insert(rq, false,
+ list_empty(list));
+ goto out;
+ default:
+ blk_mq_end_request(rq, ret);
+ break;
+ }
}

+out:
if (ret != BLK_STS_OK)
blk_mq_commit_rqs(hctx, queued);
}
--
2.30.0