diff options
| author | Eric Blake <eblake@redhat.com> | 2017-07-07 07:44:40 -0500 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2017-07-10 13:18:06 +0200 |
| commit | 5cb1a49e01db12a38790c982aafde94977f7b0ee (patch) | |
| tree | 6326520403bcfe69910738cc2fc1bff6684f835a /block/stream.c | |
| parent | f3e4ce4af336f2ea306fa0f40ec1a5149864ca8c (diff) | |
| download | qemu-5cb1a49e01db12a38790c982aafde94977f7b0ee.zip qemu-5cb1a49e01db12a38790c982aafde94977f7b0ee.tar.gz qemu-5cb1a49e01db12a38790c982aafde94977f7b0ee.tar.bz2 | |
trace: Show blockjob actions via bytes, not sectors
Upcoming patches are going to switch to byte-based interfaces
instead of sector-based. Even worse, trace_backup_do_cow_enter()
had a weird mix of cluster and sector indices.
The trace interface is low enough that there are no stability
guarantees, and therefore nothing wrong with changing our units,
even in cases like trace_backup_do_cow_skip() where we are not
changing the trace output. So make the tracing uniformly use
bytes.
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Reviewed-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/stream.c')
| -rw-r--r-- | block/stream.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/block/stream.c b/block/stream.c index 29273a5..6cb3939 100644 --- a/block/stream.c +++ b/block/stream.c @@ -168,7 +168,8 @@ static void coroutine_fn stream_run(void *opaque) copy = (ret == 1); } - trace_stream_one_iteration(s, sector_num, n, ret); + trace_stream_one_iteration(s, sector_num * BDRV_SECTOR_SIZE, + n * BDRV_SECTOR_SIZE, ret); if (copy) { ret = stream_populate(blk, sector_num, n, buf); } |