Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-27 Thread Lei Li
On 08/21/2013 11:08 PM, Eric Blake wrote: On 08/21/2013 01:18 AM, Lei Li wrote: Introduce migration capability localhost. Signed-off-by: Lei Li --- include/migration/migration.h |3 +++ migration.c | 12 qapi-schema.json |8 +++- 3

Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-23 Thread Lei Li
On 08/23/2013 04:01 PM, Paolo Bonzini wrote: Il 23/08/2013 09:51, Lei Li ha scritto: Because in these patches, the local URI is only present in the destination (which is wrong: the destination should autodetect local mode using the load-page hook). As you can see above, a "unix" URI will examin

Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-23 Thread Paolo Bonzini
Il 23/08/2013 09:51, Lei Li ha scritto: >> Because in these patches, the local URI is only present in the >> destination (which is wrong: the destination should autodetect local >> mode using the load-page hook). As you can see above, a "unix" URI will >> examine the capability and pick the approp

Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-23 Thread Lei Li
On 08/23/2013 03:40 PM, Paolo Bonzini wrote: Il 22/08/2013 22:50, Michael R. Hines ha scritto: On 08/21/2013 11:18 AM, Paolo Bonzini wrote: Il 21/08/2013 09:18, Lei Li ha scritto: } else if (strstart(uri, "unix:", &p)) { +if (s->enabled_capabilities[MIGRATION_CAPABILITY_LOCALHOS

Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-23 Thread Paolo Bonzini
Il 22/08/2013 22:50, Michael R. Hines ha scritto: > On 08/21/2013 11:18 AM, Paolo Bonzini wrote: >> Il 21/08/2013 09:18, Lei Li ha scritto: >>> } else if (strstart(uri, "unix:", &p)) { >>> +if (s->enabled_capabilities[MIGRATION_CAPABILITY_LOCALHOST]) { >>> +local_start_out

Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-22 Thread Michael R. Hines
On 08/21/2013 11:18 AM, Paolo Bonzini wrote: Il 21/08/2013 09:18, Lei Li ha scritto: } else if (strstart(uri, "unix:", &p)) { +if (s->enabled_capabilities[MIGRATION_CAPABILITY_LOCALHOST]) { +local_start_outgoing_migration(s, p, &local_err); +} unix_sta

Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-21 Thread Paolo Bonzini
Il 21/08/2013 09:18, Lei Li ha scritto: > } else if (strstart(uri, "unix:", &p)) { > +if (s->enabled_capabilities[MIGRATION_CAPABILITY_LOCALHOST]) { > +local_start_outgoing_migration(s, p, &local_err); > +} > unix_start_outgoing_migration(s, p, &local_err);

Re: [Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-21 Thread Eric Blake
On 08/21/2013 01:18 AM, Lei Li wrote: > Introduce migration capability localhost. > > Signed-off-by: Lei Li > --- > include/migration/migration.h |3 +++ > migration.c | 12 > qapi-schema.json |8 +++- > 3 files changed, 22 insertions(+),

[Qemu-devel] [PATCH 11/18] migration: introduce capability localhost

2013-08-21 Thread Lei Li
Introduce migration capability localhost. Signed-off-by: Lei Li --- include/migration/migration.h |3 +++ migration.c | 12 qapi-schema.json |8 +++- 3 files changed, 22 insertions(+), 1 deletions(-) diff --git a/include/migration/migrat