Automatic mutex management - part 6

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

Tim Wiederhake posted 10 patches 2 years ago

Download series mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20220412145727.69564-1-twiederh@redhat.com

src/conf/virstorageobj.c        | 185 +++++------
src/rpc/virnetserver.c          | 295 +++++++-----------
src/rpc/virnetserverclient.c    | 432 ++++++++++++--------------
src/security/security_manager.c | 524 +++++++++++++-------------------
src/vz/vz_driver.c              |  50 ++-
5 files changed, 604 insertions(+), 882 deletions(-)

Expand all Fold all

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

Posted by Tim Wiederhake 2 years ago

Use the recently implemented VIR_LOCK_GUARD and VIR_WITH_MUTEX_LOCK_GUARD
to simplify mutex management.

Tim Wiederhake (10):
  virstorageobj: Replace deprecated virHash functions
  virStoragePoolObjAddVol: Simplify error path
  virStorageVolObjEndAPI: Remove
  virStorageVolObjNew: Move locking to usage site
  virstorageobject: Use automatic mutex management
  virNetServerProcessClients: Remove goto
  virnetserver: Use automatic mutex management
  virnetserverclient: Use automatic mutex management
  security_manager: Use automatic mutex management
  vz_driver: Use automatic mutex management

 src/conf/virstorageobj.c        | 185 +++++------
 src/rpc/virnetserver.c          | 295 +++++++-----------
 src/rpc/virnetserverclient.c    | 432 ++++++++++++--------------
 src/security/security_manager.c | 524 +++++++++++++-------------------
 src/vz/vz_driver.c              |  50 ++-
 5 files changed, 604 insertions(+), 882 deletions(-)

-- 
2.31.1

Re: [libvirt PATCH 00/10] Automatic mutex management - part 6

Posted by Michal Prívozník 2 years ago

On 4/12/22 16:57, Tim Wiederhake wrote:
> Use the recently implemented VIR_LOCK_GUARD and VIR_WITH_MUTEX_LOCK_GUARD
> to simplify mutex management.
> 
> Tim Wiederhake (10):
>   virstorageobj: Replace deprecated virHash functions
>   virStoragePoolObjAddVol: Simplify error path
>   virStorageVolObjEndAPI: Remove
>   virStorageVolObjNew: Move locking to usage site
>   virstorageobject: Use automatic mutex management
>   virNetServerProcessClients: Remove goto
>   virnetserver: Use automatic mutex management
>   virnetserverclient: Use automatic mutex management
>   security_manager: Use automatic mutex management
>   vz_driver: Use automatic mutex management
> 
>  src/conf/virstorageobj.c        | 185 +++++------
>  src/rpc/virnetserver.c          | 295 +++++++-----------
>  src/rpc/virnetserverclient.c    | 432 ++++++++++++--------------
>  src/security/security_manager.c | 524 +++++++++++++-------------------
>  src/vz/vz_driver.c              |  50 ++-
>  5 files changed, 604 insertions(+), 882 deletions(-)
> 

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

Michal