On Fri, Feb 21, 2020 at 02:06:26PM +0100, Paolo Bonzini wrote: > On 21/02/20 13:59, Stefan Hajnoczi wrote: > > /* Add a handler to a ready list */ > > static void add_ready_handler(AioHandlerList *ready_list, > > AioHandler *node, > > int revents) > > { > > QLIST_SAFE_REMOVE(node, node_ready); /* remove from nested parent's list */ > > ^---- would cause corruption if node->node_ready was stale! > > > > Would you like me to add a comment? > No, it's okay. Are you happy with this series? Shall I include it in my next pull request or do you want to merge it? Thanks, Stefan