From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=52370 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OJmOY-0005oS-Mi for qemu-devel@nongnu.org; Wed, 02 Jun 2010 07:46:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OJmOX-0004Re-A9 for qemu-devel@nongnu.org; Wed, 02 Jun 2010 07:46:02 -0400 Received: from e36.co.us.ibm.com ([32.97.110.154]:46141) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OJmOX-0004RR-3m for qemu-devel@nongnu.org; Wed, 02 Jun 2010 07:46:01 -0400 Received: from d03relay04.boulder.ibm.com (d03relay04.boulder.ibm.com [9.17.195.106]) by e36.co.us.ibm.com (8.14.4/8.13.1) with ESMTP id o52BgkI9013247 for ; Wed, 2 Jun 2010 05:42:46 -0600 Received: from d03av03.boulder.ibm.com (d03av03.boulder.ibm.com [9.17.195.169]) by d03relay04.boulder.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id o52BjnPv027778 for ; Wed, 2 Jun 2010 05:45:50 -0600 Received: from d03av03.boulder.ibm.com (loopback [127.0.0.1]) by d03av03.boulder.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id o52Bjnv4011801 for ; Wed, 2 Jun 2010 05:45:49 -0600 Date: Wed, 2 Jun 2010 06:45:46 -0500 From: Ryan Harper Subject: Re: [Qemu-devel] Re: [PATCH] virtio-blk: assign a default serial number if none provided Message-ID: <20100602114546.GB19185@us.ibm.com> References: <20100602014854.GB16406@us.ibm.com> <20100602090050.GA19662@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100602090050.GA19662@redhat.com> List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: john cooper , Ryan Harper , qemu-devel@nongnu.org * Michael S. Tsirkin [2010-06-02 04:08]: > On Tue, Jun 01, 2010 at 08:48:54PM -0500, Ryan Harper wrote: > > This patch applies on-top of John's virtio-blk serial patches. > > > > Generate default serial numbers for virtio drives based on DriveInfo.unit which is > > incremented for each additional virtio-blk device. This provides a > > per-virtio-blk number to use in the default string: QM%05d that is used in > > hw/ide/core.c. The resulting serial number looks like: QM00001, etc. > > > > Signed-off-by: Ryan Harper > > > I think that there's a problem with this approach in that hot plug A, > hot plug B, hot unplug A is not the same as hot plug B. > So you might get guest boot failures and no easy way to > figure out why. For guests that need S/N, I think they > really must be persistent. That's true; though I think most boot drives boot via either LVM or UUID which will remain persistent. That said, if you are relying on the by-id; then of course the user will need to specify serial versus having one auto-generated. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ryanh@us.ibm.com