Re: [Qemu-devel] [PATCH v1 00/16] Introduce I/O channels framework

[Paolo Bonzini]

On 24/09/2015 16:51, Daniel P. Berrange wrote:
> Thanks, I'd still love one or more people to do a detailed review of
> the I/O channel implementations before I think about sending a PULL
> request, since there's a fair amount of code in there....

I did a review of the file, socket, command and buffer code.  I made
another small suggestion regarding error checking.

The command code could add support for shutdown by closing the parent's
end of the pipe.  It can be useful to avoid deadlocks in bidirectional
pipes.

> I'm curious now as to which bit you might be refering to here :-) IIRC
> the only place I had any interaction with coroutines was the migration
> code QEMUFile implementation, where I need to yield when reads block.

I think it was NBD.

Paolo