On Wed, 20 Sept 2023 at 05:22, Denis V. Lunev <den@openvz.org> wrote:
>
> The following changes since commit 4907644841e3200aea6475c0f72d3d987e9f3d93:
>
> Merge tag 'mem-2023-09-19' of https://github.com/davidhildenbrand/qemu into staging (2023-09-19 13:22:19 -0400)
>
> are available in the Git repository at:
>
> https://src.openvz.org/scm/~den/qemu.git tags/pull-parallels-2023-09-20
Hi Denis,
Please take a look at the following CI failure. I have dropped this
series for now.
clang -m64 -mcx16 -Ilibblock.fa.p -I. -I.. -Iqapi -Itrace -Iui
-Iui/shader -Iblock -I/usr/include/p11-kit-1 -I/usr/include/uuid
-I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include
-I/usr/include/sysprof-4 -flto -fcolor-diagnostics -Wall -Winvalid-pch
-Werror -std=gnu11 -O2 -g -fstack-protector-strong
-fsanitize=safe-stack -Wundef -Wwrite-strings -Wmissing-prototypes
-Wstrict-prototypes -Wredundant-decls -Wold-style-definition
-Wtype-limits -Wformat-security -Wformat-y2k -Winit-self
-Wignored-qualifiers -Wempty-body -Wnested-externs -Wendif-labels
-Wexpansion-to-defined -Wmissing-format-attribute
-Wno-initializer-overrides -Wno-missing-include-dirs
-Wno-shift-negative-value -Wno-string-plus-int
-Wno-typedef-redefinition -Wno-tautological-type-limit-compare
-Wno-psabi -Wno-gnu-variable-sized-type-not-at-end -Wthread-safety
-isystem /builds/qemu-project/qemu/linux-headers -isystem
linux-headers -iquote . -iquote /builds/qemu-project/qemu -iquote
/builds/qemu-project/qemu/include -iquote
/builds/qemu-project/qemu/host/include/x86_64 -iquote
/builds/qemu-project/qemu/host/include/generic -iquote
/builds/qemu-project/qemu/tcg/i386 -pthread -D_GNU_SOURCE
-D_LARGEFILE_SOURCE -fno-strict-aliasing -fno-common -fwrapv
-fsanitize=cfi-icall -fsanitize-cfi-icall-generalize-pointers
-fno-sanitize-trap=cfi-icall -fPIE -D_FILE_OFFSET_BITS=64
-D__USE_FILE_OFFSET64 -D__USE_LARGEFILE64 -DUSE_POSIX_ACLS=1 -MD -MQ
libblock.fa.p/block_parallels.c.o -MF
libblock.fa.p/block_parallels.c.o.d -o
libblock.fa.p/block_parallels.c.o -c ../block/parallels.c
../block/parallels.c:210:21: error: calling function
'bdrv_co_getlength' requires holding mutex 'graph_lock'
[-Werror,-Wthread-safety-analysis]
payload_bytes = bdrv_co_getlength(bs->file->bs);
^
../block/parallels.c:572:15: error: calling function
'bdrv_co_pdiscard' requires holding mutex 'graph_lock'
[-Werror,-Wthread-safety-analysis]
ret = bdrv_co_pdiscard(bs->file, host_off, s->cluster_size);
^
2 errors generated.
https://gitlab.com/qemu-project/qemu/-/jobs/5131277794
Stefan
>
> for you to fetch changes up to ead1064587ba6534aa2c3da6383713a009dafcb1:
>
> tests: extend test 131 to cover availability of the write-zeroes (2023-09-20 10:14:15 +0200)
>
> ----------------------------------------------------------------
> Parallels format driver:
> * regular calculation of cluster used bitmap of the image file
> * cluster allocation on the base of that bitmap (effectively allocation of
> new clusters could be done inside the image if that offset space is unused)
> * support of DISCARD and WRITE_ZEROES operations
> * image check bugfixes
> * unit tests fixes
> * unit tests covering new functionality
>
> ----------------------------------------------------------------
> Denis V. Lunev (22):
> parallels: fix formatting in bdrv_parallels initialization
> parallels: mark driver as supporting CBT
> parallels: fix memory leak in parallels_open()
> parallels: invent parallels_opts_prealloc() helper to parse prealloc opts
> parallels: return earler in fail_format branch in parallels_open()
> parallels: return earlier from parallels_open() function on error
> parallels: refactor path when we need to re-check image in parallels_open
> parallels: create mark_used() helper which sets bit in used bitmap
> tests: ensure that image validation will not cure the corruption
> parallels: fix broken parallels_check_data_off()
> parallels: add test which will validate data_off fixes through repair
> parallels: collect bitmap of used clusters at open
> tests: fix broken deduplication check in parallels format test
> tests: test self-cure of parallels image with duplicated clusters
> parallels: accept multiple clusters in mark_used()
> parallels: update used bitmap in allocate_cluster
> parallels: naive implementation of allocate_clusters with used bitmap
> parallels: improve readability of allocate_clusters
> parallels: naive implementation of parallels_co_pdiscard
> tests: extend test 131 to cover availability of the discard operation
> parallels: naive implementation of parallels_co_pwrite_zeroes
> tests: extend test 131 to cover availability of the write-zeroes
>
> block/parallels.c | 389 ++++++++++++++++++++------
> block/parallels.h | 3 +
> tests/qemu-iotests/131 | 52 ++++
> tests/qemu-iotests/131.out | 60 ++++
> tests/qemu-iotests/tests/parallels-checks | 76 ++++-
> tests/qemu-iotests/tests/parallels-checks.out | 65 ++++-
> 6 files changed, 544 insertions(+), 101 deletions(-)
>
> --
> 2.34.1
>
>