From: Paolo Bonzini <pbonzini@redhat.com>
To: qemu-devel@nongnu.org
Cc: qemu-block@nongnu.org, afaria@redhat.com
Subject: [PATCH 00/24] More coroutine_fn fixes
Date: Thu, 13 Oct 2022 14:36:47 +0200 [thread overview]
Message-ID: <20221013123711.620631-1-pbonzini@redhat.com> (raw)
Most of these were extracted from Alberto's static analysis series.
After this series, the only errors reported by the analyzer are:
- a call to bs->drv->bdrv_co_drain_begin from bdrv_open_driver. This
relies on bdrv_co_drain_begin not having to yield, which is indeed the
case right after opening but is iffy
- assigning coroutine_fn to non-coroutine_fn in the human monitor
for the two coroutine commands screendump and block_resize.
Paolo
Alberto Faria (20):
backup: remove incorrect coroutine_fn annotation
block: remove incorrect coroutine_fn annotation
monitor: add missing coroutine_fn annotation
ssh: add missing coroutine_fn annotation
block: add missing coroutine_fn annotation to prototypes
coroutine-lock: add missing coroutine_fn annotation to prototypes
coroutine-io: add missing coroutine_fn annotation to prototypes
block: add missing coroutine_fn annotation to BlockDriverState
callbacks
qcow2: add coroutine_fn annotation for indirect-called functions
commit: switch to *_co_* functions
block: switch to *_co_* functions
mirror: switch to *_co_* functions
parallels: switch to *_co_* functions
qcow: switch to *_co_* functions
qcow2: switch to *_co_* functions
qed: switch to *_co_* functions
vdi: switch to *_co_* functions
vhdx: switch to *_co_* functions
vmdk: switch to *_co_* functions
monitor: switch to *_co_* functions
Paolo Bonzini (4):
blkdebug: add missing coroutine_fn annotation for indirect-called
functions
qcow: manually add more coroutine_fn annotations
qcow2: manually add more coroutine_fn annotations
vmdk: manually add more coroutine_fn annotations
block.c | 2 +-
block/backup.c | 2 +-
block/blkdebug.c | 2 +-
block/commit.c | 2 +-
block/io.c | 8 +--
block/mirror.c | 4 +-
block/monitor/block-hmp-cmds.c | 2 +-
block/parallels.c | 28 +++++-----
block/qcow.c | 56 ++++++++++----------
block/qcow2-bitmap.c | 4 +-
block/qcow2-cluster.c | 26 ++++-----
block/qcow2-refcount.c | 18 +++----
block/qcow2-snapshot.c | 6 +--
block/qcow2.c | 32 ++++++------
block/qcow2.h | 32 ++++++------
block/qed-table.c | 2 +-
block/qed.c | 12 ++---
block/ssh.c | 6 +--
block/vdi.c | 17 +++---
block/vhdx.c | 8 +--
block/vmdk.c | 90 ++++++++++++++++----------------
blockdev.c | 2 +-
include/block/block-hmp-cmds.h | 2 +-
include/block/block-io.h | 5 +-
include/block/block_int-common.h | 12 ++---
include/monitor/hmp.h | 3 +-
include/qemu/coroutine.h | 18 ++++---
27 files changed, 202 insertions(+), 199 deletions(-)
--
2.37.3
next reply other threads:[~2022-10-13 12:43 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-13 12:36 Paolo Bonzini [this message]
2022-10-13 12:36 ` [PATCH 01/24] backup: remove incorrect coroutine_fn annotation Paolo Bonzini
2022-10-13 12:36 ` [PATCH 02/24] block: " Paolo Bonzini
2022-10-13 12:36 ` [PATCH 03/24] monitor: add missing " Paolo Bonzini
2022-10-13 12:36 ` [PATCH 04/24] ssh: " Paolo Bonzini
2022-10-13 12:36 ` [PATCH 05/24] block: add missing coroutine_fn annotation to prototypes Paolo Bonzini
2022-10-13 12:36 ` [PATCH 06/24] coroutine-lock: " Paolo Bonzini
2022-10-13 12:36 ` [PATCH 07/24] coroutine-io: " Paolo Bonzini
2022-10-13 12:36 ` [PATCH 08/24] block: add missing coroutine_fn annotation to BlockDriverState callbacks Paolo Bonzini
2022-10-13 12:36 ` [PATCH 09/24] qcow2: add coroutine_fn annotation for indirect-called functions Paolo Bonzini
2022-10-13 12:36 ` [PATCH 10/24] blkdebug: add missing " Paolo Bonzini
2022-10-13 12:36 ` [PATCH 11/24] qcow: manually add more coroutine_fn annotations Paolo Bonzini
2022-10-13 12:36 ` [PATCH 12/24] qcow2: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 13/24] vmdk: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 14/24] commit: switch to *_co_* functions Paolo Bonzini
2022-10-13 12:37 ` [PATCH 15/24] block: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 16/24] mirror: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 17/24] parallels: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 18/24] qcow: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 19/24] qcow2: " Paolo Bonzini
2022-10-27 17:05 ` Kevin Wolf
2022-10-13 12:37 ` [PATCH 20/24] qed: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 21/24] vdi: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 22/24] vhdx: " Paolo Bonzini
2022-10-13 12:37 ` [PATCH 23/24] vmdk: " Paolo Bonzini
2022-10-27 17:07 ` Kevin Wolf
2022-10-13 12:37 ` [PATCH 24/24] monitor: " Paolo Bonzini
2022-10-27 17:15 ` [PATCH 00/24] More coroutine_fn fixes Kevin Wolf
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20221013123711.620631-1-pbonzini@redhat.com \
--to=pbonzini@redhat.com \
--cc=afaria@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).