On 10/12/2016 05:19 PM, Pavel Grunt wrote:
On Wed, 2016-10-12 at 10:13 -0400, Frediano Ziglio wrote:
On Wed, 2016-10-12 at 06:57 -0400, Frediano Ziglio wrote:
When MainChannelClient was split to a separate file, the
for incrementing this field was supposed to belong to the
function (main_channel_connect_semi_seamless()), but by
incremented both there and in the client function
The bug was introduced in
server/main-channel-client.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/server/main-channel-client.c b/server/main-
index b30083f..0913028 100644
@@ -704,7 +704,6 @@ void
RedChannelClient *rcc = RED_CHANNEL_CLIENT(mcc);
- MainChannel* main_channel =
RedClient *client =
@@ -718,7 +717,6 @@ gboolean
mcc->priv->mig_wait_connect = TRUE;
mcc->priv->mig_connect_ok = FALSE;
Acked-by: Frediano Ziglio <fzig...@redhat.com>
I think we have a lack of testing, at least fast ones.
How did you discover this?
Just code inspection. While splitting out the MainChannelPrivate
into a separate patch, I noticed that this hunk was included in
"Convert RedChannel Hierarchy to GObject" patch. And it looked odd
me, so I went to look at the history of this particular code.
Did you manage to reproduce the issue?
I remember Uri can test migration locally but I never dig around
Uri, how to test it locally?
Take a look at:
That's a good test.
Seems like an n'th version of a test I wrote long ago :-)
Would be nice to add seamless migration support.
Frediano, to do it manually I basically run the same qemu-kvm
command in two terminals, one is the migration source and
the other migration destination. On a third terminal I run the client.
You have to give them different spice-port and destination
has to have incoming tcp:localhost:$migration-port
For each src/dst I add "monitor=stdio" such that it accepts
commands from stdin.
For seamless migration you want to add ",seamless-migration=on" to the
command line and issue qemu monitor command on the src:
(qemu-src) client_migrate_info spice localhost $remote-spice-port
You can verify the client is connected to dst with (not needed)
(qemu-dst) info spice
and than migrate
(qemu-src) migrate [-d (*)] tcp:localhost:$migration-port
you can make it go faster/slower (*) with
and query the status (*) with
(qemu-src) info migrate
(*) You can issue those commands during migration if
you ran migrate with "-d"
Upon successful completion quit the src as dst is now
the active instance of the VM.
Spice-devel mailing list