From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Nufpo-0007bt-Nb for qemu-devel@nongnu.org; Thu, 25 Mar 2010 01:42:24 -0400 Received: from [140.186.70.92] (port=41866 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Nufpn-0007bW-8u for qemu-devel@nongnu.org; Thu, 25 Mar 2010 01:42:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1Nufpl-00011c-SD for qemu-devel@nongnu.org; Thu, 25 Mar 2010 01:42:23 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40530) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1Nufpl-00011V-Jy for qemu-devel@nongnu.org; Thu, 25 Mar 2010 01:42:21 -0400 Message-ID: <4BAAF5CA.2010401@redhat.com> Date: Thu, 25 Mar 2010 01:34:02 -0400 From: john cooper MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] [PATCH 4/4] Add virtio disk identification support List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Rusty Russell Cc: john.cooper@redhat.com, Marc Haber , qemu-devel@nongnu.org Return serial string to the guest application via ioctl driver call. Note this form of interface to the guest userland was the consensus when the prior version using the ATA_IDENTIFY came under dispute. Signed-off-by: john cooper --- diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c index cd66806..0954193 100644 --- a/drivers/block/virtio_blk.c +++ b/drivers/block/virtio_blk.c @@ -225,6 +225,21 @@ static int virtblk_ioctl(struct block_device *bdev, fmode_t mode, struct gendisk *disk = bdev->bd_disk; struct virtio_blk *vblk = disk->private_data; + if (cmd == 'VBID') { + void *usr_data = (void __user *)data; + char *id_str; + int err; + + if (!(id_str = kmalloc(VIRTIO_BLK_ID_BYTES, GFP_KERNEL))) + err = -ENOMEM; + else if ((err = virtblk_get_id(disk, id_str))) + ; + else if (copy_to_user(usr_data, id_str, VIRTIO_BLK_ID_BYTES)) + err = -EFAULT; + if (id_str) + kfree(id_str); + return err; + } /* * Only allow the generic SCSI ioctls if the host can support it. */ -- john.cooper@redhat.com