All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: "Nicholas A. Bellinger" <nab@linux-iscsi.org>,
	target-devel <target-devel@vger.kernel.org>,
	lf-virt <virtualization@lists.linux-foundation.org>,
	kvm-devel <kvm@vger.kernel.org>,
	qemu-devel <qemu-devel@nongnu.org>,
	Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
	Zhi Yong Wu <wuzhy@cn.ibm.com>,
	Anthony Liguori <aliguori@linux.vnet.ibm.com>,
	Christoph Hellwig <hch@lst.de>, Hannes Reinecke <hare@suse.de>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>
Subject: Re: [PATCH 4/5] virtio-scsi: Add start/stop functionality for vhost-scsi
Date: Mon, 10 Sep 2012 08:42:15 +0200	[thread overview]
Message-ID: <504D8BC7.8070900@redhat.com> (raw)
In-Reply-To: <20120910062437.GD16819@redhat.com>

Il 10/09/2012 08:24, Michael S. Tsirkin ha scritto:
>> > I chose the backend name because, ideally, there would be no other
>> > difference.  QEMU _could_ implement all the goodies in vhost-scsi (such
>> > as reservations or ALUA), it just doesn't do that yet.
>> > 
>> > Paolo
> Then why do you say "It is used completely differently from
> virtio-scsi-pci"?

It is configured differently (and I haven't seen a proposal yet for how
to bridge the two), it does not interoperate, it has right now a
different set of features.

The "does not interoperate" bit is particularly important.  Say QEMU
were to implement persistent reservations (right now only a vhost-scsi
feature).  Then QEMU and vhost-scsi PR would not be interchangeable, a
reservation made by QEMU would not be visible in vhost and vice versa.

> Isn't it just a different backend?
> 
> If yes then it should be a backend option, like it is
> for virtio-net.

You mean a -drive option?  That would mean adding the logic to configure
vhost-scsi to the QEMU block layer, that's a completely different project...

Paolo

WARNING: multiple messages have this Message-ID (diff)
From: Paolo Bonzini <pbonzini@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
	kvm-devel <kvm@vger.kernel.org>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>,
	qemu-devel <qemu-devel@nongnu.org>,
	"Nicholas A. Bellinger" <nab@linux-iscsi.org>,
	lf-virt <virtualization@lists.linux-foundation.org>,
	Anthony Liguori <aliguori@linux.vnet.ibm.com>,
	target-devel <target-devel@vger.kernel.org>,
	Hannes Reinecke <hare@suse.de>, Zhi Yong Wu <wuzhy@cn.ibm.com>,
	Christoph Hellwig <hch@lst.de>
Subject: Re: [Qemu-devel] [PATCH 4/5] virtio-scsi: Add start/stop functionality for vhost-scsi
Date: Mon, 10 Sep 2012 08:42:15 +0200	[thread overview]
Message-ID: <504D8BC7.8070900@redhat.com> (raw)
In-Reply-To: <20120910062437.GD16819@redhat.com>

Il 10/09/2012 08:24, Michael S. Tsirkin ha scritto:
>> > I chose the backend name because, ideally, there would be no other
>> > difference.  QEMU _could_ implement all the goodies in vhost-scsi (such
>> > as reservations or ALUA), it just doesn't do that yet.
>> > 
>> > Paolo
> Then why do you say "It is used completely differently from
> virtio-scsi-pci"?

It is configured differently (and I haven't seen a proposal yet for how
to bridge the two), it does not interoperate, it has right now a
different set of features.

The "does not interoperate" bit is particularly important.  Say QEMU
were to implement persistent reservations (right now only a vhost-scsi
feature).  Then QEMU and vhost-scsi PR would not be interchangeable, a
reservation made by QEMU would not be visible in vhost and vice versa.

> Isn't it just a different backend?
> 
> If yes then it should be a backend option, like it is
> for virtio-net.

You mean a -drive option?  That would mean adding the logic to configure
vhost-scsi to the QEMU block layer, that's a completely different project...

Paolo

  parent reply	other threads:[~2012-09-10  6:42 UTC|newest]

Thread overview: 83+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-07  6:48 [PATCH 0/5] vhost-scsi: Add support for host virtualized target Nicholas A. Bellinger
2012-09-07  6:48 ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-07  6:48 ` [PATCH 1/5] monitor: Rename+move net_handle_fd_param -> monitor_handle_fd_param Nicholas A. Bellinger
2012-09-07  6:48   ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-07  6:48 ` Nicholas A. Bellinger
2012-09-07  6:48 ` [PATCH 2/5] vhost: Pass device path to vhost_dev_init() Nicholas A. Bellinger
2012-09-07  6:48   ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-07  6:48 ` Nicholas A. Bellinger
2012-09-07  6:48 ` [PATCH 3/5] vhost-scsi: add -vhost-scsi host device for use with tcm-vhost Nicholas A. Bellinger
2012-09-07  6:48   ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-07  6:48 ` Nicholas A. Bellinger
2012-09-07  6:48 ` [PATCH 4/5] virtio-scsi: Add start/stop functionality for vhost-scsi Nicholas A. Bellinger
2012-09-07  6:48   ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-07 16:00   ` Paolo Bonzini
2012-09-07 16:00     ` [Qemu-devel] " Paolo Bonzini
2012-09-07 19:23     ` Nicholas A. Bellinger
2012-09-07 19:23       ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-07 20:30       ` Paolo Bonzini
2012-09-07 20:30         ` [Qemu-devel] " Paolo Bonzini
2012-09-07 20:30       ` Paolo Bonzini
2012-09-08 22:40     ` Michael S. Tsirkin
2012-09-08 22:40       ` [Qemu-devel] " Michael S. Tsirkin
2012-09-10  6:16       ` Paolo Bonzini
2012-09-10  6:16         ` [Qemu-devel] " Paolo Bonzini
2012-09-10  6:24         ` Michael S. Tsirkin
2012-09-10  6:24           ` [Qemu-devel] " Michael S. Tsirkin
2012-09-10  6:42           ` Paolo Bonzini
2012-09-10  6:42           ` Paolo Bonzini [this message]
2012-09-10  6:42             ` [Qemu-devel] " Paolo Bonzini
2012-09-10  6:49             ` Michael S. Tsirkin
2012-09-10  6:49               ` [Qemu-devel] " Michael S. Tsirkin
2012-09-11 13:46           ` Anthony Liguori
2012-09-11 13:46           ` Anthony Liguori
2012-09-11 13:46             ` [Qemu-devel] " Anthony Liguori
2012-09-11 15:07             ` Michael S. Tsirkin
2012-09-11 15:07               ` [Qemu-devel] " Michael S. Tsirkin
2012-09-13 22:27               ` Nicholas A. Bellinger
2012-09-13 22:27                 ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-14  6:45                 ` Paolo Bonzini
2012-09-14  6:45                   ` [Qemu-devel] " Paolo Bonzini
2012-09-14 12:34                   ` Michael S. Tsirkin
2012-09-14 12:34                     ` [Qemu-devel] " Michael S. Tsirkin
2012-09-13 22:27               ` Nicholas A. Bellinger
2012-09-14 12:50             ` Michael S. Tsirkin
2012-09-14 12:50               ` [Qemu-devel] " Michael S. Tsirkin
2012-09-07  6:48 ` Nicholas A. Bellinger
2012-09-07  6:48 ` [PATCH 5/5] virtio-scsi: Set max_target=0 during vhost-scsi operation Nicholas A. Bellinger
2012-09-07  6:48   ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-07  6:48 ` Nicholas A. Bellinger
2012-09-11  4:36 ` [PATCH 0/5] vhost-scsi: Add support for host virtualized target Asias He
2012-09-11  4:36   ` [Qemu-devel] " Asias He
2012-09-13 20:12   ` Nicholas A. Bellinger
2012-09-13 20:12   ` Nicholas A. Bellinger
2012-09-13 20:12     ` [Qemu-devel] " Nicholas A. Bellinger
2012-09-14  8:21     ` Paolo Bonzini
2012-09-14  8:21       ` [Qemu-devel] " Paolo Bonzini
2013-01-17 16:43 ` Michael S. Tsirkin
2013-01-17 16:43   ` [Qemu-devel] " Michael S. Tsirkin
2013-01-17 21:07   ` Nicholas A. Bellinger
2013-01-17 21:07     ` [Qemu-devel] " Nicholas A. Bellinger
2013-01-21  8:50     ` Michael S. Tsirkin
2013-01-21  8:50       ` [Qemu-devel] " Michael S. Tsirkin
2013-01-28 13:01       ` Paolo Bonzini
2013-01-28 13:11         ` Michael S. Tsirkin
2013-01-28 13:29           ` Paolo Bonzini
2013-01-28 13:36             ` Michael S. Tsirkin
2013-01-28 13:33               ` Paolo Bonzini
2013-01-28 13:39                 ` Michael S. Tsirkin
2013-01-29 19:19                   ` Nicholas A. Bellinger
2013-01-29 21:03                     ` Paolo Bonzini
2013-01-29 22:38                       ` Nicholas A. Bellinger
2013-01-29 22:38                       ` Nicholas A. Bellinger
2013-01-29 21:03                     ` Paolo Bonzini
2013-01-29 19:19                   ` Nicholas A. Bellinger
2013-01-21 13:28     ` Anthony Liguori
2013-01-17 21:07   ` Nicholas A. Bellinger
2013-01-17 16:43 ` Michael S. Tsirkin
2013-04-02  3:28 ` [Qemu-devel] " Wenchao Xia
2013-04-02  3:28   ` Wenchao Xia
2013-04-02  5:07   ` Nicholas A. Bellinger
2013-04-02  5:07   ` Nicholas A. Bellinger
2013-04-02  5:07     ` Nicholas A. Bellinger
2013-04-02  3:28 ` Wenchao Xia

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=504D8BC7.8070900@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=aliguori@linux.vnet.ibm.com \
    --cc=hare@suse.de \
    --cc=hch@lst.de \
    --cc=jan.kiszka@siemens.com \
    --cc=kvm@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=nab@linux-iscsi.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@linux.vnet.ibm.com \
    --cc=target-devel@vger.kernel.org \
    --cc=virtualization@lists.linux-foundation.org \
    --cc=wuzhy@cn.ibm.com \
    --cc=wuzhy@linux.vnet.ibm.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.