From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41964) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d1ZwM-0007DU-Bt for qemu-devel@nongnu.org; Fri, 21 Apr 2017 10:53:43 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d1ZwI-0005g6-Ep for qemu-devel@nongnu.org; Fri, 21 Apr 2017 10:53:42 -0400 Received: from mail-bl2nam02on0114.outbound.protection.outlook.com ([104.47.38.114]:21680 helo=NAM02-BL2-obe.outbound.protection.outlook.com) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d1ZwI-0005fI-72 for qemu-devel@nongnu.org; Fri, 21 Apr 2017 10:53:38 -0400 From: Felipe Franciosi Date: Fri, 21 Apr 2017 14:53:33 +0000 Message-ID: <9950C259-FD65-4286-84D0-89FA00420E86@nutanix.com> References: <1488479153-21203-1-git-send-email-felipe@nutanix.com> <20170302214746.GA4711@redhat.com> In-Reply-To: <20170302214746.GA4711@redhat.com> Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-ID: <132B368A1149F142A21D3652F54A3602@namprd02.prod.outlook.com> Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH v5 0/4] Introduce vhost-user-scsi and sample application List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" , Paolo Bonzini Cc: Stefan Hajnoczi , Marc-Andre Lureau , "qemu-devel@nongnu.org" , "Daniel P. Berrange" , "james.r.harris@intel.com" , "yuanhan.liu@linux.intel.com" > On 2 Mar 2017, at 21:47, Michael S. Tsirkin wrote: >=20 > On Thu, Mar 02, 2017 at 10:25:49AM -0800, Felipe Franciosi wrote: >> Based on various discussions on the 2016 KVM Forum, I'm sending over a >> vhost-user-scsi implementation for your consideration. >>=20 >> This patchset introduces a new vhost-user SCSI device. While heavily >> based on vhost-scsi, it is implemented using vhost's userspace >> counterpart. The device has been coded and tested to work with live >> migration. >>=20 >> As part of this work, a new vhost-scsi-common device was created and >> the existing vhost-scsi device moved underneath it. The new >> vhost-user-scsi device is also placed underneath it. >>=20 >> A sample application based on the newly introduced libvhost-user is >> also included. It makes use of libiscsi for simplicity. >>=20 >> For convenience, I'm maintaining an up-to-date version of these >> patches (including some necessary fixes for libvhost-user) on: >>=20 >> https://github.com/franciozzy/qemu/tree/vus-upstream-v5 >>=20 >> See the individual patches for build and use instructions. >>=20 >> Signed-off-by: Felipe Franciosi >=20 > BTW, pls remember to ping after QEMU release. Ping! :-) F. >=20 >> v4 -> v5: >> - Add a patch to fix a build issue with libvhost-user. >>=20 >> v3 -> v4: >> - Drop configure switches and build vus by default. >> - Add sample application to .gitignore. >> - Removed a whitespace error. >> - Rebase on master. >>=20 >> v2 -> v3: >> - Rebase after vhost notifier fixes by Paolo. >> - Exposed F_HOTPLUG and F_CHANGE on vhost-user-scsi. >>=20 >> v1 -> v2: >> - Introduce new vhost-scsi-common device type. >> - Move vhost-scsi device underneath vhost-scsi-common. >> - Move sample application from tests/ to contrib/. >> - Make sample application use the glib event loop. >> - Minor fixes. >>=20 >> Felipe Franciosi (4): >> libvhost-user: replace vasprintf() to fix build >> vhost-scsi: create a vhost-scsi-common abstraction >> vus: Introduce vhost-user-scsi host device >> vus: Introduce a vhost-user-scsi sample application >>=20 >> .gitignore | 1 + >> Makefile | 3 + >> Makefile.objs | 4 + >> configure | 1 + >> contrib/libvhost-user/libvhost-user.c | 2 +- >> contrib/vhost-user-scsi/Makefile.objs | 1 + >> contrib/vhost-user-scsi/vhost-user-scsi.c | 886 ++++++++++++++++++++++++= ++++++ >> default-configs/pci.mak | 1 + >> hw/scsi/Makefile.objs | 3 +- >> hw/scsi/vhost-scsi-common.c | 143 +++++ >> hw/scsi/vhost-scsi.c | 194 ++----- >> hw/scsi/vhost-user-scsi.c | 215 ++++++++ >> hw/virtio/virtio-pci.c | 54 ++ >> hw/virtio/virtio-pci.h | 11 + >> include/hw/virtio/vhost-scsi-common.h | 48 ++ >> include/hw/virtio/vhost-scsi.h | 11 +- >> include/hw/virtio/vhost-user-scsi.h | 35 ++ >> include/hw/virtio/virtio-scsi.h | 5 + >> 18 files changed, 1469 insertions(+), 149 deletions(-) >> create mode 100644 contrib/vhost-user-scsi/Makefile.objs >> create mode 100644 contrib/vhost-user-scsi/vhost-user-scsi.c >> create mode 100644 hw/scsi/vhost-scsi-common.c >> create mode 100644 hw/scsi/vhost-user-scsi.c >> create mode 100644 include/hw/virtio/vhost-scsi-common.h >> create mode 100644 include/hw/virtio/vhost-user-scsi.h >>=20 >> --=20 >> 1.9.4