All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: "Nicholas A. Bellinger" <nab@linux-iscsi.org>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	target-devel <target-devel@vger.kernel.org>,
	linux-scsi <linux-scsi@vger.kernel.org>,
	lf-virt <virtualization@lists.linux-foundation.org>,
	kvm-devel <kvm@vger.kernel.org>,
	Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
	Zhi Yong Wu <wuzhy@cn.ibm.com>,
	Anthony Liguori <aliguori@linux.vnet.ibm.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Christoph Hellwig <hch@lst.de>, Jens Axboe <axboe@kernel.dk>,
	Hannes Reinecke <hare@suse.de>
Subject: Re: [RFC-v2 0/4] tcm_vhost+cmwq fabric driver code for-3.6
Date: Wed, 18 Jul 2012 11:00:42 -0500	[thread overview]
Message-ID: <5006DDAA.6080304@us.ibm.com> (raw)
In-Reply-To: <20120718155338.GA21817@infradead.org>

On 07/18/2012 10:53 AM, Christoph Hellwig wrote:
> On Wed, Jul 18, 2012 at 08:42:21AM -0500, Anthony Liguori wrote:
>>
>> If you add support for a new command, you need to provide userspace
>> a way to disable this command.  If you change what gets reported for
>> VPD, you need to provide userspace a way to make VPD look like what
>> it did in a previous version.
>>
>> Basically, you need to be able to make a TCM device behave 100% the
>> same as it did in an older version of the kernel.
>>
>> This is unique to virtualization due to live migration.  If you
>> migrate from a 3.6 kernel to a 3.8 kernel, you need to make sure
>> that the 3.8 kernel's TCM device behaves exactly like the 3.6 kernel
>> because the guest that is interacting with it does not realize that
>> live migration happened.
>
> I don't think these strict live migration rules apply to SCSI targets.
>
> Real life storage systems get new features and different behaviour with
> firmware upgrades all the time, and SCSI initiators deal with that just
> fine.  I don't see any reason to be more picky just because we're
> virtualized.

But would this happen while a system is running live?

I agree that in general, SCSI targets don't need this, but I'm pretty sure that 
if a guest probes for a command, you migrate to an old version, and that command 
is no longer there, badness will ensue.

It's different when you're talking about a reboot happening or a 
disconnect/reconnect due to firmware upgrade.  The OS would naturally be 
reprobing in this case.

Regards,

Anthony Liguori

>


  parent reply	other threads:[~2012-07-18 16:00 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-11 21:15 [RFC-v2 0/4] tcm_vhost+cmwq fabric driver code for-3.6 Nicholas A. Bellinger
2012-07-11 21:15 ` [RFC-v2 1/4] vhost: Separate vhost-net features from vhost features Nicholas A. Bellinger
2012-07-11 21:15 ` Nicholas A. Bellinger
2012-07-11 21:15 ` [RFC-v2 2/4] vhost: make vhost work queue visible Nicholas A. Bellinger
2012-07-11 21:15 ` Nicholas A. Bellinger
2012-07-11 21:15 ` [RFC-v2 3/4] vhost: Add vhost_scsi specific defines Nicholas A. Bellinger
2012-07-11 21:15 ` Nicholas A. Bellinger
2012-07-11 21:15 ` [RFC-v2 4/4] tcm_vhost: Initial merge for vhost level target fabric driver Nicholas A. Bellinger
2012-07-11 21:15 ` Nicholas A. Bellinger
2012-07-17 15:05 ` [RFC-v2 0/4] tcm_vhost+cmwq fabric driver code for-3.6 Michael S. Tsirkin
2012-07-17 18:55   ` Anthony Liguori
2012-07-17 19:00     ` Michael S. Tsirkin
2012-07-17 21:50     ` Nicholas A. Bellinger
2012-07-18 13:42       ` Anthony Liguori
2012-07-18 13:42       ` Anthony Liguori
2012-07-18 13:56         ` Paolo Bonzini
2012-07-18 15:33         ` Michael S. Tsirkin
2012-07-18 15:53         ` Christoph Hellwig
2012-07-18 15:53         ` Christoph Hellwig
2012-07-18 16:00           ` Michael S. Tsirkin
2012-07-18 16:42             ` Rustad, Mark D
2012-07-18 17:17               ` Michael S. Tsirkin
2012-07-18 20:12                 ` Rustad, Mark D
2012-07-18 20:12                 ` Rustad, Mark D
2012-07-18 16:42             ` Rustad, Mark D
2012-07-18 16:00           ` Anthony Liguori
2012-07-18 16:00           ` Anthony Liguori [this message]
2012-07-18 16:47             ` James Bottomley
2012-07-18 19:12               ` Anthony Liguori
2012-07-19  6:00                 ` Paolo Bonzini
2012-07-19  6:00                 ` Paolo Bonzini
2012-07-19  7:28                   ` James Bottomley
2012-07-19  7:30                     ` Paolo Bonzini
2012-07-18 19:12               ` Anthony Liguori
2012-07-18 22:45         ` Nicholas A. Bellinger
2012-07-18 22:45         ` Nicholas A. Bellinger
2012-07-17 21:17   ` Nicholas A. Bellinger
2012-07-17 21:17   ` Nicholas A. Bellinger
2012-07-17 21:34     ` Michael S. Tsirkin
2012-07-17 22:02       ` Nicholas A. Bellinger
2012-07-17 22:02       ` Nicholas A. Bellinger
2012-07-17 22:18         ` Michael S. Tsirkin
2012-07-17 22:18         ` Michael S. Tsirkin
2012-07-17 22:37           ` Nicholas A. Bellinger
2012-07-17 23:11             ` Michael S. Tsirkin
2012-07-18  0:17               ` Nicholas A. Bellinger
2012-07-17 21:58     ` Michael S. Tsirkin
2012-07-17 22:04       ` Nicholas A. Bellinger

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=5006DDAA.6080304@us.ibm.com \
    --to=aliguori@us.ibm.com \
    --cc=aliguori@linux.vnet.ibm.com \
    --cc=axboe@kernel.dk \
    --cc=hare@suse.de \
    --cc=hch@infradead.org \
    --cc=hch@lst.de \
    --cc=kvm@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=nab@linux-iscsi.org \
    --cc=pbonzini@redhat.com \
    --cc=stefanha@linux.vnet.ibm.com \
    --cc=target-devel@vger.kernel.org \
    --cc=virtualization@lists.linux-foundation.org \
    --cc=wuzhy@cn.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.