On 11/07/2017 10:39, Kevin Wolf wrote: > Can you just add a comment to this effect to qcow2/qed_do_open() then?
Good point. >>> Or maybe the cleaner way would be bdrv_drained_begin/end() around >>> everything directly in bdrv_co_invalidate_cache(). > > Should we do this anyway? Or assert that no requests are in flight? I can certainly add assertions for clarity, but wouldn't requests have failed the permission checks and caused assertion failures already? Paolo
