Re: [Qemu-devel] [PATCH v4 09/20] parallels: Switch to .bdrv_co_block_status()

2017-12-01 Thread Eric Blake
On 11/30/2017 04:12 PM, Eric Blake wrote:   BDRVParallelsState *s = bs->opaque; -    int64_t offset; +    int count; +    assert(QEMU_IS_ALIGNED(offset | bytes, BDRV_SECTOR_SIZE));   qemu_co_mutex_lock(&s->lock); -    offset = block_status(s, sector_num, nb_sectors, pnum); +    offset =

Re: [Qemu-devel] [PATCH v4 09/20] parallels: Switch to .bdrv_co_block_status()

2017-12-01 Thread Eric Blake
On 11/30/2017 03:03 AM, Vladimir Sementsov-Ogievskiy wrote: 12.10.2017 21:59, Eric Blake wrote: We are gradually moving away from sector-based interfaces, towards byte-based.  Update the parallels driver accordingly.  Note that the internal function block_status() is still sector-based, because

Re: [Qemu-devel] [PATCH v4 09/20] parallels: Switch to .bdrv_co_block_status()

2017-11-30 Thread Vladimir Sementsov-Ogievskiy
12.10.2017 21:59, Eric Blake wrote: We are gradually moving away from sector-based interfaces, towards byte-based. Update the parallels driver accordingly. Note that the internal function block_status() is still sector-based, because it is still in use by other sector-based functions; but that'

[Qemu-devel] [PATCH v4 09/20] parallels: Switch to .bdrv_co_block_status()

2017-10-12 Thread Eric Blake
We are gradually moving away from sector-based interfaces, towards byte-based. Update the parallels driver accordingly. Note that the internal function block_status() is still sector-based, because it is still in use by other sector-based functions; but that's okay because request_alignment is 51