[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 14/16] qapi: fix example of query-migrate command
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v1 14/16] qapi: fix example of query-migrate command |
Date: |
Wed, 31 Aug 2022 13:52:33 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Victor Toso <victortoso@redhat.com> writes:
> The example's return type has several missing mandatory member names.
> Fix it.
>
> Problem was noticed when using the example as a test case for Go
> bindings.
>
> Signed-off-by: Victor Toso <victortoso@redhat.com>
> ---
> qapi/migration.json | 56 +++++++++++++++++++++++++++++++++++++++++----
> 1 file changed, 52 insertions(+), 4 deletions(-)
>
> diff --git a/qapi/migration.json b/qapi/migration.json
> index 81185d4311..2ac26d1971 100644
> --- a/qapi/migration.json
> +++ b/qapi/migration.json
> @@ -295,7 +295,18 @@
> # "duplicate":123,
> # "normal":123,
> # "normal-bytes":123456,
> -# "dirty-sync-count":15
> +# "dirty-sync-count":15,
> +# "mbps":0,
> +# "skipped":0,
> +# "postcopy-requests":0,
> +# "page-size":0,
> +# "multifd-bytes":0,
> +# "pages-per-second":0,
> +# "precopy-bytes":0,
> +# "downtime-bytes":0,
> +# "postcopy-bytes":0,
> +# "dirty-sync-missed-zero-copy":0,
> +# "dirty-pages-rate":0
> # }
> # }
> # }
> @@ -321,7 +332,18 @@
> # "duplicate":123,
> # "normal":123,
> # "normal-bytes":123456,
> -# "dirty-sync-count":15
> +# "dirty-sync-count":15,
> +# "mbps":0,
> +# "skipped":0,
> +# "postcopy-requests":0,
> +# "page-size":0,
> +# "multifd-bytes":0,
> +# "pages-per-second":0,
> +# "precopy-bytes":0,
> +# "downtime-bytes":0,
> +# "postcopy-bytes":0,
> +# "dirty-sync-missed-zero-copy":0,
> +# "dirty-pages-rate":0
> # }
> # }
> # }
> @@ -342,12 +364,38 @@
> # "duplicate":123,
> # "normal":123,
> # "normal-bytes":123456,
> -# "dirty-sync-count":15
> +# "dirty-sync-count":15,
> +# "mbps":1,
> +# "skipped":1,
> +# "postcopy-requests":1,
> +# "page-size":1,
> +# "multifd-bytes":1,
> +# "pages-per-second":1,
> +# "precopy-bytes":1,
> +# "downtime-bytes":1,
> +# "postcopy-bytes":1,
> +# "dirty-sync-missed-zero-copy":1,
> +# "dirty-pages-rate":1
> # },
> # "disk":{
> # "total":20971520,
> # "remaining":20880384,
> -# "transferred":91136
> +# "transferred":91136,
> +# "duplicate":2,
> +# "skipped":2,
> +# "normal":2,
> +# "normal-bytes":2,
> +# "dirty-pages-rate":2,
> +# "mbps":2,
> +# "dirty-sync-count":2,
> +# "postcopy-requests":2,
> +# "page-size":2,
> +# "multifd-bytes":2,
> +# "pages-per-second":2,
> +# "precopy-bytes":2,
> +# "downtime-bytes":2,
> +# "postcopy-bytes":2,
> +# "dirty-sync-missed-zero-copy":2
> # }
> # }
> # }
Full example output is fairly long. Perhaps that's what we want. Or
perhaps we'd prefer abridged output. See also my review of PATCH 03.
- [PATCH v1 09/16] qapi: fix example of DEVICE_UNPLUG_GUEST_ERROR event, (continued)
[PATCH v1 14/16] qapi: fix example of query-migrate command, Victor Toso, 2022/08/30
- Re: [PATCH v1 14/16] qapi: fix example of query-migrate command,
Markus Armbruster <=
[PATCH v1 10/16] qapi: fix example of MEM_UNPLUG_ERROR event, Victor Toso, 2022/08/30
[PATCH v1 15/16] qapi: fix examples of events missing timestamp, Victor Toso, 2022/08/30
[PATCH v1 13/16] qapi: fix example of query-hotpluggable-cpus command, Victor Toso, 2022/08/30
[PATCH v1 16/16] RFC: add a generator for qapi's examples, Victor Toso, 2022/08/30