From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH] virtio-spec: add field for scsi command size Date: Thu, 04 Jul 2013 11:05:13 +0200 Message-ID: <51D53AC9.80506@redhat.com> References: <20130613080259.GA11175@redhat.com> <20130613081047.GA11269@redhat.com> <20130617063746.GA5693@redhat.com> <87r4fyn1wx.fsf@rustcorp.com.au> <20130619082417.GA2558@redhat.com> <51C16BC2.2060302@redhat.com> <8761x9mrmk.fsf@rustcorp.com.au> <51C2CABD.70000@redhat.com> <87ppv3tb3y.fsf@rustcorp.com.au> <51D16EB8.3050709@redhat.com> <20130704073912.GE2095@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130704073912.GE2095@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: "Michael S. Tsirkin" Cc: virtualization@lists.linux-foundation.org List-Id: virtualization@lists.linuxfoundation.org Il 04/07/2013 09:39, Michael S. Tsirkin ha scritto: >>>>> > > > > Mainly because I'm not sure that *all* devices are now safe. Are they? >>>> > > > >>>> > > > virtio-scsi's implementation in QEMU is not safe (been delaying that for >>>> > > > too long, sorry), but the spec is safe. >>> > > >>> > > Then if we added a transport feature, we couldn't use it :( >> > >> > Transport feature bits are still negotiated per device though. >> > virtio-scsi devices in QEMU would not negotiate that feature. >> > >> > Paolo >> > >>> > > So I think it needs to be a per-device feature bit. > We don't know that it's helpful for any devices besides -net > so I'd say keep it simple. Then we should stop calling it a "device model bug". If we call it a bug, we must strive for support in all devices. If we don't care, we should stop calling it a bug. Paolo