On 9/28/20 10:42 PM, Chuan Zheng wrote:
Make dirty_rate field optional, present dirty rate only when querying the rate has completed. The qmp results is shown as follow: @unstarted: {"return":{"status":"unstarted","start-time":0,"calc-time":0},"id":"libvirt-12"} @measuring: {"return":{"status":"measuring","start-time":102931,"calc-time":1},"id":"libvirt-85"} @measured: {"return":{"status":"measured","dirty-rate":4,"start-time":150146,"calc-time":1},"id":"libvirt-15"}
The "id":"libvirt-..." obviously come from different libvirt sessions, since they are in non-monotonic order. Not a show-stopper, since it is only a commit comment.
Signed-off-by: Chuan Zheng <zhengch...@huawei.com> Reviewed-by: David Edmondson <david.edmond...@oracle.com> --- migration/dirtyrate.c | 3 +-- qapi/migration.json | 8 +++----- 2 files changed, 4 insertions(+), 7 deletions(-)
Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org