From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:49929) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TLPb7-0008LW-4q for qemu-devel@nongnu.org; Mon, 08 Oct 2012 22:31:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TLPb6-00081e-77 for qemu-devel@nongnu.org; Mon, 08 Oct 2012 22:31:05 -0400 Received: from mx1.redhat.com ([209.132.183.28]:7026) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TLPb5-00081W-V3 for qemu-devel@nongnu.org; Mon, 08 Oct 2012 22:31:04 -0400 Message-ID: <50738B8E.1080303@redhat.com> Date: Tue, 09 Oct 2012 10:27:26 +0800 From: Dave Young MIME-Version: 1.0 References: <20120921133031.GA1682@darkstar.nay.redhat.com> <505C768A.5070801@redhat.com> <20120923023709.GA2742@darkstar.nay.redhat.com> <20121005081458.GC1399@stefanha-thinkpad.redhat.com> In-Reply-To: <20121005081458.GC1399@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2] virtio-blk: add default serial id List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: qemu-devel@nongnu.org On 10/05/2012 04:14 PM, Stefan Hajnoczi wrote: > On Sun, Sep 23, 2012 at 10:37:09AM +0800, Dave Young wrote: >> For the serial number decreasing issue, I think there's only these two ways to >> select, there's no ideal way to resolve this issue. >> My use case for this is for the kdump kernel to find proper disks, >> after 1st kernel crashing 2nd kernel need find right disk to dump vmcore. >> In this case v1 and v2 aproaches are both find to me. >> >> From my point of view, patch v1 is better though, I think unpluging 100000 is >> not a sane use case. It's not likely to happen. > > I'm not sure auto-assigning serial numbers is a good idea. The guest can use > the serial number in /etc/fstab or other places where it expects the serial > number to be persistent. > > Your patch does not provide persistent serial numbers, so a change to the QEMU > invocation could result in different serial numbers. The guest will get > confused or perhaps refuse to boot. Yes, it introduce confusion, but in this way at least the serial number can be persistent across guest reboot. Traditionally ide disks use this way as well, such as QEMU_HARDISK_00001, I think guest should not use this in /etc/fstab. > > I'd prefer if we don't expose a temporary serial number at all in order to > avoid issues like this. > > Stefan -- Thanks Dave