On Fri, Apr 15, 2022 at 03:18:41PM +0200, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > --- > block/block-backend.c | 18 +++++++++--------- > block/io.c | 24 ++++++++++++------------ > 2 files changed, 21 insertions(+), 21 deletions(-) >
> @@ -1580,8 +1580,8 @@ int blk_pread(BlockBackend *blk, int64_t offset, void > *buf, int bytes) > return ret < 0 ? ret : bytes; > } > > -int blk_pwrite(BlockBackend *blk, int64_t offset, const void *buf, int bytes, > - BdrvRequestFlags flags) > +int coroutine_fn blk_pwrite(BlockBackend *blk, int64_t offset, const void > *buf, int bytes, > + BdrvRequestFlags flags) Long line, worth rewrapping differently? The functions with _co_ in the name are obvious, the others might be worth a comment why it is okay. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
