From: Keith Busch <kbusch@kernel.org> mainline inclusion from mainline-v7.2-rc1 commit b7f40ab50190e2500c3c297d15e00040dca47feb category: feature bugzilla: https://atomgit.com/openeuler/kernel/issues/10302 Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?i... -------------------------------- A user can enable io accounting for passthrough requests, so export the helper that checks if the request should be tracked. This will enable stacking drivers to to report iostats for passthrough workloads. Since the stacking request_queue may not be the one providing the request, the API has to add a parameter for the caller to specify which one to check. Reviewed-by: Nilay Shroff <nilay@linux.ibm.com> Reviewed-by: Nitesh Shetty <nj.shetty@samsung.com> Signed-off-by: Keith Busch <kbusch@kernel.org> Reviewed-by: Christoph Hellwig <hch@lst.de> Link: https://patch.msgid.link/20260528010041.1533124-2-kbusch@meta.com Signed-off-by: Jens Axboe <axboe@kernel.dk> Conflicts: block/blk.h block/blk-mq.c include/linux/blk-mq.h [Mainline moves blk_rq_passthrough_stats() from block/blk-mq.c to include/linux/blk-mq.h. In this backport it starts in block/blk.h, so removes it from block/blk.h and adds it to include/linux/blk-mq.h with the new queue argument. The caller is blk_account_io_start() in block/blk-mq.c, which passes req->q.] Signed-off-by: Zizhi Wo <wozizhi@huawei.com> --- block/blk-mq.c | 3 ++- block/blk.h | 30 ------------------------------ include/linux/blk-mq.h | 41 +++++++++++++++++++++++++++++++++++++++++ 3 files changed, 43 insertions(+), 31 deletions(-) diff --git a/block/blk-mq.c b/block/blk-mq.c index 1ae58d002b62..30536befeaf3 100644 --- a/block/blk-mq.c +++ b/block/blk-mq.c @@ -1044,11 +1044,12 @@ static inline void blk_account_io_start(struct request *req) /* * RQF_IO_STAT is preset at allocation; clear it for passthrough * requests that cannot be accounted. */ - if (blk_rq_is_passthrough(req) && !blk_rq_passthrough_stats(req)) + if (blk_rq_is_passthrough(req) && + !blk_rq_passthrough_stats(req, req->q)) req->rq_flags &= ~RQF_IO_STAT; if (blk_do_io_stat(req)) { /* * All non-passthrough requests are created from a bio with one diff --git a/block/blk.h b/block/blk.h index 732ca24e16ea..041539a2289d 100644 --- a/block/blk.h +++ b/block/blk.h @@ -358,40 +358,10 @@ bool blk_rq_merge_ok(struct request *rq, struct bio *bio); enum elv_merge blk_try_merge(struct request *rq, struct bio *bio); void blk_set_default_limits(struct queue_limits *lim); int blk_dev_init(void); -static inline bool blk_rq_passthrough_stats(struct request *req) -{ - struct bio *bio = req->bio; - - if (!blk_queue_passthrough_stat(req->q)) - return false; - - /* Requests without a bio do not transfer data. */ - if (!bio) - return false; - - /* - * Stats are accumulated in the bdev, so must have one attached to a - * bio to track stats. Most drivers do not set the bdev for passthrough - * requests, but nvme is one that will set it. - */ - if (!bio->bi_bdev) - return false; - - /* - * We don't know what a passthrough command does, but we know the - * payload size and data direction. Ensuring the size is aligned to the - * block size filters out most commands with payloads that don't - * represent sector access. - */ - if (blk_rq_bytes(req) & (bdev_logical_block_size(bio->bi_bdev) - 1)) - return false; - return true; -} - /* * Contribute to IO statistics IFF: * * a) it's attached to a gendisk, and * b) the queue had IO stats enabled when this request was started diff --git a/include/linux/blk-mq.h b/include/linux/blk-mq.h index 8ab506d96604..69abd003ad38 100644 --- a/include/linux/blk-mq.h +++ b/include/linux/blk-mq.h @@ -1324,6 +1324,47 @@ static inline bool blk_req_can_dispatch_to_zone(struct request *rq) { return true; } #endif /* CONFIG_BLK_DEV_ZONED */ +/** + * blk_rq_passthrough_stats - check if this request should account stats + * @rq: request to check + * @q: the queue accumulating the stats + * + * Note, @q does not necessarily need to be the request_queue that provides + * @rq. + * + * Return: true if stats should be accounted. + */ +static inline bool blk_rq_passthrough_stats(struct request *rq, + struct request_queue *q) +{ + struct bio *bio = rq->bio; + + if (!blk_queue_passthrough_stat(q)) + return false; + + /* Requests without a bio do not transfer data. */ + if (!bio) + return false; + + /* + * Stats are accumulated in the bdev, so must have one attached to a + * bio to track stats. Most drivers do not set the bdev for passthrough + * requests, but nvme is one that will set it. + */ + if (!bio->bi_bdev) + return false; + + /* + * We don't know what a passthrough command does, but we know the + * payload size and data direction. Ensuring the size is aligned to the + * block size filters out most commands with payloads that don't + * represent sector access. + */ + if (blk_rq_bytes(rq) & (bdev_logical_block_size(bio->bi_bdev) - 1)) + return false; + return true; +} + #endif /* BLK_MQ_H */ -- 2.52.0