diff options
author | Bart Van Assche <bvanassche@acm.org> | 2023-05-17 19:42:25 +0200 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2023-05-19 03:47:49 +0200 |
commit | 3b463cbea908a9c8d4b9eda09765070506864cbe (patch) | |
tree | 5043426831e7c3b68cad34d3391b7ccd4a362ce5 /block/mq-deadline.c | |
parent | block: mq-deadline: Clean up deadline_check_fifo() (diff) | |
download | linux-3b463cbea908a9c8d4b9eda09765070506864cbe.tar.xz linux-3b463cbea908a9c8d4b9eda09765070506864cbe.zip |
block: mq-deadline: Simplify deadline_skip_seq_writes()
Make the deadline_skip_seq_writes() code shorter without changing its
functionality.
Reviewed-by: Damien Le Moal <dlemoal@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Cc: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Link: https://lore.kernel.org/r/20230517174230.897144-8-bvanassche@acm.org
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/mq-deadline.c')
-rw-r--r-- | block/mq-deadline.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/block/mq-deadline.c b/block/mq-deadline.c index cea91ba4a6ea..56782ee93522 100644 --- a/block/mq-deadline.c +++ b/block/mq-deadline.c @@ -304,14 +304,11 @@ static struct request *deadline_skip_seq_writes(struct deadline_data *dd, struct request *rq) { sector_t pos = blk_rq_pos(rq); - sector_t skipped_sectors = 0; - while (rq) { - if (blk_rq_pos(rq) != pos + skipped_sectors) - break; - skipped_sectors += blk_rq_sectors(rq); + do { + pos += blk_rq_sectors(rq); rq = deadline_latter_request(rq); - } + } while (rq && blk_rq_pos(rq) == pos); return rq; } |