Re: [Qemu-devel] [PATCH v2 2/9] migration: Fix a potential issue

2016-06-13 Thread Amit Shah
On (Mon) 13 Jun 2016 [05:07:39], Li, Liang Z wrote: > > > > > +static void wait_for_decompress_done(void) { > > > > > +int idx, thread_count; > > > > > + > > > > > +if (!migrate_use_compression()) { > > > > > +return; > > > > > +} > > > > > + > > > > > +thread_count =

Re: [Qemu-devel] [PATCH v2 2/9] migration: Fix a potential issue

2016-06-12 Thread Li, Liang Z
> > > > +static void wait_for_decompress_done(void) { > > > > +int idx, thread_count; > > > > + > > > > +if (!migrate_use_compression()) { > > > > +return; > > > > +} > > > > + > > > > +thread_count = migrate_decompress_threads(); > > > > +qemu_mutex_lock(_done_lock); >

Re: [Qemu-devel] [PATCH v2 2/9] migration: Fix a potential issue

2016-06-12 Thread Amit Shah
On (Fri) 10 Jun 2016 [15:03:15], Li, Liang Z wrote: > > Subject: Re: [PATCH v2 2/9] migration: Fix a potential issue > > > > On (Thu) 05 May 2016 [15:32:52], Liang Li wrote: > > > At the end of live migration and before vm_start() on the destination > > > side, we should make sure all the

Re: [Qemu-devel] [PATCH v2 2/9] migration: Fix a potential issue

2016-06-10 Thread Li, Liang Z
> Subject: Re: [PATCH v2 2/9] migration: Fix a potential issue > > On (Thu) 05 May 2016 [15:32:52], Liang Li wrote: > > At the end of live migration and before vm_start() on the destination > > side, we should make sure all the decompression tasks are finished, if > > this can not be guaranteed,

Re: [Qemu-devel] [PATCH v2 2/9] migration: Fix a potential issue

2016-06-10 Thread Amit Shah
On (Thu) 05 May 2016 [15:32:52], Liang Li wrote: > At the end of live migration and before vm_start() on the destination > side, we should make sure all the decompression tasks are finished, if > this can not be guaranteed, the VM may get the incorrect memory data, > or the updated memory may be

[Qemu-devel] [PATCH v2 2/9] migration: Fix a potential issue

2016-05-05 Thread Liang Li
At the end of live migration and before vm_start() on the destination side, we should make sure all the decompression tasks are finished, if this can not be guaranteed, the VM may get the incorrect memory data, or the updated memory may be overwritten by the decompression thread. Add the code to