From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752016Ab2GQJMD (ORCPT ); Tue, 17 Jul 2012 05:12:03 -0400 Received: from bedivere.hansenpartnership.com ([66.63.167.143]:38826 "EHLO bedivere.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750842Ab2GQJMB (ORCPT ); Tue, 17 Jul 2012 05:12:01 -0400 Message-ID: <1342516317.3039.35.camel@dabdike.int.hansenpartnership.com> Subject: Re: [PATCH] sd: do not set changed flag on all unit attention conditions From: James Bottomley To: Paolo Bonzini Cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org Date: Tue, 17 Jul 2012 10:11:57 +0100 In-Reply-To: <5005285E.8060706@redhat.com> References: <1342454772-9018-1-git-send-email-pbonzini@redhat.com> <1342455503.3176.42.camel@dabdike.int.hansenpartnership.com> <50044D56.6000400@redhat.com> <1342511100.3039.9.camel@dabdike.int.hansenpartnership.com> <50052390.7030908@redhat.com> <1342514444.3039.23.camel@dabdike.int.hansenpartnership.com> <5005285E.8060706@redhat.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3 Content-Transfer-Encoding: 7bit Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2012-07-17 at 10:54 +0200, Paolo Bonzini wrote: > Il 17/07/2012 10:40, James Bottomley ha scritto: > >> > > >> > It's not specific to virtio-scsi, in fact I expect that virtio-scsi will > >> > be almost always used with non-removable disks. > >> > > >> > However, QEMU's SCSI target is not used just for virtio-scsi (for > >> > example it can be used for USB storage), and it lets you mark a disk as > >> > removable---why? because there exists real hardware that presents itself > >> > as an SBC removable disk. The only thing that is specific to > >> > virtualization, is support for online resizing (which generates a unit > >> > attention condition CAPACITY DATA HAS CHANGED). > > So what's the problem? If you're doing pass through of a physical disk, > > we pick up removable from its inquiry string ... a physical removable > > device doesn't get resized. If you have a virtual disk you want to > > resize, you don't set the removable flag in the inquiry data. > > In practice people will do what you said, and it's not a problem. > > However, there's nothing that prevents you from running qemu with a > removable SCSI disk, and then resizing it. I would like this to work, > because SBC allows it and there's no reason why it shouldn't. There's no such thing in the market today as a removable disk that's resizeable. Removable disks are for things like backup cartridges and ageing jazz drives. Worse: most removeable devices today are USB card readers whose standards compliance varies from iffy to non existent. Resizeable disks are currently the province of storage arrays. We don't do stuff just because the standards allows it; just the opposite: we try to use the smallest implementations from the standards we can get away with just because the more things we do, the more exceptions and broken devices we come across. James James