[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 00/16] Introduce I/O channels framework
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH v1 00/16] Introduce I/O channels framework |
Date: |
Thu, 24 Sep 2015 15:51:53 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Sep 22, 2015 at 02:23:20PM +0200, Paolo Bonzini wrote:
>
>
> On 18/09/2015 15:18, Daniel P. Berrange wrote:
> > David Gilbert had some feedback on my previous RFC series which
> > I have incorporated into this new posting. I did forget his
> > comments about using #defines instead of magic constants in
> > the websock patch, so that's still on my todo list.
> >
> > Finally I've listed myself as maintainer for the new io/ and
> > include/io/ directory prefixes.
>
> I only made a few very small remarks. The API is reasonable and you've
> proved its utility by using it in the rest of your project.
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 remember I had some remarks about the rest of the conversion and
> especially the integration with coroutines, but it's not part of this
> series.
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.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
- Re: [Qemu-devel] [PATCH v1 09/16] io: pull Buffer code out of VNC module, (continued)
- [Qemu-devel] [PATCH v1 03/16] sockets: allow port to be NULL when listening on IP address, Daniel P. Berrange, 2015/09/18
- [Qemu-devel] [PATCH v1 01/16] sockets: add helpers for creating SocketAddress from a socket, Daniel P. Berrange, 2015/09/18
- [Qemu-devel] [PATCH v1 15/16] io: add QIOChannelCommand class, Daniel P. Berrange, 2015/09/18
- [Qemu-devel] [PATCH v1 13/16] io: add QIOChannelTLS class, Daniel P. Berrange, 2015/09/18
- [Qemu-devel] [PATCH v1 12/16] io: add QIOChannelFile class, Daniel P. Berrange, 2015/09/18
- [Qemu-devel] [PATCH v1 05/16] osdep: add qemu_fork() wrapper for safely handling signals, Daniel P. Berrange, 2015/09/18
- [Qemu-devel] [PATCH v1 02/16] sockets: move qapi_copy_SocketAddress into qemu-sockets.c, Daniel P. Berrange, 2015/09/18
- [Qemu-devel] [PATCH v1 14/16] io: add QIOChannelWebsock class, Daniel P. Berrange, 2015/09/18
- Re: [Qemu-devel] [PATCH v1 00/16] Introduce I/O channels framework, Paolo Bonzini, 2015/09/22
- Re: [Qemu-devel] [PATCH v1 00/16] Introduce I/O channels framework,
Daniel P. Berrange <=