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 li...@linux.vnet.ibm.com --- include/migration/migration.h |3 +++ migration.c | 12 qapi-schema.json

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]) { +

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_LOCALHOST])

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 appropriate

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 examine

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); +}

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

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

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 li...@linux.vnet.ibm.com --- include/migration/migration.h |3 +++ migration.c | 12 qapi-schema.json |8 +++- 3 files changed,

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); }