Re: [libvirt] [PATCH 08/19] qemu: Consolidate qemuMigrationPrepare{Direct, Tunnel}

2011-07-13 Thread Jiri Denemark
On Mon, Jul 11, 2011 at 21:28:30 +0100, Daniel P. Berrange wrote: > On Fri, Jul 08, 2011 at 01:34:13AM +0200, Jiri Denemark wrote: > > Most of the code in these two functions is supposed to be identical but > > currently it isn't (which is natural since the code is duplicated). > > Let's move commo

Re: [libvirt] [PATCH 08/19] qemu: Consolidate qemuMigrationPrepare{Direct, Tunnel}

2011-07-11 Thread Daniel P. Berrange
On Fri, Jul 08, 2011 at 01:34:13AM +0200, Jiri Denemark wrote: > Most of the code in these two functions is supposed to be identical but > currently it isn't (which is natural since the code is duplicated). > Let's move common parts of these functions into qemuMigrationPrepareAny. Were there any a

[libvirt] [PATCH 08/19] qemu: Consolidate qemuMigrationPrepare{Direct, Tunnel}

2011-07-07 Thread Jiri Denemark
Most of the code in these two functions is supposed to be identical but currently it isn't (which is natural since the code is duplicated). Let's move common parts of these functions into qemuMigrationPrepareAny. --- src/qemu/qemu_migration.c | 255 +++- 1