[v2] Use lock guard macros in block

[PATCH v2 0/4] Use lock guard macros in block

Gan Qixin posted 4 patches 4 years, 11 months ago

Diff against v1
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201203075055.127773-1-ganqixin@huawei.com

Maintainers: Peter Lieven <pl@kamp.de>, Max Reitz <mreitz@redhat.com>, Alberto Garcia <berto@igalia.com>, Ronnie Sahlberg <ronniesahlberg@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Kevin Wolf <kwolf@redhat.com>

block/accounting.c      | 32 +++++++++++++-------------
block/curl.c            | 28 +++++++++++------------
block/iscsi.c           | 50 ++++++++++++++++++++---------------------
block/throttle-groups.c | 48 +++++++++++++++++++--------------------
4 files changed, 76 insertions(+), 82 deletions(-)

Expand all Fold all

[PATCH v2 0/4] Use lock guard macros in block

Posted by Gan Qixin 4 years, 11 months ago

v1->v2:

-Patch1:
    Add Paolo Bonzini reviewed tag and delete the .c suffix in the commit
message.

-Patch2:
    Add Paolo Bonzini reviewed tag and delete the .c suffix in the commit
message.

-Patch3:
    Delete the .c suffix in the commit.
    Changes suggested by Kevin Wolf: Fix wrong indentation format.
    
-Patch4:
    Delete the .c suffix in the commit.
    Changes suggested by Kevin Wolf: Replace QEMU_LOCK_GUARD with
WITH_QEMU_LOCK_GUARD, and delete the redundant qemu_mutex_unlock().

Gan Qixin (4):
  block/accounting: Use lock guard macros
  block/curl: Use lock guard macros
  block/throttle-groups: Use lock guard macros
  block/iscsi: Use lock guard macros

 block/accounting.c      | 32 +++++++++++++-------------
 block/curl.c            | 28 +++++++++++------------
 block/iscsi.c           | 50 ++++++++++++++++++++---------------------
 block/throttle-groups.c | 48 +++++++++++++++++++--------------------
 4 files changed, 76 insertions(+), 82 deletions(-)

-- 
2.27.0

Re: [PATCH v2 0/4] Use lock guard macros in block

Posted by Kevin Wolf 4 years, 11 months ago

Am 03.12.2020 um 08:50 hat Gan Qixin geschrieben:
> v1->v2:
> 
> -Patch1:
>     Add Paolo Bonzini reviewed tag and delete the .c suffix in the commit
> message.
> 
> -Patch2:
>     Add Paolo Bonzini reviewed tag and delete the .c suffix in the commit
> message.
> 
> -Patch3:
>     Delete the .c suffix in the commit.
>     Changes suggested by Kevin Wolf: Fix wrong indentation format.
>     
> -Patch4:
>     Delete the .c suffix in the commit.
>     Changes suggested by Kevin Wolf: Replace QEMU_LOCK_GUARD with
> WITH_QEMU_LOCK_GUARD, and delete the redundant qemu_mutex_unlock().

Thanks, applied to the block branch.

Kevin