Re: [Qemu-devel] [PATCH v2 3/5] migration: do explicit incoming setup for rdma

2018-06-28 Thread Peter Xu
On Wed, Jun 27, 2018 at 03:13:04PM +0100, Dr. David Alan Gilbert wrote: > * Peter Xu (pet...@redhat.com) wrote: > > On Wed, Jun 27, 2018 at 01:25:45PM +0100, Dr. David Alan Gilbert wrote: > > > * Peter Xu (pet...@redhat.com) wrote: > > > > RDMA does not support postcopy recovery, so no need to go i

Re: [Qemu-devel] [PATCH v2 3/5] migration: do explicit incoming setup for rdma

2018-06-27 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > On Wed, Jun 27, 2018 at 01:25:45PM +0100, Dr. David Alan Gilbert wrote: > > * Peter Xu (pet...@redhat.com) wrote: > > > RDMA does not support postcopy recovery, so no need to go into such > > > logic. Instead of calling migration_fd_process_incoming(), let's

Re: [Qemu-devel] [PATCH v2 3/5] migration: do explicit incoming setup for rdma

2018-06-27 Thread Peter Xu
On Wed, Jun 27, 2018 at 01:25:45PM +0100, Dr. David Alan Gilbert wrote: > * Peter Xu (pet...@redhat.com) wrote: > > RDMA does not support postcopy recovery, so no need to go into such > > logic. Instead of calling migration_fd_process_incoming(), let's call > > the two functions that setup the inc

Re: [Qemu-devel] [PATCH v2 3/5] migration: do explicit incoming setup for rdma

2018-06-27 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > RDMA does not support postcopy recovery, so no need to go into such > logic. Instead of calling migration_fd_process_incoming(), let's call > the two functions that setup the incoming migration. There should have > no functional change at all. Can you expl

[Qemu-devel] [PATCH v2 3/5] migration: do explicit incoming setup for rdma

2018-06-27 Thread Peter Xu
RDMA does not support postcopy recovery, so no need to go into such logic. Instead of calling migration_fd_process_incoming(), let's call the two functions that setup the incoming migration. There should have no functional change at all. Signed-off-by: Peter Xu --- migration/rdma.c | 3 ++- 1