Re: [Qemu-devel] [PATCH 1/1] migration: fix ram decompression race deadlock

2016-06-09 Thread Amit Shah
On (Tue) 24 May 2016 [08:35:32], Denis V. Lunev wrote: > On 05/24/2016 05:07 AM, Li, Liang Z wrote: > > Thanks for your patch! And it can help to fix this issue. > > Actually, the is not the only issue in the current multi-thread > > (de)compression code. > > So I submitted a path set to fix all

Re: [Qemu-devel] [PATCH 1/1] migration: fix ram decompression race deadlock

2016-05-23 Thread Denis V. Lunev
On 05/24/2016 05:07 AM, Li, Liang Z wrote: Adding Liang Li to CC for his comments as the author of the feature. On (Fri) 13 May 2016 [10:27:01], Denis V. Lunev wrote: From: Maxim Nestratov There is a race in between do_data_decompress and start_decompression. do_data_decompress() while

Re: [Qemu-devel] [PATCH 1/1] migration: fix ram decompression race deadlock

2016-05-23 Thread Li, Liang Z
> Adding Liang Li to CC for his comments as the author of the feature. > > On (Fri) 13 May 2016 [10:27:01], Denis V. Lunev wrote: > > From: Maxim Nestratov > > > > There is a race in between do_data_decompress and start_decompression. > > > > do_data_decompress() > > while (!quit_decomp_threa

Re: [Qemu-devel] [PATCH 1/1] migration: fix ram decompression race deadlock

2016-05-23 Thread Amit Shah
Adding Liang Li to CC for his comments as the author of the feature. On (Fri) 13 May 2016 [10:27:01], Denis V. Lunev wrote: > From: Maxim Nestratov > > There is a race in between do_data_decompress and start_decompression. > > do_data_decompress() > while (!quit_decomp_thread) { > q

Re: [Qemu-devel] [PATCH 1/1] migration: fix ram decompression race deadlock

2016-05-19 Thread Denis V. Lunev
On 05/13/2016 10:27 AM, Denis V. Lunev wrote: From: Maxim Nestratov There is a race in between do_data_decompress and start_decompression. do_data_decompress() while (!quit_decomp_thread) { qemu_mutex_lock(¶m->mutex); while (!param->start && !quit_decomp_thread) {

[Qemu-devel] [PATCH 1/1] migration: fix ram decompression race deadlock

2016-05-13 Thread Denis V. Lunev
From: Maxim Nestratov There is a race in between do_data_decompress and start_decompression. do_data_decompress() while (!quit_decomp_thread) { qemu_mutex_lock(¶m->mutex); while (!param->start && !quit_decomp_thread) { qemu_cond_wait(¶m->cond, ¶m->mutex);