On Fri, Dec 23, 2022 at 04:21:33PM +0100, Paolo Bonzini wrote: > On 12/22/22 21:36, Marcel Holtmann wrote: > > The libvhost-user and libvduse libraries are also useful for external > > usage outside of QEMU and thus it would be nice if their files could > > be just copied and used. However due to different compiler settings, a > > lot of manual fixups are needed. This is the first attempt at some > > obvious fixes that can be done without any harm to the code and its > > readability. > > > > Marcel Holtmann (12): > > libvhost-user: Provide _GNU_SOURCE when compiling outside of QEMU > > libvhost-user: Replace typeof with __typeof__ > > libvhost-user: Cast rc variable to avoid compiler warning > > libvhost-user: Use unsigned int i for some for-loop iterations > > libvhost-user: Declare uffdio_register early to make it C90 compliant > > libvhost-user: Change dev->postcopy_ufd assignment to make it C90 > > compliant > > libvduse: Provide _GNU_SOURCE when compiling outside of QEMU > > libvduse: Switch to unsigned int for inuse field in struct VduseVirtq > > libvduse: Fix assignment in vring_set_avail_event > > libvhost-user: Fix assignment in vring_set_avail_event > > libvhost-user: Add extra compiler warnings > > libvduse: Add extra compiler warnings > > > > subprojects/libvduse/libvduse.c | 9 ++++-- > > subprojects/libvduse/meson.build | 8 ++++- > > subprojects/libvhost-user/libvhost-user.c | 36 +++++++++++++---------- > > subprojects/libvhost-user/meson.build | 8 ++++- > > 4 files changed, 42 insertions(+), 19 deletions(-) > > > > Looks good, but what happened to "libvhost-user: Switch to unsigned int for > inuse field in struct VuVirtq"? > > (I can pick it up from v3, no need to respin). > > Paolo
I merged that one IIRC. Paolo I wandered whether if you are going to be merging patches in these areas you wanted to be added to MAINTAINERS. -- MST
