From: "Michael S. Tsirkin" <mst@redhat.com>
To: Felipe Franciosi <felipe@nutanix.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
Stefan Hajnoczi <stefanha@redhat.com>,
Marc-Andre Lureau <marcandre.lureau@redhat.com>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
yuanhan.liu@linux.intel.com, Jason Wang <jasowang@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application
Date: Wed, 29 Mar 2017 15:40:35 +0300 [thread overview]
Message-ID: <20170329124035.GC21803@redhat.com> (raw)
In-Reply-To: <1488472983-9108-1-git-send-email-felipe@nutanix.com>
On Thu, Mar 02, 2017 at 08:43:00AM -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.
>
> 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.
>
> 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.
>
> A sample application based on the newly introduced libvhost-user is
> also included. It makes use of libiscsi for simplicity.
>
> For convenience, I'm maintaining an up-to-date version of these
> patches (including some necessary fixes for libvhost-user) on:
>
> https://github.com/franciozzy/qemu/tree/vus-upstream-v4
>
> See the individual patches for build and use instructions.
>
> Signed-off-by: Felipe Franciosi <felipe@nutanix.com>
Cc Yuanhan who was interested in this topic.
> v3 -> v4:
> - Drop configure switches and build vus by default.
> - Add sample application to .gitignore.
> - Removed a whitespace error.
> - Rebase on master.
>
> v2 -> v3:
> - Rebase after vhost notifier fixes by Paolo.
> - Exposed F_HOTPLUG and F_CHANGE on vhost-user-scsi.
>
> 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.
>
> Felipe Franciosi (3):
> vhost-scsi: create a vhost-scsi-common abstraction
> vus: Introduce vhost-user-scsi host device
> vus: Introduce a vhost-user-scsi sample application
>
> .gitignore | 1 +
> Makefile | 3 +
> Makefile.objs | 4 +
> configure | 1 +
> 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 +
> 17 files changed, 1468 insertions(+), 148 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
>
> --
> 1.9.4
next prev parent reply other threads:[~2017-03-29 12:40 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-02 16:43 [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and sample application Felipe Franciosi
2017-03-02 16:43 ` [Qemu-devel] [PATCH v4 1/3] vhost-scsi: create a vhost-scsi-common abstraction Felipe Franciosi
2017-03-02 16:43 ` [Qemu-devel] [PATCH v4 2/3] vus: Introduce vhost-user-scsi host device Felipe Franciosi
2017-03-02 16:43 ` [Qemu-devel] [PATCH v4 3/3] vus: Introduce a vhost-user-scsi sample application Felipe Franciosi
2017-03-02 16:48 ` [Qemu-devel] [PATCH v4 0/3] Introduce vhost-user-scsi and " no-reply
2017-03-02 16:50 ` no-reply
2017-03-02 16:55 ` no-reply
2017-03-02 17:01 ` Felipe Franciosi
2017-03-02 17:17 ` Daniel P. Berrange
2017-03-02 17:21 ` Felipe Franciosi
2017-03-02 17:32 ` Daniel P. Berrange
2017-03-02 17:39 ` Felipe Franciosi
2017-03-02 18:08 ` Paolo Bonzini
2017-03-02 18:12 ` Felipe Franciosi
2017-03-02 19:26 ` Eric Blake
2017-03-29 12:40 ` Michael S. Tsirkin [this message]
2017-03-29 22:14 ` Felipe Franciosi
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=20170329124035.GC21803@redhat.com \
--to=mst@redhat.com \
--cc=felipe@nutanix.com \
--cc=jasowang@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=yuanhan.liu@linux.intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.