From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rusty Russell Subject: Re: [PATCH 2/2] Add serial number support for virtio_blk, V4a Date: Thu, 4 Jun 2009 16:01:16 +0930 Message-ID: <200906041601.16539.rusty@rustcorp.com.au> References: <4A1F6130.9070705@redhat.com> <4A26E607.3070901@redhat.com> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="Boundary-00=_0o2JKyrOHPfF/PI" Content-Transfer-Encoding: 7bit Cc: qemu-devel@nongnu.org, KVM list , Christoph Hellwig To: john cooper Return-path: In-Reply-To: <4A26E607.3070901@redhat.com> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org --Boundary-00=_0o2JKyrOHPfF/PI Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit On Thu, 4 Jun 2009 06:37:19 am john cooper wrote: > This patch extracts the opaque data from pci i/o > region 0 via the added VIRTIO_BLK_F_IDENTIFY > field. By convention this data takes the form of > that returned by an ATA IDENTIFY DEVICE command, > however the driver (except for structure size) > makes no interpretation of the data. The structure > data is copied wholesale to userspace via a > HDIO_GET_IDENTITY ioctl command (eg: hdparm -i ). Thanks, applied with one subtraction: > +/* mapped into pci i/o region 0 > + */ > struct virtio_blk_config I removed this comment; it's true for virtio_pci, but not true in general. Thanks! Rusty. --Boundary-00=_0o2JKyrOHPfF/PI Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: 7bit On Thu, 4 Jun 2009 06:37:19 am john cooper wrote:
> This patch extracts the opaque data from pci i/o
> region 0 via the added VIRTIO_BLK_F_IDENTIFY
> field. By convention this data takes the form of
> that returned by an ATA IDENTIFY DEVICE command,
> however the driver (except for structure size)
> makes no interpretation of the data. The structure
> data is copied wholesale to userspace via a
> HDIO_GET_IDENTITY ioctl command (eg: hdparm -i <dev>).


Thanks, applied with one subtraction:


> +/* mapped into pci i/o region 0
> + */
> struct virtio_blk_config


I removed this comment; it's true for virtio_pci, but not true in general.


Thanks!
Rusty.

--Boundary-00=_0o2JKyrOHPfF/PI-- From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MC6fm-0000zr-A3 for qemu-devel@nongnu.org; Thu, 04 Jun 2009 02:43:35 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MC6fN-0000lP-In for qemu-devel@nongnu.org; Thu, 04 Jun 2009 02:43:17 -0400 Received: from [199.232.76.173] (port=42027 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MC6fN-0000hB-0d for qemu-devel@nongnu.org; Thu, 04 Jun 2009 02:43:09 -0400 Received: from ozlabs.org ([203.10.76.45]:42031) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1MC6U2-0003oB-RL for qemu-devel@nongnu.org; Thu, 04 Jun 2009 02:31:27 -0400 From: Rusty Russell Date: Thu, 4 Jun 2009 16:01:16 +0930 References: <4A1F6130.9070705@redhat.com> <4A26E607.3070901@redhat.com> In-Reply-To: <4A26E607.3070901@redhat.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="Boundary-00=_0o2JKyrOHPfF/PI" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200906041601.16539.rusty@rustcorp.com.au> Subject: [Qemu-devel] Re: [PATCH 2/2] Add serial number support for virtio_blk, V4a List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: john cooper Cc: qemu-devel@nongnu.org, KVM list , Christoph Hellwig --Boundary-00=_0o2JKyrOHPfF/PI Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit On Thu, 4 Jun 2009 06:37:19 am john cooper wrote: > This patch extracts the opaque data from pci i/o > region 0 via the added VIRTIO_BLK_F_IDENTIFY > field. By convention this data takes the form of > that returned by an ATA IDENTIFY DEVICE command, > however the driver (except for structure size) > makes no interpretation of the data. The structure > data is copied wholesale to userspace via a > HDIO_GET_IDENTITY ioctl command (eg: hdparm -i ). Thanks, applied with one subtraction: > +/* mapped into pci i/o region 0 > + */ > struct virtio_blk_config I removed this comment; it's true for virtio_pci, but not true in general. Thanks! Rusty. --Boundary-00=_0o2JKyrOHPfF/PI Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: 7bit On Thu, 4 Jun 2009 06:37:19 am john cooper wrote:
> This patch extracts the opaque data from pci i/o
> region 0 via the added VIRTIO_BLK_F_IDENTIFY
> field. By convention this data takes the form of
> that returned by an ATA IDENTIFY DEVICE command,
> however the driver (except for structure size)
> makes no interpretation of the data. The structure
> data is copied wholesale to userspace via a
> HDIO_GET_IDENTITY ioctl command (eg: hdparm -i <dev>).


Thanks, applied with one subtraction:


> +/* mapped into pci i/o region 0
> + */
> struct virtio_blk_config


I removed this comment; it's true for virtio_pci, but not true in general.


Thanks!
Rusty.

--Boundary-00=_0o2JKyrOHPfF/PI--