qemu: Fix deadlock if create qemuProcessReconnect thread failed

[libvirt] [PATCH v4 0/4] qemu: Fix deadlock if create qemuProcessReconnect thread failed

Wang Yechao posted 4 patches 5 years, 6 months ago

Download series mbox

Failed in applying to current master (apply log)

src/qemu/qemu_domain.c  | 79 ++++++++++++++++++++++++++++++++++++++++---------
src/qemu/qemu_domain.h  |  3 ++
src/qemu/qemu_process.c |  2 +-
3 files changed, 69 insertions(+), 15 deletions(-)

Expand all Fold all

[libvirt] [PATCH v4 0/4] qemu: Fix deadlock if create qemuProcessReconnect thread failed

Posted by Wang Yechao 5 years, 6 months ago

v3 patch:
https://www.redhat.com/archives/libvir-list/2018-September/msg00950.html

---
Changes in v4:
 - Split up v3 into four parts 

Wang Yechao (4):
  qemu: Split up qemuDomainRemoveInactive
  qemu: Create a qemuDomainRemoveInactiveLocked
  qemu: Create a qemuDomainRemoveInactiveJobLocked
  qemu: Fix deadlock if create qemuProcessReconnect thread failed

 src/qemu/qemu_domain.c  | 79 ++++++++++++++++++++++++++++++++++++++++---------
 src/qemu/qemu_domain.h  |  3 ++
 src/qemu/qemu_process.c |  2 +-
 3 files changed, 69 insertions(+), 15 deletions(-)

-- 
1.8.3.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v4 0/4] qemu: Fix deadlock if create qemuProcessReconnect thread failed

Posted by John Ferlan 5 years, 6 months ago


On 9/21/18 12:35 AM, Wang Yechao wrote:
> v3 patch:
> https://www.redhat.com/archives/libvir-list/2018-September/msg00950.html
> 
> ---
> Changes in v4:
>  - Split up v3 into four parts 
> 
> Wang Yechao (4):
>   qemu: Split up qemuDomainRemoveInactive
>   qemu: Create a qemuDomainRemoveInactiveLocked
>   qemu: Create a qemuDomainRemoveInactiveJobLocked
>   qemu: Fix deadlock if create qemuProcessReconnect thread failed
> 
>  src/qemu/qemu_domain.c  | 79 ++++++++++++++++++++++++++++++++++++++++---------
>  src/qemu/qemu_domain.h  |  3 ++
>  src/qemu/qemu_process.c |  2 +-
>  3 files changed, 69 insertions(+), 15 deletions(-)
> 

Thanks for splitting. I cleaned up the commit messages, merged patches
2&3 (must be able successfully build after each patch), and pushed the
series.

Reviewed-by: John Ferlan <jferlan@redhat.com>

John

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list