[v2] Still more coroutine and various fixes in block layer

[PATCH v2 0/9] Still more coroutine and various fixes in block layer

Emanuele Giuseppe Esposito posted 9 patches 1 year, 6 months ago

Diff against v1 v3 v4 v5 v6 v7
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20221104095700.4117433-1-eesposit@redhat.com

Maintainers: Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, John Snow <jsnow@redhat.com>, Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>, Eric Blake <eblake@redhat.com>, Fam Zheng <fam@euphon.net>

There is a newer version of this series

block.c                            | 111 +++++++++++++++++------------
block/block-backend.c              |  21 ++++++
block/block-copy.c                 |  15 ++--
block/commit.c                     |   4 +-
block/dirty-bitmap.c               |  66 ++++++++---------
block/vmdk.c                       |  36 +++++-----
include/block/block-global-state.h |   3 +-
include/sysemu/block-backend-io.h  |   9 +++
nbd/server.c                       |  43 +++++------
qemu-img.c                         |   4 +-
10 files changed, 182 insertions(+), 130 deletions(-)

Expand all Fold all

[PATCH v2 0/9] Still more coroutine and various fixes in block layer

Posted by Emanuele Giuseppe Esposito 1 year, 6 months ago

This is a dump of all minor coroutine-related fixes found while looking
around and testing various things in the QEMU block layer.

Patches aim to:
- add missing coroutine_fn annotation to the functions
- simplify to avoid the typical "if in coroutine: fn()
  // else create_coroutine(fn)" already present in generated_co_wraper
  functions.
- make sure that if a BlockDriver callback is defined as coroutine_fn, then
  it is always running in a coroutine.

Emanuele
---
v2:
* clarified commit message in patches 2/3/6 on why we add coroutine_fn

Emanuele Giuseppe Esposito (9):
  block: call bdrv_co_drain_begin in a coroutine
  block-copy: add missing coroutine_fn annotations
  nbd/server.c: add missing coroutine_fn annotations
  block-backend: replace bdrv_*_above with blk_*_above
  block: distinguish between bdrv_create running in coroutine and not
  block/vmdk: add missing coroutine_fn annotations
  block: bdrv_create_file is a coroutine_fn
  block: bdrv_create is never called in non-coroutine context
  block/dirty-bitmap: remove unnecessary qemu_in_coroutine() case

 block.c                            | 111 +++++++++++++++++------------
 block/block-backend.c              |  21 ++++++
 block/block-copy.c                 |  15 ++--
 block/commit.c                     |   4 +-
 block/dirty-bitmap.c               |  66 ++++++++---------
 block/vmdk.c                       |  36 +++++-----
 include/block/block-global-state.h |   3 +-
 include/sysemu/block-backend-io.h  |   9 +++
 nbd/server.c                       |  43 +++++------
 qemu-img.c                         |   4 +-
 10 files changed, 182 insertions(+), 130 deletions(-)

-- 
2.31.1

Re: [PATCH v2 0/9] Still more coroutine and various fixes in block layer

Posted by Paolo Bonzini 1 year, 6 months ago

On 11/4/22 10:56, Emanuele Giuseppe Esposito wrote:
> This is a dump of all minor coroutine-related fixes found while looking
> around and testing various things in the QEMU block layer.
> 
> Patches aim to:
> - add missing coroutine_fn annotation to the functions
> - simplify to avoid the typical "if in coroutine: fn()
>    // else create_coroutine(fn)" already present in generated_co_wraper
>    functions.
> - make sure that if a BlockDriver callback is defined as coroutine_fn, then
>    it is always running in a coroutine.

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>