From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:58955) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1h1H7M-0006HL-CK for qemu-devel@nongnu.org; Tue, 05 Mar 2019 15:56:53 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1h1H7L-0004KK-E3 for qemu-devel@nongnu.org; Tue, 05 Mar 2019 15:56:52 -0500 Received: from mail-qt1-f196.google.com ([209.85.160.196]:34382) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1h1H7J-0004II-Jo for qemu-devel@nongnu.org; Tue, 05 Mar 2019 15:56:51 -0500 Received: by mail-qt1-f196.google.com with SMTP id w4so10465849qtc.1 for ; Tue, 05 Mar 2019 12:56:49 -0800 (PST) Date: Tue, 5 Mar 2019 15:56:45 -0500 From: "Michael S. Tsirkin" Message-ID: <20190305155635-mutt-send-email-mst@kernel.org> References: <20190227103017.29515-1-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20190227103017.29515-1-marcandre.lureau@redhat.com> Subject: Re: [Qemu-devel] [PATCH v4 00/12] vhost-user-backend & vhost-user-input List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?Marc-Andr=E9?= Lureau Cc: qemu-devel@nongnu.org, kraxel@redhat.com On Wed, Feb 27, 2019 at 11:30:05AM +0100, Marc-André Lureau wrote: > Hi, > > This series is based on previously discussed "[PATCH v4 00/29] > vhost-user for input & GPU" and "vhost-user: define conventions for > vhost-user backends" work. The GPU part is left off for now. > > This series introduces a "vhost-user-backend": an helper object to be > used by vhost-user devices to ease with backend initialization and > handling. As a simple showcase, a "vhost-user-input-pci" device is > introduced, which can be used with the "contrib: add vhost-user-input" > example. vhost-user-input isn't meant to be installed, discovered or > used by libvirt: no installation is done (no vhost-user JSON file is > provided either). > > thanks can you rebase on top of master pls? > v4: > - drop "RFC: add explicit can_migrate to vhost_user_backend_dev_init()" > - remove useless "cmd" struct field leftover > > v3: > - add previously sent patch "libvhost-user: fix clang enum-conversion > warning" to fix clang build > - "define conventions for vhost-user backends" updates after Eric review > - Drop user-creatable from vhost-user-backend > - Make vhost-user-input-pci take a chardev= (instead of vhost-user=) > > v2: > - rebased (VhostUserInputPCI got most of the changes, due to split) > - added "RFC: add explicit can_migrate to > vhost_user_backend_dev_init()" to attempt to address Michael > concerns about migration. > > Marc-André Lureau (12): > libvhost-user: fix clang enum-conversion warning > vhost-user: define conventions for vhost-user backends > vhost-user: simplify vhost_user_init/vhost_user_cleanup > libvhost-user: exit by default on VHOST_USER_NONE > vhost-user: wrap some read/write with retry handling > Add vhost-user-backend > vhost-user: split vhost_user_read() > vhost-user: add vhost_user_input_get_config() > libvhost-user-glib: export vug_source_new() > libvhost-user: add vu_queue_unpop() > Add vhost-user-input-pci > contrib: add vhost-user-input > > contrib/libvhost-user/libvhost-user-glib.h | 3 + > contrib/libvhost-user/libvhost-user.h | 17 +- > include/hw/virtio/vhost-backend.h | 4 + > include/hw/virtio/vhost-user-blk.h | 2 +- > include/hw/virtio/vhost-user-scsi.h | 2 +- > include/hw/virtio/vhost-user.h | 2 +- > include/hw/virtio/virtio-input.h | 14 + > include/sysemu/vhost-user-backend.h | 58 +++ > backends/cryptodev-vhost-user.c | 18 +- > backends/vhost-user.c | 213 +++++++++++ > contrib/libvhost-user/libvhost-user-glib.c | 11 +- > contrib/libvhost-user/libvhost-user.c | 19 +- > contrib/vhost-user-input/main.c | 398 +++++++++++++++++++++ > hw/block/vhost-user-blk.c | 22 +- > hw/input/vhost-user-input.c | 99 +++++ > hw/scsi/vhost-user-scsi.c | 20 +- > hw/virtio/vhost-stub.c | 4 +- > hw/virtio/vhost-user-input-pci.c | 53 +++ > hw/virtio/vhost-user.c | 118 +++++- > net/vhost-user.c | 13 +- > MAINTAINERS | 5 + > Makefile | 3 + > Makefile.objs | 1 + > backends/Makefile.objs | 3 +- > configure | 3 + > contrib/vhost-user-input/Makefile.objs | 1 + > default-configs/virtio.mak | 1 + > docs/interop/vhost-user.json | 232 ++++++++++++ > docs/interop/vhost-user.txt | 109 +++++- > hw/input/Makefile.objs | 1 + > hw/virtio/Makefile.objs | 1 + > 31 files changed, 1363 insertions(+), 87 deletions(-) > create mode 100644 include/sysemu/vhost-user-backend.h > create mode 100644 backends/vhost-user.c > create mode 100644 contrib/vhost-user-input/main.c > create mode 100644 hw/input/vhost-user-input.c > create mode 100644 hw/virtio/vhost-user-input-pci.c > create mode 100644 contrib/vhost-user-input/Makefile.objs > create mode 100644 docs/interop/vhost-user.json > > -- > 2.21.0