Re: [PULL v2 00/40] Misc patches for 2023-08-31

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PATCH v2 11/12] hw/vmapple/virtio-blk: Add support for apple virtio-blk

nd. > > It then exposes 2 subclasses from that that we can use to expose root and > aux virtio-blk devices: "vmapple-virtio-root" and "vmapple-virtio-aux". > > Signed-off-by: Alexander Graf Aside from my comments below: Reviewed-by: Stefan Hajnoczi >

Re: [PATCH v2 08/12] hw/vmapple/bdif: Introduce vmapple backdoor interface

parses a boot record and then loads the next stage... I posted comments below. Otherwise: Reviewed-by: Stefan Hajnoczi > > Signed-off-by: Alexander Graf > > --- > > v1 -> v2: > > - Adapt to system_ss meson.build target > --- > include/hw/vmapple

Re: [PATCH 4/7] block/dirty-bitmap: Clean up local variable shadowing

else rename variables. > > Signed-off-by: Markus Armbruster > --- > block/monitor/bitmap-qmp-cmds.c | 2 +- > block/qcow2-bitmap.c| 3 +-- > 2 files changed, 2 insertions(+), 3 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [PATCH 5/7] block/vdi: Clean up local variable shadowing

else rename variables. > > Signed-off-by: Markus Armbruster > --- > block/vdi.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [PATCH 6/7] block: Clean up local variable shadowing

5 files changed, 25 insertions(+), 25 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [PULL 0/1] Quick fix patches

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 00/24] target-arm queue

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL v2 0/2] xen-virtio-2-tag

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 00/12] testing and gdbstub updates

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

[PATCH v3 4/4] io: follow coroutine AioContext in qio_channel_yield()

the iohandler AioHandler. Code that formerly called qio_channel_attach_aio_context() now calls qio_channel_set_follow_coroutine_ctx(ioc, true) once after the QIOChannel is created. Signed-off-by: Stefan Hajnoczi Reviewed-by: Eric Blake Acked-by: Daniel P. Berrangé --- include/io/channel-util.h

[PATCH v3 3/4] io: check there are no qio_channel_yield() coroutines during ->finalize()

Callers must clean up their coroutines before calling object_unref(OBJECT(ioc)) to prevent an fd handler leak. Add an assertion to check this. This patch is preparation for the fd handler changes that follow. Signed-off-by: Stefan Hajnoczi Reviewed-by: Daniel P. Berrangé Reviewed-by: Eric

[PATCH v3 1/4] nbd: drop unused nbd_receive_negotiate() aio_context argument

aio_context is always NULL, so drop it. Suggested-by: Fabiano Rosas Signed-off-by: Stefan Hajnoczi Reviewed-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé --- include/block/nbd.h | 3 +-- nbd/client-connection.c | 3 +-- nbd/client.c| 5 ++--- qemu-nbd.c | 4

[PATCH v3 2/4] nbd: drop unused nbd_start_negotiate() aio_context argument

aio_context is always NULL, so drop it. Suggested-by: Fabiano Rosas Signed-off-by: Stefan Hajnoczi Reviewed-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé --- nbd/client.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/nbd/client.c b/nbd/client.c index

[PATCH v3 0/4] io: follow coroutine AioContext in qio_channel_yield()

oroutines running in multiple threads. This series changes the QIOChannel API to make that possible. Stefan Hajnoczi (4): nbd: drop unused nbd_receive_negotiate() aio_context argument nbd: drop unused nbd_start_negotiate() aio_context argument io: check there are no qio_channel_yield() coroutines

Re: [PULL v3 0/5] Block patches

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PATCH v2 0/5] tcg patch queue

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 0/3] Dirty page rate and dirty page limit 20230829 patches

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PATCH 7/7] vhost-user: call VHOST_USER_SET_VRING_ENABLE synchronously

On Wed, 30 Aug 2023 at 09:30, Laszlo Ersek wrote: > > On 8/30/23 14:10, Stefan Hajnoczi wrote: > > On Sun, 27 Aug 2023 at 14:31, Laszlo Ersek wrote: > >> > >> (1) The virtio-1.0 specification > >> <http://docs.oasis-open.org/virtio/virtio/v1.0/virtio-v

Re: [PULL 0/1] Quick fix patches

Hi, The patch introduces the following build failure: cc -m64 -mcx16 -Isubprojects/libvhost-user/libvhost-user.a.p -Isubprojects/libvhost-user -I../subprojects/libvhost-user -fdiagnostics-color=auto -Wall -Winvalid-pch -Werror -std=gnu99 -O2 -g -Wsign-compare -Wdeclaration-after-statement

Re: [PATCH 7/7] vhost-user: call VHOST_USER_SET_VRING_ENABLE synchronously

On Sun, 27 Aug 2023 at 14:31, Laszlo Ersek wrote: > > (1) The virtio-1.0 specification > writes: > > > 3 General Initialization And Device Operation > > 3.1 Device Initialization > > 3.1.1 Driver Requirements: Device

[PULL v3 4/5] tests/qemu-iotests/197: add testcase for CoR with subclusters

: ../block/io.c:1236: bdrv_co_do_copy_on_readv: Assertion `skip_bytes < pnum' failed. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID: <20230711172553.234055-4-andrey.

[PULL v3 1/5] block-migration: Ensure we don't crash during migration cleanup

0) at ../block.c:7095 #2 0x55bbae13 in block_migration_cleanup_bmds () at ../migration/block.c:690 Signed-off-by: Fabiano Rosas Message-id: 20230731203338.27581-1-faro...@suse.de Signed-off-by: Stefan Hajnoczi --- migration/block.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(

[PULL v3 5/5] aio-posix: zero out io_uring sqe user_data

mon-io_uring in the future (and I'm working on that). Signed-off-by: Stefan Hajnoczi Message-ID: <20230426212639.82310-1-stefa...@redhat.com> --- util/fdmon-io_uring.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/util/fdmon-io_uring.c b/util/fdmon-io_uring.c index 17ec18b7bd..16054c5e

[PULL v3 2/5] block: add subcluster_size field to BlockDriverInfo

() implementation, we simply set it equal to the cluster size thus treating each cluster as having a single subcluster. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID

[PULL v3 0/5] Block patches

o Rosas (1): block-migration: Ensure we don't crash during migration cleanup Stefan Hajnoczi (1): aio-posix: zero out io_uring sqe user_data include/block/block-common.h | 5 include/block/block-io.h | 8 +++--- block.c | 7 + block/io.c

[PULL v3 3/5] block/io: align requests to subcluster_size

ssertion `skip_bytes < pnum' failed. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID: <20230711172553.234055-3-andrey.drobys...@virtuozzo.com> --- include/block/b

Re: [PATCH v9 0/4] hw/ufs: Add Universal Flash Storage (UFS) support

On Thu, 3 Aug 2023 at 07:49, Jeuk Kim wrote: > > Dear Stefan, > I'm really sorry, but could you please put this patch series > instead of v8, which was previously merged into block-next? > The fixes from v8 are below. > Please let me know if you have any comments or issues. The CI hit a test

Re: [PATCH v9 0/4] hw/ufs: Add Universal Flash Storage (UFS) support

On Thu, 3 Aug 2023 at 07:49, Jeuk Kim wrote: > > Dear Stefan, > I'm really sorry, but could you please put this patch series > instead of v8, which was previously merged into block-next? > The fixes from v8 are below. > Please let me know if you have any comments or issues. Hi, This series fails

Re: [PATCH v2 4/4] io: follow coroutine AioContext in qio_channel_yield()

On Tue, Aug 29, 2023 at 02:32:46PM -0500, Eric Blake wrote: > On Tue, Aug 29, 2023 at 12:06:22PM -0400, Stefan Hajnoczi wrote: > > The ongoing QEMU multi-queue block layer effort makes it possible for > > multiple > > threads to process I/O in parallel. The nbd block dri

Re: [PULL 0/3] Dirty page rate and dirty page limit 20230828 patches

On Tue, 29 Aug 2023 at 12:30, Yong Huang wrote: > On Tue, Aug 29, 2023 at 4:01 AM Stefan Hajnoczi wrote: >> >> On Mon, 28 Aug 2023 at 10:36, Hyman Huang wrote: >> > >> > From: Hyman >> > >> > The following changes since commit >> >

[PULL v2 8/8] tests/qemu-iotests/197: add testcase for CoR with subclusters

: ../block/io.c:1236: bdrv_co_do_copy_on_readv: Assertion `skip_bytes < pnum' failed. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID: <20230711172553.234055-4-andrey.

[PULL v2 7/8] block/io: align requests to subcluster_size

ssertion `skip_bytes < pnum' failed. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID: <20230711172553.234055-3-andrey.drobys...@virtuozzo.com> --- include/block/b

[PULL v2 1/8] hw/ufs: Initial commit for emulated Universal-Flash-Storage

to handle UFS logical unit and Transfer Request will follow. Signed-off-by: Jeuk Kim Reviewed-by: Stefan Hajnoczi Message-id: 9f3db32fe1c708090a6bb764d456973b5abef55f.1691062912.git.jeuk20@samsung.com Signed-off-by: Stefan Hajnoczi --- MAINTAINERS |6 + docs/specs/pci-ids.rst

[PULL v2 2/8] hw/ufs: Support for Query Transfer Requests

From: Jeuk Kim This commit makes the UFS device support query and nop out transfer requests. The next patch would be support for UFS logical unit and scsi command transfer request. Signed-off-by: Jeuk Kim Reviewed-by: Stefan Hajnoczi Message-id

[PULL v2 4/8] tests/qtest: Introduce tests for UFS

From: Jeuk Kim This patch includes the following tests Test mmio read Test ufs device initialization and ufs-lu recognition Test I/O (Performs a write followed by a read to verify) Signed-off-by: Jeuk Kim Acked-by: Thomas Huth Reviewed-by: Stefan Hajnoczi Message-id

[PULL v2 6/8] block: add subcluster_size field to BlockDriverInfo

() implementation, we simply set it equal to the cluster size thus treating each cluster as having a single subcluster. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID

[PULL v2 5/8] block-migration: Ensure we don't crash during migration cleanup

0) at ../block.c:7095 #2 0x55bbae13 in block_migration_cleanup_bmds () at ../migration/block.c:690 Signed-off-by: Fabiano Rosas Message-id: 20230731203338.27581-1-faro...@suse.de Signed-off-by: Stefan Hajnoczi --- migration/block.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(

[PULL v2 0/8] Block patches

The following changes since commit 813bac3d8d70d85cb7835f7945eb9eed84c2d8d0: Merge tag '2023q3-bsd-user-pull-request' of https://gitlab.com/bsdimp/qemu into staging (2023-08-29 08:58:00 -0400) are available in the Git repository at: https://gitlab.com/stefanha/qemu.git

[PULL v2 3/8] hw/ufs: Support for UFS logical unit

From: Jeuk Kim This commit adds support for ufs logical unit. The LU handles processing for the SCSI command, unit descriptor query request. This commit enables the UFS device to process IO requests. Signed-off-by: Jeuk Kim Reviewed-by: Stefan Hajnoczi Message-id

[PATCH v2 4/4] io: follow coroutine AioContext in qio_channel_yield()

the iohandler AioHandler. Code that formerly called qio_channel_attach_aio_context() now calls qio_channel_set_follow_coroutine_ctx(ioc, true) once after the QIOChannel is created. Signed-off-by: Stefan Hajnoczi --- include/io/channel-util.h| 23 ++ include/io/channel.h

[PATCH v2 1/4] nbd: drop unused nbd_receive_negotiate() aio_context argument

aio_context is always NULL, so drop it. Suggested-by: Fabiano Rosas Signed-off-by: Stefan Hajnoczi --- include/block/nbd.h | 3 +-- nbd/client-connection.c | 3 +-- nbd/client.c| 5 ++--- qemu-nbd.c | 4 ++-- 4 files changed, 6 insertions(+), 9 deletions(-) diff

[PATCH v2 2/4] nbd: drop unused nbd_start_negotiate() aio_context argument

aio_context is always NULL, so drop it. Suggested-by: Fabiano Rosas Signed-off-by: Stefan Hajnoczi --- nbd/client.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/nbd/client.c b/nbd/client.c index 16ec10c8a9..bd7e200136 100644 --- a/nbd/client.c +++ b/nbd

[PATCH v2 3/4] io: check there are no qio_channel_yield() coroutines during ->finalize()

Callers must clean up their coroutines before calling object_unref(OBJECT(ioc)) to prevent an fd handler leak. Add an assertion to check this. This patch is preparation for the fd handler changes that follow. Signed-off-by: Stefan Hajnoczi Reviewed-by: Daniel P. Berrangé Reviewed-by: Eric

[PATCH v2 0/4] io: follow coroutine AioContext in qio_channel_yield()

ltiple threads to process I/O in parallel. The nbd block driver is not compatible with the multi-queue block layer yet because QIOChannel cannot be used easily from coroutines running in multiple threads. This series changes the QIOChannel API to make that possible. Stefan Hajnoczi (4): nb

Re: [PULL 00/36] 2023q3 bsd user patches

Applied, thanks. In the future, please invoke git-request-pull(1) with the public HTTPS repo URL to make it easier for anyone to fetch the changes: > are available in the Git repository at: > > g...@gitlab.com:bsdimp/qemu.git tags/2023q3-bsd-user-pull-request ^^ Please update the

Re: [PULL 00/14] Python, i386 changes for 2023-08-28

Hi Paolo, I see a new test error that may have been introduced by this pull request: https://gitlab.com/qemu-project/qemu/-/jobs/4968468877#L131 AVOCADO Downloading avocado tests VM image for s390x Failed to load plugin from module "avocado.plugins.journal": ImportError("Module 'sqlite3' is not

Re: [PATCH v2 00/48] tcg patch queue

On Thu, 24 Aug 2023 at 14:29, Richard Henderson wrote: > > The following changes since commit 50e7a40af372ee5931c99ef7390f5d3d6fbf6ec4: > > Merge tag 'pull-target-arm-20230824' of > https://git.linaro.org/people/pmaydell/qemu-arm into staging (2023-08-24 > 10:08:33 -0400) > > are available in

Re: [PULL 0/5] Devel hppa priv cleanup2 patches

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 00/14] Python, i386 changes for 2023-08-28

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PATCH v2 00/48] tcg patch queue

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 0/3] Dirty page rate and dirty page limit 20230828 patches

I see you sent a second pull request that includes the emails. Please send a v3 with a signed tag. Thanks! Stefan

Re: [PULL 0/3] Dirty page rate and dirty page limit 20230828 patches

On Mon, 28 Aug 2023 at 10:36, Hyman Huang wrote: > > From: Hyman > > The following changes since commit 50e7a40af372ee5931c99ef7390f5d3d6fbf6ec4: > > Merge tag 'pull-target-arm-20230824' of > https://git.linaro.org/people/pmaydell/qemu-arm into staging (2023-08-24 > 10:08:33 -0400) > > are

Re: [PATCH 2/2] io: follow coroutine AioContext in qio_channel_yield()

On Thu, Aug 24, 2023 at 12:26:05PM +0100, Daniel P. Berrangé wrote: > On Wed, Aug 23, 2023 at 07:45:04PM -0400, Stefan Hajnoczi wrote: > > The ongoing QEMU multi-queue block layer effort makes it possible for > > multiple > > threads to process I/O in parallel.

Re: [PATCH 2/2] io: follow coroutine AioContext in qio_channel_yield()

On Thu, Aug 24, 2023 at 01:09:59PM -0300, Fabiano Rosas wrote: > Stefan Hajnoczi writes: > > > @@ -2089,10 +2088,6 @@ static void nbd_attach_aio_context(BlockDriverState > > *bs, > > * the reconnect_delay_timer cannot be active here. > > */ > >

Re: [PATCH v2 0/3] block: align CoR requests to subclusters

On Thu, Aug 24, 2023 at 06:47:20PM +0300, Andrey Drobyshev wrote: > On 8/24/23 17:32, Stefan Hajnoczi wrote: > > On Wed, Aug 23, 2023 at 03:50:55PM +0300, Andrey Drobyshev wrote: > >> On 8/22/23 22:58, John Snow wrote: > >>> On Tue, Aug 22, 2023 at 1:33 PM

Re: [PATCH] hw/block/dataplane/virtio-block: Avoid dynamic stack allocation

/block/dataplane/virtio-blk.c | 11 --- > 1 file changed, 8 insertions(+), 3 deletions(-) Hi Peter, I recently sent a patch series that removes notify_guest_bh() completely: https://lore.kernel.org/qemu-devel/20230817155847.3605115-5-stefa...@redhat.com/ If it's urgent we can merge you

Re: [PATCH 2/2] io: follow coroutine AioContext in qio_channel_yield()

On Thu, Aug 24, 2023 at 12:26:05PM +0100, Daniel P. Berrangé wrote: > On Wed, Aug 23, 2023 at 07:45:04PM -0400, Stefan Hajnoczi wrote: > > The ongoing QEMU multi-queue block layer effort makes it possible for > > multiple > > threads to process I/O in parallel.

Re: [PATCH] aio-posix: zero out io_uring sqe user_data

On Wed, Apr 26, 2023 at 05:26:39PM -0400, Stefan Hajnoczi wrote: > liburing does not clear sqe->user_data. We must do it ourselves to avoid > undefined behavior in process_cqe() when user_data is used. > > Note that fdmon-io_uring is currently disabled, so this is a latent

[PULL 8/8] tests/qemu-iotests/197: add testcase for CoR with subclusters

-io: ../block/io.c:1236: bdrv_co_do_copy_on_readv: Assertion `skip_bytes < pnum' failed. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID: <20230711172553.23

[PULL 5/8] block-migration: Ensure we don't crash during migration cleanup

0) at ../block.c:7095 #2 0x55bbae13 in block_migration_cleanup_bmds () at ../migration/block.c:690 Signed-off-by: Fabiano Rosas Message-id: 20230731203338.27581-1-faro...@suse.de Signed-off-by: Stefan Hajnoczi --- migration/block.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(

[PULL 7/8] block/io: align requests to subcluster_size

_on_readv: Assertion `skip_bytes < pnum' failed. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID: <20230711172553.234055-3-andrey.drobys...@virtuozzo.com> ---

[PULL 3/8] hw/ufs: Support for UFS logical unit

From: Jeuk Kim This commit adds support for ufs logical unit. The LU handles processing for the SCSI command, unit descriptor query request. This commit enables the UFS device to process IO requests. Signed-off-by: Jeuk Kim Reviewed-by: Stefan Hajnoczi Message-id

[PULL 6/8] block: add subcluster_size field to BlockDriverInfo

() implementation, we simply set it equal to the cluster size thus treating each cluster as having a single subcluster. Reviewed-by: Eric Blake Reviewed-by: Denis V. Lunev Signed-off-by: Andrey Drobyshev Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Stefan Hajnoczi Message-ID

[PULL 4/8] tests/qtest: Introduce tests for UFS

From: Jeuk Kim This patch includes the following tests Test mmio read Test ufs device initialization and ufs-lu recognition Test I/O (Performs a write followed by a read to verify) Signed-off-by: Jeuk Kim Acked-by: Thomas Huth Reviewed-by: Stefan Hajnoczi Message-id

[PULL 2/8] hw/ufs: Support for Query Transfer Requests

From: Jeuk Kim This commit makes the UFS device support query and nop out transfer requests. The next patch would be support for UFS logical unit and scsi command transfer request. Signed-off-by: Jeuk Kim Reviewed-by: Stefan Hajnoczi Message-id

[PULL 0/8] Block patches

The following changes since commit b0dd9a7d6dd15a6898e9c585b521e6bec79b25aa: Open 8.2 development tree (2023-08-22 07:14:07 -0700) are available in the Git repository at: https://gitlab.com/stefanha/qemu.git tags/block-pull-request for you to fetch changes up to

[PULL 1/8] hw/ufs: Initial commit for emulated Universal-Flash-Storage

to handle UFS logical unit and Transfer Request will follow. Signed-off-by: Jeuk Kim Reviewed-by: Stefan Hajnoczi Message-id: 9f3db32fe1c708090a6bb764d456973b5abef55f.1691062912.git.jeuk20@samsung.com Signed-off-by: Stefan Hajnoczi --- MAINTAINERS |6 + docs/specs/pci-ids.rst

Re: [PULL 00/35] target-arm queue

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 00/31] loongarch-to-apply queue

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 00/12] First batch of s390x patches for QEMU 8.2

On Thu, 24 Aug 2023 at 02:53, Thomas Huth wrote: > > On 23/08/2023 18.34, Stefan Hajnoczi wrote: > > On Wed, Aug 23, 2023 at 01:45:32PM +0200, Thomas Huth wrote: > >> The following changes since commit > >> b0dd9a7d6dd15a6898e9c585b521e6bec79b25aa: > >> >

Re: [PATCH] .gitlab-ci.d/cirrus.yml: Update FreeBSD to v13.2

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PATCH v2 0/3] block: align CoR requests to subclusters

On Wed, Aug 23, 2023 at 03:50:55PM +0300, Andrey Drobyshev wrote: > On 8/22/23 22:58, John Snow wrote: > > On Tue, Aug 22, 2023 at 1:33 PM Andrey Drobyshev > > wrote: > >> > >> On 8/16/23 12:22, Andrey Drobyshev wrote: > >>> On 7/31/23 17:51, Andrey Drobyshev wrote: > On 7/24/23 16:11,

Re: [PULL 00/48] tcg patch queue

On Wed, 23 Aug 2023 at 16:24, Richard Henderson wrote: > > The following changes since commit b0dd9a7d6dd15a6898e9c585b521e6bec79b25aa: > > Open 8.2 development tree (2023-08-22 07:14:07 -0700) > > are available in the Git repository at: > > https://gitlab.com/rth7680/qemu.git

Re: [PATCH v2 1/4] softmmu: Support concurrent bounce buffers

On Wed, Aug 23, 2023 at 04:54:06PM -0400, Peter Xu wrote: > On Wed, Aug 23, 2023 at 10:08:08PM +0200, Mattias Nissler wrote: > > On Wed, Aug 23, 2023 at 7:35 PM Peter Xu wrote: > > > On Wed, Aug 23, 2023 at 02:29:02AM -0700, Mattias Nissler wrote: > > > > diff --git a/softmmu/vl.c b/softmmu/vl.c

Re: Starting the QEMU 8.2 release cycle

On Thu, 24 Aug 2023 at 05:25, Peter Maydell wrote: > > On Wed, 23 Aug 2023 at 12:23, Stefan Hajnoczi wrote: > > > > Hi, > > The QEMU 8.2 release cycle has begun. I am merging qemu.git pull > > requests for this release. You do not need to CC me on pull requests &g

[PATCH v2 1/4] block: remove AIOCBInfo->get_aio_context()

its blk_aio_cancel() caller, and poll the main loop AioContext. The purpose of this cleanup is to identify bdrv_aio_cancel() as an API that does not work with the multi-queue block layer. Signed-off-by: Stefan Hajnoczi --- include/block/aio.h| 1 - include/block/block-global

[PATCH v2 4/4] block-coroutine-wrapper: use qemu_get_current_aio_context()

is only called with the BQL held from the main loop thread. The output of qemu-iotests 051 is sensitive to event loop activity. Update the output because the monitor BH runs at a different time, causing prompts to be printed differently in the output. Signed-off-by: Stefan Hajnoczi --- scripts

[PATCH v2 0/4] block-backend: process I/O in the current AioContext

oop thread in addition to the IOThread: https://lore.kernel.org/qemu-devel/20230823234504.1387239-1-stefa...@redhat.com/T/#t Based-on: 20230823234504.1387239-1-stefa...@redhat.com Stefan Hajnoczi (4): block: remove AIOCBInfo->get_aio_context() block-backend: process I/O in the current AioC

[PATCH v2 3/4] block-backend: process zoned requests in the current AioContext

Process zoned requests in the current thread's AioContext instead of in the BlockBackend's AioContext. There is no need to use the BlockBackend's AioContext thanks to CoMutex bs->wps->colock, which protects zone metadata. Signed-off-by: Stefan Hajnoczi --- block/block-backend.

[PATCH v2 2/4] block-backend: process I/O in the current AioContext

accessed by the completion callback. - The remaining emulated devices and exports run with qemu_get_aio_context() == blk_get_aio_context(). Signed-off-by: Stefan Hajnoczi --- block/block-backend.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/block/block-backend.c b

[PATCH 2/2] io: follow coroutine AioContext in qio_channel_yield()

the iohandler AioHandler. Code that formerly called qio_channel_attach_aio_context() now calls qio_channel_set_follow_coroutine_ctx(ioc, true) once after the QIOChannel is created. Signed-off-by: Stefan Hajnoczi --- include/io/channel.h | 34 +++-- include/qemu/vhost-user-server.h

[PATCH 0/2] io: follow coroutine AioContext in qio_channel_yield()

the QIOChannel API to make that possible. Stefan Hajnoczi (2): io: check there are no qio_channel_yield() coroutines during ->finalize() io: follow coroutine AioContext in qio_channel_yield() include/io/channel.h | 34 - include/qemu/vhost-user-server.h | 1 + bl

[PATCH 1/2] io: check there are no qio_channel_yield() coroutines during ->finalize()

Callers must clean up their coroutines before calling object_unref(OBJECT(ioc)) to prevent an fd handler leak. Add an assertion to check this. This patch is preparation for the fd handler changes that follow. Signed-off-by: Stefan Hajnoczi --- io/channel.c | 4 1 file changed, 4

Re: [PULL 00/12] First batch of s390x patches for QEMU 8.2

On Wed, Aug 23, 2023 at 01:45:32PM +0200, Thomas Huth wrote: > The following changes since commit b0dd9a7d6dd15a6898e9c585b521e6bec79b25aa: > > Open 8.2 development tree (2023-08-22 07:14:07 -0700) > > are available in the Git repository at: > > https://gitlab.com/thuth/qemu.git

Re: NVMe ZNS last zone size

On Wed, 23 Aug 2023 at 14:53, Klaus Jensen wrote: > > On Aug 23 22:58, Sam Li wrote: > > Stefan Hajnoczi 于2023年8月23日周三 22:41写道： > > > > > > On Wed, 23 Aug 2023 at 10:24, Sam Li wrote: > > > > > > > > Hi Stefan, > > > > > >

Re: NVMe ZNS last zone size

On Wed, 23 Aug 2023 at 10:24, Sam Li wrote: > > Hi Stefan, > > Stefan Hajnoczi 于2023年8月23日周三 21:26写道： > > > > Hi Sam and Klaus, > > Val is adding nvme-io_uring ZNS support to libblkio > > (https://gitlab.com/libblkio/libblkio/-/merge_requests/221) and asked

NVMe ZNS last zone size

Hi Sam and Klaus, Val is adding nvme-io_uring ZNS support to libblkio (https://gitlab.com/libblkio/libblkio/-/merge_requests/221) and asked how to test the size of the last zone when the namespace's total size is not a multiple of the zone size. My understanding is that the zoned storage model

Re: [PULL 00/12] First batch of s390x patches for QEMU 8.2

Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PATCH v2] docs/about/license: Update LICENSE URL

On Tue, Aug 22, 2023 at 02:57:16PM +0200, Philippe Mathieu-Daudé wrote: > In early 2021 (see commit 2ad784339e "docs: update README to use > GitLab repo URLs") almost all of the code base was converted to > point to GitLab instead of git.qemu.org. During 2023, git.qemu.org > switched from a git

Starting the QEMU 8.2 release cycle

Hi, The QEMU 8.2 release cycle has begun. I am merging qemu.git pull requests for this release. You do not need to CC me on pull requests because I am monitoring qemu-devel. The proposed schedule is as follows: 2023-08-22 Beginning of development phase 2023-11-14 Soft feature freeze.

Re: [PATCH v2 4/4] iotests: test the zoned format feature for qcow2 file

zoned-qcow2.out | 140 +++ > 2 files changed, 275 insertions(+) > create mode 100755 tests/qemu-iotests/tests/zoned-qcow2 > create mode 100644 tests/qemu-iotests/tests/zoned-qcow2.out Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [PATCH v2 3/4] qcow2: add zoned emulation capability

On Mon, Aug 14, 2023 at 04:58:01PM +0800, Sam Li wrote: > By adding zone operations and zoned metadata, the zoned emulation > capability enables full emulation support of zoned device using > a qcow2 file. The zoned device metadata includes zone type, > zoned device state and write pointer of each

Re: [PATCH 21/21] block: Mark bdrv_add/del_child() and caller GRAPH_WRLOCK

nged, 27 insertions(+), 30 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [PATCH 20/21] block: Mark bdrv_unref_child() GRAPH_WRLOCK

drv-drain.c | 8 ++-- > 10 files changed, 50 insertions(+), 8 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [PATCH 19/21] block: Mark bdrv_root_unref_child() GRAPH_WRLOCK

| 2 ++ > 4 files changed, 9 insertions(+), 4 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [PATCH 17/21] block: Take graph rdlock in bdrv_drop_intermediate()

On Thu, Aug 17, 2023 at 02:50:16PM +0200, Kevin Wolf wrote: > The function reads the parents list, so it needs to hold the graph lock. > > Signed-off-by: Kevin Wolf > --- > block.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/block.c b/block.c > index 7df8780d6e..a82389f742

Re: [PATCH 16/21] block: Mark bdrv_parent_cb_change_media() GRAPH_RDLOCK

On Thu, Aug 17, 2023 at 02:50:15PM +0200, Kevin Wolf wrote: > The function reads the parents list, so it needs to hold the graph lock. > > Signed-off-by: Kevin Wolf > --- > block.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) Reviewed-by: Stefan Hajnocz

Re: [PATCH 15/21] block: Mark bdrv_child_perm() GRAPH_RDLOCK

bdrv_child_perm() conveniently already hold the lock. > > Signed-off-by: Kevin Wolf > --- > include/block/block_int-common.h | 10 +- > block.c | 11 ++- > block/copy-before-write.c| 10 +- > 3 files changed, 16 insertions(+),

Re: [PATCH 14/21] block: Mark bdrv_get_cumulative_perm() and callers GRAPH_RDLOCK

| 35 -- > blockdev.c | 6 + > 4 files changed, 40 insertions(+), 14 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

<    5   6   7   8   9   10   11   12   13   14   >