From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=46027 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OHzQZ-0008FS-75 for qemu-devel@nongnu.org; Fri, 28 May 2010 09:16:44 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OHzQX-0005cW-QD for qemu-devel@nongnu.org; Fri, 28 May 2010 09:16:42 -0400 Received: from e6.ny.us.ibm.com ([32.97.182.146]:34809) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OHzQX-0005cL-Nl for qemu-devel@nongnu.org; Fri, 28 May 2010 09:16:41 -0400 Received: from d01relay03.pok.ibm.com (d01relay03.pok.ibm.com [9.56.227.235]) by e6.ny.us.ibm.com (8.14.3/8.13.1) with ESMTP id o4SDEs52019634 for ; Fri, 28 May 2010 09:14:54 -0400 Received: from d01av02.pok.ibm.com (d01av02.pok.ibm.com [9.56.224.216]) by d01relay03.pok.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id o4SDGdVY114238 for ; Fri, 28 May 2010 09:16:39 -0400 Received: from d01av02.pok.ibm.com (loopback [127.0.0.1]) by d01av02.pok.ibm.com (8.14.3/8.13.1/NCO v10.0 AVout) with ESMTP id o4SDGcqU029359 for ; Fri, 28 May 2010 10:16:39 -0300 Date: Fri, 28 May 2010 08:16:29 -0500 From: Ryan Harper Subject: Re: [Qemu-devel] [PATCH 0/4] Add virtio disk identification support Message-ID: <20100528131629.GD29645@us.ibm.com> References: <4BAAF541.4090605@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4BAAF541.4090605@redhat.com> List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: john cooper Cc: Rusty Russell , Marc Haber , qemu-devel@nongnu.org * john cooper [2010-03-25 00:45]: > This series adds the minimal support to qemu and virtio_blk > to support passing of a virtio_blk serial id string from qemu > through the guest driver and to the guest userland. > > This is derived in part from a patch set posted by Rusty some > time ago, but has been minimized to remove support for prior > versions which attempted to provide the same functionality via > pci config/io space. This version rather uses a virtio request > as proposed in Rusty's example. > > Also removed is the packaging of the serial/id string within > the glorious bag of bits returned by the ATA_IDENTIFY command. > Here we transfer only the 20 bytes of serial/id string from > qemu to the guest userland. In the proposed interface, this > is made available by an ioctl() into the virtio_blk driver > however other interfaces (eg: /sys) have also been proposed. > A code snippet is attached below as an example of ioctl usage. > > The resulting code is quite minimal and I believe it addresses > all concerns raised in prior versions. > > -john > > > > #include > #include > #include > #include > #include > > #define IOCTL_CMD 'VBID' > > main() > { > int fd, rv; > char buf[512]; > > bzero(buf, sizeof (buf)); > if ((fd = open("/dev/vda", O_RDONLY)) < 0) > perror("open"); > else if (ioctl(fd, IOCTL_CMD, buf) < 0) > perror("ioctl"); > else > printf("[%s]\n", buf); > } Would we want to patch up blkid command to use this so distro stacks can extract the serial and build the typical /dev/disk/by-id/ links ? -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ryanh@us.ibm.com