diff options
| author |   Christoph Hellwig <hch@lst.de> | 2022-07-27 12:22:56 -0400 |
|---|---|---|
| committer |   Jens Axboe <axboe@kernel.dk> | 2022-08-02 17:22:54 -0600 |
| commit | 51d798cdb5c219000fe76a3ffbcaa846b689ba80 (patch) | |
| tree | f80c9ae63c33edb1fbb31c1e92d4c46253aea4c4 /block/blk-mq.c | |
| parent | block: change the blk_queue_split calling convention (diff) | |
| download | linux-dev-51d798cdb5c219000fe76a3ffbcaa846b689ba80.tar.xz linux-dev-51d798cdb5c219000fe76a3ffbcaa846b689ba80.zip | |
block: change the blk_queue_bounce calling convention
The double indirect bio leads to somewhat suboptimal code generation.
Instead return the (original or split) bio, and make sure the
request_queue arguments to the lower level helpers is passed after the
bio to avoid constant reshuffling of the argument passing registers.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
Reviewed-by: Johannes Thumshirn <johannes.thumshirn@wdc.com>
Link: https://lore.kernel.org/r/20220727162300.3089193-3-hch@lst.de
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/blk-mq.c')
| -rw-r--r-- | block/blk-mq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/blk-mq.c b/block/blk-mq.c index 790f55453f1b..7adba3eeba1c 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -2815,7 +2815,7 @@ void blk_mq_submit_bio(struct bio *bio) unsigned int nr_segs = 1; blk_status_t ret; - blk_queue_bounce(q, &bio); + bio = blk_queue_bounce(bio, q); if (bio_may_exceed_limits(bio, q)) bio = __bio_split_to_limits(bio, q, &nr_segs); |