Re: [PATCH 24/24] block: Protect bs->file with graph_lock

[Eric Blake]

On Fri, Oct 27, 2023 at 05:53:33PM +0200, Kevin Wolf wrote:
> Almost all functions that access bs->file already take the graph
> lock now. Add locking to the remaining users and finally annotate the
> struct field itself as protected by the graph lock.
> 
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
>  block/parallels.h                |  5 +++--
>  block/qed.h                      |  2 +-
>  include/block/block_int-common.h |  2 +-
>  block.c                          | 11 ++++++++---
>  block/blkreplay.c                |  8 +++++++-
>  block/copy-before-write.c        |  2 +-
>  block/crypto.c                   |  6 ++++++
>  block/dmg.c                      | 16 ++++++++++------
>  block/parallels-ext.c            | 21 ++++++++++-----------
>  block/parallels.c                |  6 ++++--
>  block/preallocate.c              | 19 ++++++++++++++-----
>  block/qed.c                      | 12 ++++++++++--
>  block/raw-format.c               |  9 ++++++---
>  block/replication.c              |  5 ++++-
>  block/vmdk.c                     | 14 ++++++++++++--
>  15 files changed, 97 insertions(+), 41 deletions(-)
> 

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.
Virtualization:  qemu.org | libguestfs.org