diff options
author | Hannes Reinecke <hare@suse.de> | 2014-10-01 08:32:31 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2014-10-01 10:17:42 -0400 |
commit | 4a0efdc933680d908de11712a774a2c9492c3d5a (patch) | |
tree | abeb8de55841295acad03143bfff7e09709ff33c /block | |
parent | c611529e7cd3465ec0eada0f44200e8420c38908 (diff) |
block: misplaced rq_complete tracepoint
The rq_complete tracepoint was never issued for empty requests,
causing the resulting blktrace information to never show any
completion for those request.
Signed-off-by: Hannes Reinecke <hare@suse.de>
Acked-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'block')
-rw-r--r-- | block/blk-core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/blk-core.c b/block/blk-core.c index e1c2775c7597..4aa9ccd02a50 100644 --- a/block/blk-core.c +++ b/block/blk-core.c | |||
@@ -2400,11 +2400,11 @@ bool blk_update_request(struct request *req, int error, unsigned int nr_bytes) | |||
2400 | { | 2400 | { |
2401 | int total_bytes; | 2401 | int total_bytes; |
2402 | 2402 | ||
2403 | trace_block_rq_complete(req->q, req, nr_bytes); | ||
2404 | |||
2403 | if (!req->bio) | 2405 | if (!req->bio) |
2404 | return false; | 2406 | return false; |
2405 | 2407 | ||
2406 | trace_block_rq_complete(req->q, req, nr_bytes); | ||
2407 | |||
2408 | /* | 2408 | /* |
2409 | * For fs requests, rq is just carrier of independent bio's | 2409 | * For fs requests, rq is just carrier of independent bio's |
2410 | * and each partial completion should be handled separately. | 2410 | * and each partial completion should be handled separately. |