diff options
author | Jianpeng Ma <majianpeng@gmail.com> | 2013-09-11 15:21:07 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2013-09-11 15:21:07 -0400 |
commit | 7aef2e780b13973ea60aed8c556107dabde6a495 (patch) | |
tree | 5265a24ffeaea32e2cefacf83962764d6d661a36 /block | |
parent | 6e4664525b1db28f8c4e1130957f70a94c19213e (diff) |
block: trace all devices plug operation
In func blk_queue_bio, if list of plug is empty,it will call
blk_trace_plug.
If process deal with a single device,it't ok.But if process deal with
multi devices,it only trace the first device.
Using request_count to judge, it can soleve this problem.
In addition, i modify the comment.
Signed-off-by: Jianpeng Ma <majianpeng@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block')
-rw-r--r-- | block/blk-core.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/block/blk-core.c b/block/blk-core.c index 93a18d1d3da8..91037f74668e 100644 --- a/block/blk-core.c +++ b/block/blk-core.c | |||
@@ -1549,11 +1549,9 @@ get_rq: | |||
1549 | if (plug) { | 1549 | if (plug) { |
1550 | /* | 1550 | /* |
1551 | * If this is the first request added after a plug, fire | 1551 | * If this is the first request added after a plug, fire |
1552 | * of a plug trace. If others have been added before, check | 1552 | * of a plug trace. |
1553 | * if we have multiple devices in this plug. If so, make a | ||
1554 | * note to sort the list before dispatch. | ||
1555 | */ | 1553 | */ |
1556 | if (list_empty(&plug->list)) | 1554 | if (!request_count) |
1557 | trace_block_plug(q); | 1555 | trace_block_plug(q); |
1558 | else { | 1556 | else { |
1559 | if (request_count >= BLK_MAX_REQUEST_COUNT) { | 1557 | if (request_count >= BLK_MAX_REQUEST_COUNT) { |