[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 20/24] block: Add missing GRAPH_RDLOCK annotations
From: |
Eric Blake |
Subject: |
Re: [PATCH 20/24] block: Add missing GRAPH_RDLOCK annotations |
Date: |
Mon, 30 Oct 2023 16:19:04 -0500 |
User-agent: |
NeoMutt/20231023 |
On Fri, Oct 27, 2023 at 05:53:29PM +0200, Kevin Wolf wrote:
> This adds GRAPH_RDLOCK to some driver callbacks that are already called
> with the graph lock held, and which will need the annotation because
> they access bs->file, but don't have it yet.
>
> This also covers a few callbacks that were not marked GRAPH_RDLOCK
> before, but where updating BlockDriver is trivially possible.
>
> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
> ---
> block/qcow2.h | 11 ++++++-----
> include/block/block_int-common.h | 17 +++++++++--------
> block/blkdebug.c | 13 +++++--------
> block/blkverify.c | 2 +-
> block/copy-before-write.c | 2 +-
> block/filter-compress.c | 3 ++-
> block/io.c | 2 ++
> block/parallels.c | 12 ++++--------
> block/preallocate.c | 4 ++--
> block/qcow.c | 2 +-
> block/qcow2.c | 4 ++--
> block/qed.c | 10 ++++------
> block/raw-format.c | 16 ++++++++--------
> block/snapshot-access.c | 2 +-
> block/vdi.c | 9 ++++-----
> block/vhdx.c | 6 +++---
> 16 files changed, 55 insertions(+), 60 deletions(-)
>
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc.
Virtualization: qemu.org | libguestfs.org
- Re: [PATCH 06/24] block: Mark bdrv_filter_or_cow_bs() and callers GRAPH_RDLOCK, (continued)
- [PATCH 07/24] block: Mark bdrv_skip_implicit_filters() and callers GRAPH_RDLOCK, Kevin Wolf, 2023/10/27
- [PATCH 17/24] block: Protect bs->backing with graph_lock, Kevin Wolf, 2023/10/27
- [PATCH 16/24] block: Mark bdrv_replace_node() GRAPH_WRLOCK, Kevin Wolf, 2023/10/27
- [PATCH 18/24] blkverify: Add locking for request_fn, Kevin Wolf, 2023/10/27
- [PATCH 20/24] block: Add missing GRAPH_RDLOCK annotations, Kevin Wolf, 2023/10/27
- Re: [PATCH 20/24] block: Add missing GRAPH_RDLOCK annotations,
Eric Blake <=
- [PATCH 19/24] block: Introduce bdrv_co_change_backing_file(), Kevin Wolf, 2023/10/27
- [PATCH 12/24] block: Mark bdrv_cow_child() and callers GRAPH_RDLOCK, Kevin Wolf, 2023/10/27
- [PATCH 22/24] vhdx: Take locks for accessing bs->file, Kevin Wolf, 2023/10/27
- [PATCH 23/24] block: Take graph lock for most of .bdrv_open, Kevin Wolf, 2023/10/27
- [PATCH 24/24] block: Protect bs->file with graph_lock, Kevin Wolf, 2023/10/27