[libvirt PATCH 00/10] Automatic mutex management - part 2

Tim Wiederhake posted 10 patches 2 years, 2 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20220211103046.34651-1-twiederh@redhat.com
src/secret/secret_driver.c   |  62 ++++++++------------
src/util/virfirewall.c       |  13 ++---
src/util/virlockspace.c      | 106 +++++++++++------------------------
src/util/virtpm.c            |  39 ++++---------
src/vbox/vbox_common.c       |  18 +-----
src/vmware/vmware_driver.c   | 100 ++++++++++-----------------------
src/vz/vz_driver.c           |  44 +++++++--------
tests/qemumonitortestutils.c |  65 +++++++++------------
tests/qemusecuritymock.c     |  88 ++++++++++-------------------
tools/virsh.c                |  12 ++--
tools/virt-admin.c           |  12 ++--
tools/vsh.c                  |   8 +--
12 files changed, 201 insertions(+), 366 deletions(-)
[libvirt PATCH 00/10] Automatic mutex management - part 2
Posted by Tim Wiederhake 2 years, 2 months ago
Use the recently implemented VIR_LOCK_GUARD and VIR_WITH_MUTEX_LOCK_GUARD
to simplify mutex management.

Tim Wiederhake (10):
  vz: Use automatic mutex management
  vmware: Use automatic mutex management
  secret: Factor out mutex
  secret: Use automatic mutex management
  virlockspace: Use automatic mutex management
  virtpm: Use automatic mutex management
  vbox: Use automatic mutex management
  tools: Use automatic mutex management
  qemusecuritymock: Use automatic mutex management
  qemumonitortestutils: Use automatic mutex management

 src/secret/secret_driver.c   |  62 ++++++++------------
 src/util/virfirewall.c       |  13 ++---
 src/util/virlockspace.c      | 106 +++++++++++------------------------
 src/util/virtpm.c            |  39 ++++---------
 src/vbox/vbox_common.c       |  18 +-----
 src/vmware/vmware_driver.c   | 100 ++++++++++-----------------------
 src/vz/vz_driver.c           |  44 +++++++--------
 tests/qemumonitortestutils.c |  65 +++++++++------------
 tests/qemusecuritymock.c     |  88 ++++++++++-------------------
 tools/virsh.c                |  12 ++--
 tools/virt-admin.c           |  12 ++--
 tools/vsh.c                  |   8 +--
 12 files changed, 201 insertions(+), 366 deletions(-)

-- 
2.31.1


Re: [libvirt PATCH 00/10] Automatic mutex management - part 2
Posted by Michal Prívozník 2 years, 2 months ago
On 2/11/22 11:30, Tim Wiederhake wrote:
> Use the recently implemented VIR_LOCK_GUARD and VIR_WITH_MUTEX_LOCK_GUARD
> to simplify mutex management.
> 
> Tim Wiederhake (10):
>   vz: Use automatic mutex management
>   vmware: Use automatic mutex management
>   secret: Factor out mutex
>   secret: Use automatic mutex management
>   virlockspace: Use automatic mutex management
>   virtpm: Use automatic mutex management
>   vbox: Use automatic mutex management
>   tools: Use automatic mutex management
>   qemusecuritymock: Use automatic mutex management
>   qemumonitortestutils: Use automatic mutex management
> 
>  src/secret/secret_driver.c   |  62 ++++++++------------
>  src/util/virfirewall.c       |  13 ++---
>  src/util/virlockspace.c      | 106 +++++++++++------------------------
>  src/util/virtpm.c            |  39 ++++---------
>  src/vbox/vbox_common.c       |  18 +-----
>  src/vmware/vmware_driver.c   | 100 ++++++++++-----------------------
>  src/vz/vz_driver.c           |  44 +++++++--------
>  tests/qemumonitortestutils.c |  65 +++++++++------------
>  tests/qemusecuritymock.c     |  88 ++++++++++-------------------
>  tools/virsh.c                |  12 ++--
>  tools/virt-admin.c           |  12 ++--
>  tools/vsh.c                  |   8 +--
>  12 files changed, 201 insertions(+), 366 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal