Consolidate the two calls to blk_rq_get_max_sectors into one using a local variable. Signed-off-by: Christoph Hellwig <hch@xxxxxx> --- block/blk-merge.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/block/blk-merge.c b/block/blk-merge.c index 14ce19607cd8a..8ed50952e93ad 100644 --- a/block/blk-merge.c +++ b/block/blk-merge.c @@ -595,17 +595,18 @@ static inline unsigned int blk_rq_get_max_sectors(struct request *rq, sector_t offset) { struct request_queue *q = rq->q; + unsigned int max_sectors; if (blk_rq_is_passthrough(rq)) return q->limits.max_hw_sectors; + max_sectors = blk_queue_get_max_sectors(q, req_op(rq)); if (!q->limits.chunk_sectors || req_op(rq) == REQ_OP_DISCARD || req_op(rq) == REQ_OP_SECURE_ERASE) - return blk_queue_get_max_sectors(q, req_op(rq)); + return max_sectors; - return min(blk_max_size_offset(q, offset, 0), - blk_queue_get_max_sectors(q, req_op(rq))); + return min(max_sectors, blk_max_size_offset(q, offset, 0)); } static inline int ll_new_hw_segment(struct request *req, struct bio *bio, -- 2.30.2 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/dm-devel