Re: [Qemu-devel] [PATCH RFC 6/6] migration: enable return path for precopy

2017-05-31 Thread Juan Quintela
Peter Xu wrote: ... >> > Here what I wanted to achieve is that: >> > >> > a. for postcopy, we should try to enable return path, and it must >> >succeed >> > >> > b. for the case when enable_return_path is set, we try to enable return >> >path, but even if it failed, we can still continue

Re: [Qemu-devel] [PATCH RFC 6/6] migration: enable return path for precopy

2017-05-31 Thread Peter Xu
On Wed, May 31, 2017 at 09:43:21AM +0200, Juan Quintela wrote: > Peter Xu wrote: > > On Tue, May 30, 2017 at 05:59:10PM +0200, Juan Quintela wrote: > >> Peter Xu wrote: > >> > Let this be a flag, default to on. Turn it off for <=2.9 versions. > >> > > >> > After this patch, return path will be on

Re: [Qemu-devel] [PATCH RFC 6/6] migration: enable return path for precopy

2017-05-31 Thread Juan Quintela
Peter Xu wrote: > On Tue, May 30, 2017 at 05:59:10PM +0200, Juan Quintela wrote: >> Peter Xu wrote: >> > Let this be a flag, default to on. Turn it off for <=2.9 versions. >> > >> > After this patch, return path will be on even for pre-copy migration as >> > long as the transport support, e.g., f

Re: [Qemu-devel] [PATCH RFC 6/6] migration: enable return path for precopy

2017-05-31 Thread Peter Xu
On Tue, May 30, 2017 at 05:59:10PM +0200, Juan Quintela wrote: > Peter Xu wrote: > > Let this be a flag, default to on. Turn it off for <=2.9 versions. > > > > After this patch, return path will be on even for pre-copy migration as > > long as the transport support, e.g., for socket typed transpor

Re: [Qemu-devel] [PATCH RFC 6/6] migration: enable return path for precopy

2017-05-30 Thread Juan Quintela
Peter Xu wrote: > Let this be a flag, default to on. Turn it off for <=2.9 versions. > > After this patch, return path will be on even for pre-copy migration as > long as the transport support, e.g., for socket typed transport > including "tcp|udp|unix" typed. > > This will naturally fix the bug m

[Qemu-devel] [PATCH RFC 6/6] migration: enable return path for precopy

2017-05-18 Thread Peter Xu
Let this be a flag, default to on. Turn it off for <=2.9 versions. After this patch, return path will be on even for pre-copy migration as long as the transport support, e.g., for socket typed transport including "tcp|udp|unix" typed. This will naturally fix the bug mentioned below, when destinat