On Fri, Sep 25, 2015 at 12:15:46PM +0200, Paolo Bonzini wrote:
>
>
> On 18/09/2015 15:19, Daniel P. Berrange wrote:
> > + if (fds || nfds) {
> > + error_setg_errno(errp, EINVAL,
> > + "Channel does not support buffer descriptor
> > passing");
> > + return -1;
> > + }
>
> Typo (file descriptor passing) and the check could be hoisted to
> io/channel.c by using features.
>
> In fact capabilities could perhaps be stored statically in struct
> QIOChannel as a bitmap?
In general capabilities are static per class, but the socket code is an
exception, because whether you can do FD passing is a runtime check
based on the type of socket you opened. I might still be able to hoist
that up into the base class though, so will investigate..
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 :|
