[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] migration: expose qemu_announce_self() via qmp
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH] migration: expose qemu_announce_self() via qmp |
Date: |
Tue, 17 Jan 2017 11:41:57 +0000 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
* Germano Veit Michel (address@hidden) wrote:
> qemu_announce_self() is triggered by qemu at the end of migrations
> to update the network regarding the path to the guest l2addr.
>
> however it is also useful when there is a network change such as
> an active bond slave swap. Essentially, it's the same as a migration
> from a network perspective - the guest moves to a different point
> in the network topology.
>
> this exposes the function via qmp.
>
> Signed-off-by: Germano Veit Michel <address@hidden>
Hi Germano,
I think there are a couple of problems which mean it's
not quite this simple.
a) The 'count' variable in qemu_announce_self_once is static
initialised; so I think qemu will only ever run the announce
series once; so it would do it after migrate, but then I think
if you did a qemu_announce_self again I think it would send
one packet and then not repeat it.
Similarly if you called qemu_announce_self from qmp twice I'm
not sure the 2nd time would only send the one packet.
b) The static *time in qemu_announce_self means I'm worried about
what happens if someone was to call qmp_announce_self before the
previous round of qmp_announce_self had finished; I think you'd
end up with 2 timers racing on one pointer.
Dave
> ---
> migration/savevm.c | 5 +++++
> qapi-schema.json | 18 ++++++++++++++++++
> 2 files changed, 23 insertions(+)
>
> diff --git a/migration/savevm.c b/migration/savevm.c
> index f9c06e9..7a817dc 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -143,6 +143,11 @@ void qemu_announce_self(void)
> qemu_announce_self_once(&timer);
> }
>
> +void qmp_announce_self(Error **errp)
> +{
> + qemu_announce_self();
> +}
> +
> /***********************************************************/
> /* savevm/loadvm support */
>
> diff --git a/qapi-schema.json b/qapi-schema.json
> index ce20f16..6854266 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -4786,3 +4786,21 @@
> # Since: 2.7
> ##
> { 'command': 'query-hotpluggable-cpus', 'returns': ['HotpluggableCPU'] }
> +
> +##
> +# @announce-self:
> +#
> +# Trigger generation of broadcast RARP frames to update network switches.
> +# This can be useful when network bonds fail-over the active slave.
> +#
> +# Arguments: None.
> +#
> +# Example:
> +#
> +# -> { "execute": "announce-self" }
> +# <- { "return": {} }
> +#
> +# Since: 2.9
> +##
> +{ 'command': 'announce-self' }
> +
> --
> 2.9.3
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK