On 09/08/2017 07:51 AM, Kevin Wolf wrote: > Am 30.08.2017 um 23:05 hat Eric Blake geschrieben: >> Thanks to recent cleanups, all callers were scaling a return value >> of sectors into bytes; do the scaling internally instead. >> >> Signed-off-by: Eric Blake <ebl...@redhat.com> >> Reviewed-by: John Snow <js...@redhat.com> >> Reviewed-by: Juan Quintela <quint...@redhat.com> > >> diff --git a/block/mirror.c b/block/mirror.c >> index af13f5d658..cc47e21814 100644 >> --- a/block/mirror.c >> +++ b/block/mirror.c >> @@ -811,11 +811,10 @@ static void coroutine_fn mirror_run(void *opaque) > > There is one more place before this one that needs to be converted, > mirror_iteration() at line 343: > > trace_mirror_restart_iter(s, bdrv_get_dirty_count(s->dirty_bitmap) * > BDRV_SECTOR_SIZE);
Good catch. (I guess I'm a victim of sitting on a patch for several months, across several rebases...) -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature