Commit 9c6227e0 authored by Jan Kara's avatar Jan Kara Committed by Jens Axboe

block: Initialize bio priority earlier

Bio's IO priority needs to be initialized before we try to merge the bio
with other bios. Otherwise we could merge bios which would otherwise
receive different IO priorities leading to possible QoS issues.
Reviewed-by: default avatarDamien Le Moal <damien.lemoal@opensource.wdc.com>
Tested-by: default avatarDamien Le Moal <damien.lemoal@opensource.wdc.com>
Signed-off-by: default avatarJan Kara <jack@suse.cz>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20220623074840.5960-8-jack@suse.czSigned-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 82b74cac
...@@ -2814,6 +2814,8 @@ void blk_mq_submit_bio(struct bio *bio) ...@@ -2814,6 +2814,8 @@ void blk_mq_submit_bio(struct bio *bio)
if (!bio_integrity_prep(bio)) if (!bio_integrity_prep(bio))
return; return;
bio_set_ioprio(bio);
rq = blk_mq_get_cached_request(q, plug, &bio, nr_segs); rq = blk_mq_get_cached_request(q, plug, &bio, nr_segs);
if (!rq) { if (!rq) {
if (!bio) if (!bio)
...@@ -2825,8 +2827,6 @@ void blk_mq_submit_bio(struct bio *bio) ...@@ -2825,8 +2827,6 @@ void blk_mq_submit_bio(struct bio *bio)
trace_block_getrq(bio); trace_block_getrq(bio);
bio_set_ioprio(bio);
rq_qos_track(q, rq, bio); rq_qos_track(q, rq, bio);
blk_mq_bio_to_request(rq, bio, nr_segs); blk_mq_bio_to_request(rq, bio, nr_segs);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment