mbox

[PULL,00/22] Block layer patches

Message ID 20230217150829.283583-1-kwolf@redhat.com
State New
Headers show

Pull-request

https://repo.or.cz/qemu/kevin.git tags/for-upstream

Message

Kevin Wolf Feb. 17, 2023, 3:08 p.m. UTC
The following changes since commit 6dffbe36af79e26a4d23f94a9a1c1201de99c261:

  Merge tag 'migration-20230215-pull-request' of https://gitlab.com/juan.quintela/qemu into staging (2023-02-16 13:09:51 +0000)

are available in the Git repository at:

  https://repo.or.cz/qemu/kevin.git tags/for-upstream

for you to fetch changes up to a4d5224c2cb650b5a401d626d3f36e42e6987aa7:

  hbitmap: fix hbitmap_status() return value for first dirty bit case (2023-02-17 14:34:24 +0100)

----------------------------------------------------------------
Block layer patches

- configure: Enable -Wthread-safety if present
- no_co_wrapper to fix bdrv_open*() calls from coroutine context
- curl fixes, including enablement of newer libcurl versions
- MAINTAINERS: drop Vladimir from parallels block driver
- hbitmap: fix hbitmap_status() return value for first dirty bit case
- file-posix: Fix assertion failure in write_zeroes after moving
  bdrv_getlength() to co_wrapper

----------------------------------------------------------------
Andrey Zhadchenko (1):
      hbitmap: fix hbitmap_status() return value for first dirty bit case

Anton Johansson (1):
      block: Handle curl 7.55.0, 7.85.0 version changes

Emanuele Giuseppe Esposito (3):
      util/qemu-thread-posix: use TSA_NO_TSA to suppress clang TSA warnings in FreeBSD
      bsd-user/mmap: use TSA_NO_TSA to suppress clang TSA warnings in FreeBSD
      block/file-posix: don't use functions calling AIO_WAIT_WHILE in worker threads

Hanna Czenczek (1):
      curl: Fix error path in curl_open()

Kevin Wolf (14):
      configure: Enable -Wthread-safety if present
      block-coroutine-wrapper: Introduce no_co_wrapper
      block: Create no_co_wrappers for open functions
      luks: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      parallels: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      qcow: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      qcow2: Fix open/create to open images with no_co_wrapper
      qed: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      vdi: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      vhdx: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      vmdk: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      vpc: Fix .bdrv_co_create(_opts) to open images with no_co_wrapper
      block: Fix bdrv_co_create_opts_simple() to open images with no_co_wrapper
      block: Assert non-coroutine context for bdrv_open_inherit()

Stefano Garzarella (1):
      block: temporarily hold the new AioContext of bs_top in bdrv_append()

Vladimir Sementsov-Ogievskiy (1):
      MAINTAINERS: drop Vladimir from parallels block driver

 configure                                   |  1 +
 bsd-user/qemu.h                             |  5 +-
 include/block/block-common.h                | 14 +++++
 include/block/block-global-state.h          | 35 ++++++++----
 include/exec/exec-all.h                     |  5 +-
 include/qemu/hbitmap.h                      |  2 +-
 include/qemu/thread.h                       | 14 +++--
 include/sysemu/block-backend-global-state.h | 21 ++++++--
 block.c                                     | 40 ++++++++++----
 block/crypto.c                              | 19 ++++---
 block/curl.c                                | 50 +++++++++++++----
 block/file-posix.c                          |  2 +-
 block/parallels.c                           | 10 ++--
 block/qcow.c                                | 10 ++--
 block/qcow2.c                               | 43 +++++++--------
 block/qed.c                                 | 10 ++--
 block/vdi.c                                 | 10 ++--
 block/vhdx.c                                | 10 ++--
 block/vmdk.c                                | 22 ++++----
 block/vpc.c                                 | 10 ++--
 util/hbitmap.c                              |  2 +-
 util/qemu-thread-posix.c                    |  2 +-
 scripts/block-coroutine-wrapper.py          | 83 +++++++++++++++++++++++++----
 MAINTAINERS                                 |  2 -
 block/meson.build                           |  1 +
 25 files changed, 295 insertions(+), 128 deletions(-)

Comments

Peter Maydell Feb. 20, 2023, 5:01 p.m. UTC | #1
On Fri, 17 Feb 2023 at 15:08, Kevin Wolf <kwolf@redhat.com> wrote:
>
> The following changes since commit 6dffbe36af79e26a4d23f94a9a1c1201de99c261:
>
>   Merge tag 'migration-20230215-pull-request' of https://gitlab.com/juan.quintela/qemu into staging (2023-02-16 13:09:51 +0000)
>
> are available in the Git repository at:
>
>   https://repo.or.cz/qemu/kevin.git tags/for-upstream
>
> for you to fetch changes up to a4d5224c2cb650b5a401d626d3f36e42e6987aa7:
>
>   hbitmap: fix hbitmap_status() return value for first dirty bit case (2023-02-17 14:34:24 +0100)
>
> ----------------------------------------------------------------
> Block layer patches
>
> - configure: Enable -Wthread-safety if present
> - no_co_wrapper to fix bdrv_open*() calls from coroutine context
> - curl fixes, including enablement of newer libcurl versions
> - MAINTAINERS: drop Vladimir from parallels block driver
> - hbitmap: fix hbitmap_status() return value for first dirty bit case
> - file-posix: Fix assertion failure in write_zeroes after moving
>   bdrv_getlength() to co_wrapper
>

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

thanks
-- PMM
Philippe Mathieu-Daudé Feb. 20, 2023, 7:02 p.m. UTC | #2
On 20/2/23 18:01, Peter Maydell wrote:
> On Fri, 17 Feb 2023 at 15:08, Kevin Wolf <kwolf@redhat.com> wrote:
>>
>> The following changes since commit 6dffbe36af79e26a4d23f94a9a1c1201de99c261:
>>
>>    Merge tag 'migration-20230215-pull-request' of https://gitlab.com/juan.quintela/qemu into staging (2023-02-16 13:09:51 +0000)
>>
>> are available in the Git repository at:
>>
>>    https://repo.or.cz/qemu/kevin.git tags/for-upstream
>>
>> for you to fetch changes up to a4d5224c2cb650b5a401d626d3f36e42e6987aa7:
>>
>>    hbitmap: fix hbitmap_status() return value for first dirty bit case (2023-02-17 14:34:24 +0100)
>>
>> ----------------------------------------------------------------
>> Block layer patches
>>
>> - configure: Enable -Wthread-safety if present
>> - no_co_wrapper to fix bdrv_open*() calls from coroutine context
>> - curl fixes, including enablement of newer libcurl versions
>> - MAINTAINERS: drop Vladimir from parallels block driver
>> - hbitmap: fix hbitmap_status() return value for first dirty bit case
>> - file-posix: Fix assertion failure in write_zeroes after moving
>>    bdrv_getlength() to co_wrapper
>>
> 
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

Applied as commit 2d89cb1fe5 ;)
Peter Maydell Feb. 21, 2023, 11:10 a.m. UTC | #3
On Mon, 20 Feb 2023 at 19:02, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> On 20/2/23 18:01, Peter Maydell wrote:
> > On Fri, 17 Feb 2023 at 15:08, Kevin Wolf <kwolf@redhat.com> wrote:
> >>
> >> The following changes since commit 6dffbe36af79e26a4d23f94a9a1c1201de99c261:
> >>
> >>    Merge tag 'migration-20230215-pull-request' of https://gitlab.com/juan.quintela/qemu into staging (2023-02-16 13:09:51 +0000)
> >>
> >> are available in the Git repository at:
> >>
> >>    https://repo.or.cz/qemu/kevin.git tags/for-upstream
> >>
> >> for you to fetch changes up to a4d5224c2cb650b5a401d626d3f36e42e6987aa7:
> >>
> >>    hbitmap: fix hbitmap_status() return value for first dirty bit case (2023-02-17 14:34:24 +0100)
> >>
> >> ----------------------------------------------------------------
> >> Block layer patches
> >>
> >> - configure: Enable -Wthread-safety if present
> >> - no_co_wrapper to fix bdrv_open*() calls from coroutine context
> >> - curl fixes, including enablement of newer libcurl versions
> >> - MAINTAINERS: drop Vladimir from parallels block driver
> >> - hbitmap: fix hbitmap_status() return value for first dirty bit case
> >> - file-posix: Fix assertion failure in write_zeroes after moving
> >>    bdrv_getlength() to co_wrapper
> >>
> >
> > Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
>
> Applied as commit 2d89cb1fe5 ;)

Whoops, yes, wrong canned-email. Try this one:


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/8.0
for any user-visible changes.

-- PMM