On 1/18/22 17:27, Emanuele Giuseppe Esposito wrote:
* Finally fully enable assert_bdrv_graph_writable(), checking also for the drains, in patch 11.
Wow, this is definitely not just moving code around! It's great that you could pull this off. It gives me a lot more confidence that the locking model and the graph/IO split are correct, and also a lot more confidence about the logic to move the drains back and forth.
It does add a lot more complication to the API with all the _unlocked variants, but I think we're getting closer and closer to the endgame for the AioContext lock.
Paolo