Re: [Qemu-devel] [PATCH v3 0/3] scsi-generic and BLKSECTGET

2017-02-09 Thread Paolo Bonzini
On 08/02/2017 23:33, Max Reitz wrote: > Noticed now that Paolo has already merged this. Well, then out it goes. > Too bad the swap of BDRV_REQUEST_MAX_BYTES and BDRV_REQUEST_MAX_SECTORS > is not in his tree. > > Well, I'll make it an own patch then. Sorry, that was by mistake. :( Paolo

Re: [Qemu-devel] [PATCH v3 0/3] scsi-generic and BLKSECTGET

2017-01-24 Thread Eric Farman
On 01/24/2017 07:09 AM, Fam Zheng wrote: On Tue, 01/24 12:23, Paolo Bonzini wrote: On 22/01/2017 15:29, Fam Zheng wrote: On Fri, 01/20 17:25, Eric Farman wrote: Changes: v2->v3: - Move byte/sector conversions to patch 2 [Fam Zheng] - Rename "max_sectors" when holding a byte value [Fam

Re: [Qemu-devel] [PATCH v3 0/3] scsi-generic and BLKSECTGET

2017-01-24 Thread Fam Zheng
On Tue, 01/24 12:23, Paolo Bonzini wrote: > > > On 22/01/2017 15:29, Fam Zheng wrote: > > On Fri, 01/20 17:25, Eric Farman wrote: > >> Changes: > >> v2->v3: > >> - Move byte/sector conversions to patch 2 [Fam Zheng] > >> - Rename "max_sectors" when holding a byte value [Fam Zheng] > >> v1->

Re: [Qemu-devel] [PATCH v3 0/3] scsi-generic and BLKSECTGET

2017-01-24 Thread Paolo Bonzini
On 22/01/2017 15:29, Fam Zheng wrote: > On Fri, 01/20 17:25, Eric Farman wrote: >> Changes: >> v2->v3: >> - Move byte/sector conversions to patch 2 [Fam Zheng] >> - Rename "max_sectors" when holding a byte value [Fam Zheng] >> v1->v2: >> - Patch 3, make hdev_get_max_transfer_length return

Re: [Qemu-devel] [PATCH v3 0/3] scsi-generic and BLKSECTGET

2017-01-22 Thread Fam Zheng
On Fri, 01/20 17:25, Eric Farman wrote: > Changes: > v2->v3: > - Move byte/sector conversions to patch 2 [Fam Zheng] > - Rename "max_sectors" when holding a byte value [Fam Zheng] > v1->v2: > - Patch 3, make hdev_get_max_transfer_length return bytes [Fam Zheng] Reviewed-by: Fam Zheng

[Qemu-devel] [PATCH v3 0/3] scsi-generic and BLKSECTGET

2017-01-20 Thread Eric Farman
In the Linux kernel, I see two (three) places where the BLKSECTGET ioctl is handled: (1) block/(compat_)ioctl.c -- (compat_)blkdev_ioctl (2) drivers/scsi/sg.c -- sg_ioctl The former has been around forever[1], and returns a short value measured in sectors. A sector is generally assumed to be 512