Commit b9c54f56 authored by Konstantin Khlebnikov's avatar Konstantin Khlebnikov Committed by Jens Axboe

block: account merge of two requests

Also rename blk_account_io_merge() into blk_account_io_merge_request() to
distinguish it from merging request and bio.

[hch: rebased]
Signed-off-by: default avatarKonstantin Khlebnikov <khlebnikov@yandex-team.ru>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 58d4f14f
...@@ -669,18 +669,16 @@ void blk_rq_set_mixed_merge(struct request *rq) ...@@ -669,18 +669,16 @@ void blk_rq_set_mixed_merge(struct request *rq)
rq->rq_flags |= RQF_MIXED_MERGE; rq->rq_flags |= RQF_MIXED_MERGE;
} }
static void blk_account_io_merge(struct request *req) static void blk_account_io_merge_request(struct request *req)
{ {
if (blk_do_io_stat(req)) { if (blk_do_io_stat(req)) {
struct hd_struct *part;
part_stat_lock(); part_stat_lock();
part = req->part; part_stat_inc(req->part, merges[op_stat_group(req_op(req))]);
hd_struct_put(req->part);
hd_struct_put(part);
part_stat_unlock(); part_stat_unlock();
} }
} }
/* /*
* Two cases of handling DISCARD merge: * Two cases of handling DISCARD merge:
* If max_discard_segments > 1, the driver takes every bio * If max_discard_segments > 1, the driver takes every bio
...@@ -792,7 +790,7 @@ static struct request *attempt_merge(struct request_queue *q, ...@@ -792,7 +790,7 @@ static struct request *attempt_merge(struct request_queue *q,
/* /*
* 'next' is going away, so update stats accordingly * 'next' is going away, so update stats accordingly
*/ */
blk_account_io_merge(next); blk_account_io_merge_request(next);
/* /*
* ownership of bio passed from next to req, return 'next' for * ownership of bio passed from next to req, return 'next' for
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment