On 09/09/2015 08:06, Fam Zheng wrote:
> This patch will probably go in before aio_disable_clients,

Actually I think it's blocked by aio_disable_clients.

> if any, but I'm not
> quite confident about the interface yet: listing a precise set of clients from
> monitor is an ugly coupling between modules:
> 
>     aio_disable_clients(bdrv_get_aio_context(bs),
>                         AIO_CLIENT_NBD | AIO_CLIENT_IOEVENTFD | 
> AIO_CLIENT_FOO);
> 
>     ...
> 
>     aio_enble_clients(bdrv_get_aio_context(bs),
>                       AIO_CLIENT_NBD | AIO_CLIENT_IOEVENTFD | AIO_CLIENT_FOO);
> 
> I prefer at least an abstraction:
> 
>     bdrv_quiesce_begin(bs);
> 
>     ...
> 
>     bdrv_quiesce_end(bs);

That's fine.

Paolo

> My point is maybe we should leave documenting this to whichever series that
> introduces it?

Reply via email to