On 6/16/22 3:14 PM, Bart Van Assche wrote: > On 6/16/22 13:50, Jens Axboe wrote: >> On 6/13/22 10:32 AM, Bart Van Assche wrote: >>> BFQ uses io_start_time_ns. That member variable is only set if I/O >>> statistics are enabled. Hence this patch that enables I/O statistics >>> at the time BFQ is associated with a request queue. >>> >>> Compile-tested only. >> >> Have you runtime tested it now? > > This patch has been tested lightly: I ran blktests in a VM against a > kernel that includes this patch. I'm not a BFQ expert so I was hoping > for feedback from Paolo. Sounds good - and Jan reviewed it, and I consider him the defactor BFQ maintainer anyway these days :-) -- Jens Axboe
