On Mon, 10 Jul 2023 18:55:16 +0800, chengming.zhou@xxxxxxxxx wrote: > The iocost rely on rq start_time_ns and alloc_time_ns to tell saturation > state of the block device. Most of the time request is allocated after > rq_qos_throttle() and its alloc_time_ns or start_time_ns won't be affected. > > But for plug batched allocation introduced by the commit 47c122e35d7e > ("block: pre-allocate requests if plug is started and is a batch"), we can > rq_qos_throttle() after the allocation of the request. This is what the > blk_mq_get_cached_request() does. > > [...] Applied, thanks! [1/1] blk-mq: fix start_time_ns and alloc_time_ns for pre-allocated rq commit: 5c17f45e91f5035c1b317e93b3dfb01088ac2902 Best regards, -- Jens Axboe