From mboxrd@z Thu Jan  1 00:00:00 1970
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Subject: Re: In blkif.h, sector unit is not clear
Date: Tue, 14 Oct 2014 11:49:32 -0400
Message-ID: <20141014154932.GD30965@laptop.dumpdata.com>
References: <20141008162053.GB1742@perard.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Content-Disposition: inline
In-Reply-To: <20141008162053.GB1742@perard.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Anthony PERARD <anthony.perard@citrix.com>
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>, Ian Campbell <ian.campbell@citrix.com>, Jan Beulich <jbeulich@suse.com>, Xen Devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On Wed, Oct 08, 2014 at 05:20:53PM +0100, Anthony PERARD wrote:
> I'm trying to understand the different device properties exposed on
> xenstore for a block device. And there is one property that is
> contradictory within blkif.h, "sectors". There are two possible units for
> it, either "sector-size" or 512-byte. Which one it is?
> 
> My guess is that it does not matter too much since "sector-size" is
> probably always reported as 512.


Looking at the code it appears that it will accept any 'sector-size'
values and program them to the Linux API. And I believe that for
CD-ROMs it will program them with the correct sector value (which is
2048).

But for the 'gref' and 'fsect','lset' it will always be in 512
byte units.

> 
> Here is the interesting part of xen/include/public/io/blkif.h:
> 
> @@ -198,1 +198,1 @@
>   *------------------------- Backend Device Properties -------------------------
> @@ -241,18 +241,18 @@
>   * sector-size
>   *      Values:         <uint32_t>
>   *
>   *      The logical sector size, in bytes, of the backend device.
>   *
>   * physical-sector-size
>   *      Values:         <uint32_t>
>   *
>   *      The physical sector size, in bytes, of the backend device.
>   *
>   * sectors
>   *      Values:         <uint64_t>
>   *
>   *      The size of the backend device, expressed in units of its logical
>   *      sector size ("sector-size").
> 
> HERE: unit for "sectors" is "sector-size"
> 
> @@ -547,10 +547,10 @@
>  /*
>   * NB. first_sect and last_sect in blkif_request_segment, as well as
>   * sector_number in blkif_request, are always expressed in 512-byte units.
>   * However they must be properly aligned to the real sector size of the
>   * physical disk, which is reported in the "physical-sector-size" node in
>   * the backend xenbus info. Also the xenbus "sectors" node is expressed in
>   * 512-byte units.
>   */
>  struct blkif_request_segment {
>      grant_ref_t gref;        /* reference to I/O buffer frame        */
> 
> HERE: unit for "sectors" is 512-byte
> 
> 
> Regards,
> 
> -- 
> Anthony PERARD