Re: [Qemu-devel] [PATCH v6 12/21] qmp: Add dirty bitmap status field in query-block

[Eric Blake]

On 04/17/2015 05:50 PM, John Snow wrote:
> Add the "frozen" status booleans, to inform clients
> when a bitmap is occupied doing a task.
> 
> Signed-off-by: Fam Zheng <address@hidden>
> Signed-off-by: John Snow <address@hidden>
> Reviewed-by: Max Reitz <address@hidden>
> Reviewed-by: Stefan Hajnoczi <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> ---
>  block.c              | 1 +
>  qapi/block-core.json | 5 ++++-
>  2 files changed, 5 insertions(+), 1 deletion(-)

> +++ b/qapi/block-core.json
> @@ -336,10 +336,13 @@
>  #
>  # @granularity: granularity of the dirty bitmap in bytes (since 1.4)
>  #
> +# @frozen: whether the dirty bitmap is frozen (Since 2.4)
> +#
>  # Since: 1.3
>  ##
>  { 'type': 'BlockDirtyInfo',
> -  'data': {'*name': 'str', 'count': 'int', 'granularity': 'uint32'} }
> +  'data': {'*name': 'str', 'count': 'int', 'granularity': 'uint32',
> +           'frozen': 'bool'} }

Just thinking aloud here - internally, we have a tri-state situation
(enabled, disabled, or frozen).  I know we aren't exposing disabled to
the end user yet (no use case for that), but would it be better to make
this output an enum type (two values for now, 'enabled' and 'frozen') to
make it easier to add a third value later, without having to add yet
another boolean?

But it's not the end of the world to expose a single boolean now (we'd
just have to maintain it forever, even if we add later states), and
adding an enum now just adds complexity that we may not need, so:

Reviewed-by: Eric Blake <address@hidden>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature