Commit 06965037 authored by Chaitanya Kulkarni's avatar Chaitanya Kulkarni Committed by Jens Axboe
Browse files

block: open code __blk_account_io_done() 



There is only one caller for __blk_account_io_done(), the function
is small enough to fit in its caller blk_account_io_done().

Remove the function and opencode in the its caller
blk_account_io_done().

Signed-off-by: default avatarChaitanya Kulkarni <kch@nvidia.com>
Link: https://lore.kernel.org/r/20230327073427.4403-2-kch@nvidia.com


Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent e165fb4d

block/blk-mq.c

+9 −13
Original line number Diff line number Diff line
@@ -953,8 +953,15 @@ bool blk_update_request(struct request *req, blk_status_t error, 
}

EXPORT_SYMBOL_GPL(blk_update_request);



static void __blk_account_io_done(struct request *req, u64 now)

static inline void blk_account_io_done(struct request *req, u64 now)

{

	/*

	 * Account IO completion.  flush_rq isn't accounted as a

	 * normal IO on queueing nor completion.  Accounting the

	 * containing request is enough.

	 */

	if (blk_do_io_stat(req) && req->part &&

	    !(req->rq_flags & RQF_FLUSH_SEQ)) {

		const int sgrp = op_stat_group(req_op(req));



		part_stat_lock();
@@ -963,17 +970,6 @@ static void __blk_account_io_done(struct request *req, u64 now) 
		part_stat_add(req->part, nsecs[sgrp], now - req->start_time_ns);

		part_stat_unlock();

	}



static inline void blk_account_io_done(struct request *req, u64 now)

{

	/*

	 * Account IO completion.  flush_rq isn't accounted as a

	 * normal IO on queueing nor completion.  Accounting the

	 * containing request is enough.

	 */

	if (blk_do_io_stat(req) && req->part &&

	    !(req->rq_flags & RQF_FLUSH_SEQ))

		__blk_account_io_done(req, now);

}



static inline void blk_account_io_start(struct request *req)