libvhost-user: lower dependency on QEMU headers

[PATCH 0/2] libvhost-user: lower dependency on QEMU headers

marcandre.lureau@redhat.com posted 2 patches 3 years, 5 months ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201118080902.30033-1-marcandre.lureau@redhat.com

contrib/libvhost-user/libvhost-user.c | 127 +++++++++++++++++---------
1 file changed, 83 insertions(+), 44 deletions(-)

Expand all Fold all

[PATCH 0/2] libvhost-user: lower dependency on QEMU headers

Posted by marcandre.lureau@redhat.com 3 years, 5 months ago

From: Marc-André Lureau <marcandre.lureau@redhat.com>

Hi,

libvhost-user is meant to be free of glib dependency, and easily
copyable/reusable outside of QEMU. Clean-up some dependencies that crept in
recently (the one remaining is qemu/atomic.h, from which a subset is used)

Marc-André Lureau (2):
  libvhost-user: replace qemu/bswap.h with glibc endian.h
  libvhost-user: replace qemu/memfd.h usage

 contrib/libvhost-user/libvhost-user.c | 127 +++++++++++++++++---------
 1 file changed, 83 insertions(+), 44 deletions(-)

-- 
2.29.0

Re: [PATCH 0/2] libvhost-user: lower dependency on QEMU headers

Posted by Stefan Hajnoczi 3 years, 5 months ago

On Wed, Nov 18, 2020 at 12:09:00PM +0400, marcandre.lureau@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
> 
> Hi,
> 
> libvhost-user is meant to be free of glib dependency, and easily
> copyable/reusable outside of QEMU. Clean-up some dependencies that crept in
> recently (the one remaining is qemu/atomic.h, from which a subset is used)
> 
> Marc-André Lureau (2):
>   libvhost-user: replace qemu/bswap.h with glibc endian.h
>   libvhost-user: replace qemu/memfd.h usage
> 
>  contrib/libvhost-user/libvhost-user.c | 127 +++++++++++++++++---------
>  1 file changed, 83 insertions(+), 44 deletions(-)

Let's find a way to enforce this. How about a libvhost-user-only build
in the CI system without glib2-devel? Also maybe a way to poison QEMU
headers? It might be simpler to move contrib/libvhost-user to a separate
git repo though.

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>