From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49991) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c4smo-0006zk-An for qemu-devel@nongnu.org; Thu, 10 Nov 2016 12:05:15 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c4smk-0000As-48 for qemu-devel@nongnu.org; Thu, 10 Nov 2016 12:05:14 -0500 Received: from mx1.redhat.com ([209.132.183.28]:44052) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c4smj-0000AW-VO for qemu-devel@nongnu.org; Thu, 10 Nov 2016 12:05:10 -0500 Date: Thu, 10 Nov 2016 17:05:03 +0000 From: "Daniel P. Berrange" Message-ID: <20161110170503.GE31855@redhat.com> Reply-To: "Daniel P. Berrange" References: <1478794177-29377-1-git-send-email-mst@redhat.com> <1478794177-29377-6-git-send-email-mst@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1478794177-29377-6-git-send-email-mst@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL 05/47] contrib: add libvhost-user List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: qemu-devel@nongnu.org, Peter Maydell , =?utf-8?Q?Marc-Andr=C3=A9?= Lureau , Felipe Franciosi , Paolo Bonzini , Stefan Hajnoczi , Eric Blake , Markus Armbruster On Thu, Nov 10, 2016 at 06:12:03PM +0200, Michael S. Tsirkin wrote: > From: Marc-Andr=C3=A9 Lureau >=20 > Add a library to help implementing vhost-user backend (or slave). >=20 > Dealing with vhost-user as an application developer isn't so easy: you > have all the trouble with any protocol: validation, unix ancillary data= , > shared memory, eventfd, logging, and on top of that you need to deal > with virtio queues, if possible efficiently. >=20 > qemu test has a nice vhost-user testing application vhost-user-bridge, > which implements most of vhost-user, and virtio.c which implements > virtqueues manipulation. Based on these two, I tried to make a simple > library, reusable for tests or development of new vhost-user scenarios. By "new vhost-user scenarios" does this mean facilitating creation of vhost user clients by out of tree 3rd parties ? Unless I'm missing something, the change doesn't actually build any .so of this code, so only thing that can use it right now is other QEMU code. If we want 3rd parties to use this, then it seems to me that LGPLv2+ or even a BSD like license would be more applicable than GPLv2+. Also we would need to state what API/ABI stability guarantees we're going to provide and create a versioned ELF .so. In general this patch looks like it is something to propose in 2.9, rather than pushing feature work in post-freeze. Regards, Daniel --=20 |: http://berrange.com -o- http://www.flickr.com/photos/dberrange= / :| |: http://libvirt.org -o- http://virt-manager.or= g :| |: http://entangle-photo.org -o- http://search.cpan.org/~danberr= / :|