On 3/27/23 4:12 PM, Christoph Hellwig wrote: > On Mon, Mar 27, 2023 at 12:34:26AM -0700, Chaitanya Kulkarni wrote: >> There is only one caller for __blk_account_io_start(), the function >> is small enough to fit in its caller blk_account_io_start(). >> >> Remove the function and opencode in the its caller >> blk_account_io_start(). > > Having the account slow path in a separate function actually is nice, > same for the next patch. Then it should be made explicitly out-of-line, the compiler would've inlined that anyway. -- Jens Axboe