[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 24/28] migration: Make events a capability
From: |
Jiri Denemark |
Subject: |
Re: [Qemu-devel] [PULL 24/28] migration: Make events a capability |
Date: |
Wed, 8 Jul 2015 08:14:40 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Jul 07, 2015 at 15:09:05 +0200, Juan Quintela wrote:
> Make check fails with events. THis is due to the parser/lexer that it
> uses. Just in case that they are more broken parsers, just only send
> events when there are capabilities.
>
> Signed-off-by: Juan Quintela <address@hidden>
> Reviewed-by: Dr. David Alan Gilbert <address@hidden>
...
> diff --git a/qapi-schema.json b/qapi-schema.json
> index 106008c..1285b8c 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -523,6 +523,9 @@
> # minimize migration traffic. The feature is disabled by default.
> # (since 2.4 )
> #
> +# @events: generate events for each migration state change
> +# (since 2.4 )
> +#
> # @auto-converge: If enabled, QEMU will automatically throttle down the guest
> # to speed up convergence of RAM migration. (since 1.6)
> #
> @@ -530,7 +533,7 @@
> ##
> { 'enum': 'MigrationCapability',
> 'data': ['xbzrle', 'rdma-pin-all', 'auto-converge', 'zero-blocks',
> - 'compress'] }
> + 'compress', 'events'] }
>
Perhaps I messed something up, but I don't see this new capability
documented in qmp-commands.hx and
{"execute":"query-migrate-capabilities"} does not report its status:
{
"return": [
{
"state": false,
"capability": "xbzrle"
},
{
"state": false,
"capability": "rdma-pin-all"
},
{
"state": false,
"capability": "auto-converge"
},
{
"state": false,
"capability": "zero-blocks"
},
{
"state": false,
"capability": "compress"
}
]
}
Blindly setting it does not work either:
{
"execute": "migrate-set-capabilities",
"arguments": {
"capabilities": [
{
"capability": "events",
"state": "true"
}
]
}
}
returns
{
"error": {
"class": "GenericError",
"desc": "Invalid parameter 'events'"
}
}
Jirka
- [Qemu-devel] [PULL 18/28] migration: Add configuration section, (continued)
- [Qemu-devel] [PULL 18/28] migration: Add configuration section, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 17/28] vmstate: Create optional sections, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 19/28] migration: Use cmpxchg correctly, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 20/28] migration: ensure we start in NONE state, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 22/28] migration: No need to call trace_migrate_set_state(), Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 23/28] migration: create migration event, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 21/28] migration: Use always helper to set state, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 24/28] migration: Make events a capability, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 25/28] migration: Add migration events on target side, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 26/28] check_section_footers: Check the correct section_id, Juan Quintela, 2015/07/07
- [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Juan Quintela, 2015/07/07
- Re: [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Kevin Wolf, 2015/07/08
- Re: [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Paolo Bonzini, 2015/07/08
- Re: [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Wen Congyang, 2015/07/08
- Re: [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Paolo Bonzini, 2015/07/09
- Re: [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Wen Congyang, 2015/07/09
- Re: [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Paolo Bonzini, 2015/07/09
- Re: [Qemu-devel] [PULL 27/28] migration: protect migration_bitmap, Wen Congyang, 2015/07/09