qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: qemu-devel@nongnu.org, mst@redhat.com, xieyongji@bytedance.com
Cc: marcel@holtmann.org
Subject: [PATCH v2 00/10] Compiler warning fixes for libvhost-user,libvduse
Date: Tue, 20 Dec 2022 20:27:12 +0100	[thread overview]
Message-ID: <cover.1671563795.git.marcel@holtmann.org> (raw)

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.

Please note that 10/10 in this series is marked as RFC since I have no
idea what is correct here. From the structures point of view the
assignment that is done makes no sense to me. I might have to dig into
specification to figure it out or it better be commented accordingly
to tell the compiler that it got this part wrong.

Marcel Holtmann (10):
  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
  libvhost-user: Switch to unsigned int for inuse field in struct VuVirtq
  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

 subprojects/libvduse/libvduse.c           | 11 ++++++--
 subprojects/libvhost-user/libvhost-user.c | 31 ++++++++++++++---------
 subprojects/libvhost-user/libvhost-user.h |  2 +-
 3 files changed, 29 insertions(+), 15 deletions(-)

-- 
2.38.1



             reply	other threads:[~2022-12-20 19:28 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-20 19:27 Marcel Holtmann [this message]
2022-12-20 19:27 ` [PATCH v2 01/10] libvhost-user: Provide _GNU_SOURCE when compiling outside of QEMU Marcel Holtmann
2022-12-20 19:27 ` [PATCH v2 02/10] libvhost-user: Replace typeof with __typeof__ Marcel Holtmann
2022-12-20 21:48   ` Philippe Mathieu-Daudé
2022-12-20 19:27 ` [PATCH v2 03/10] libvhost-user: Cast rc variable to avoid compiler warning Marcel Holtmann
2022-12-20 19:27 ` [PATCH v2 04/10] libvhost-user: Use unsigned int i for some for-loop iterations Marcel Holtmann
2022-12-20 19:27 ` [PATCH v2 05/10] libvhost-user: Declare uffdio_register early to make it C90 compliant Marcel Holtmann
2022-12-20 21:49   ` Philippe Mathieu-Daudé
2022-12-20 19:27 ` [PATCH v2 06/10] libvhost-user: Change dev->postcopy_ufd assignment " Marcel Holtmann
2022-12-20 21:50   ` Philippe Mathieu-Daudé
2022-12-20 19:27 ` [PATCH v2 07/10] libvhost-user: Switch to unsigned int for inuse field in struct VuVirtq Marcel Holtmann
2022-12-20 19:27 ` [PATCH v2 08/10] libvduse: Provide _GNU_SOURCE when compiling outside of QEMU Marcel Holtmann
2022-12-20 19:27 ` [PATCH v2 09/10] libvduse: Switch to unsigned int for inuse field in struct VduseVirtq Marcel Holtmann
2022-12-21 11:32   ` Yongji Xie
2022-12-20 19:27 ` [RFC v2 10/10] libvduse: Fix assignment in vring_set_avail_event Marcel Holtmann
2022-12-21 11:44   ` Yongji Xie
2022-12-21 13:08     ` Marcel Holtmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1671563795.git.marcel@holtmann.org \
    --to=marcel@holtmann.org \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=xieyongji@bytedance.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).