On Mon, Apr 19, 2021 at 10:55:38AM +0200, Emanuele Giuseppe Esposito wrote: > Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com> > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > Signed-off-by: Emanuele Giuseppe Esposito <eespo...@redhat.com> > --- > include/block/block_int.h | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/include/block/block_int.h b/include/block/block_int.h > index a1aad5ad2d..67a0777e12 100644 > --- a/include/block/block_int.h > +++ b/include/block/block_int.h > @@ -154,7 +154,9 @@ struct BlockDriver { > */ > bool supports_backing; > > - /* For handling image reopen for split or non-split files */ > + /* For handling image reopen for split or non-split files. Called > + * with no I/O pending.
Does "Called with no I/O pending" mean "Called in a drained section"? > + */ > int (*bdrv_reopen_prepare)(BDRVReopenState *reopen_state, > BlockReopenQueue *queue, Error **errp); > void (*bdrv_reopen_commit)(BDRVReopenState *reopen_state); > @@ -168,6 +170,7 @@ struct BlockDriver { > /* Protocol drivers should implement this instead of bdrv_open */ > int (*bdrv_file_open)(BlockDriverState *bs, QDict *options, int flags, > Error **errp); > + /* Called from main thread. */ > void (*bdrv_close)(BlockDriverState *bs); > > > -- > 2.30.2 >
signature.asc
Description: PGP signature