Re: [Qemu-devel] [PATCH v4 04/15] block: add image streaming block job

[Kevin Wolf]

Am 12.01.2012 12:39, schrieb Stefan Hajnoczi:
> On Thu, Jan 12, 2012 at 10:59 AM, Kevin Wolf <address@hidden> wrote:
>> Am 06.01.2012 15:01, schrieb Stefan Hajnoczi:
>>> +    buf = qemu_blockalign(bs, STREAM_BUFFER_SIZE);
>>> +    s->common.len = bdrv_getlength(bs);
>>
>> No error check?
> 
> Will fix.
> 
>>> +    bdrv_get_geometry(bs, (uint64_t *)&end);
>>
>> Why call bdrv_getlength() twice? end = s->common.len >> BDRV_SECTOR_BITS
>> should be the same.
> 
> Okay, I'll change it.  I got sick of BDRV_SECTOR_* and called twice instead.

Well, you can try and change everything in the streaming code to bytes
instead of sectors. We should probably do this sooner or later anyway.
Sectors of 512 bytes are a completely arbitrary unit that doesn't make
much sense generally.

Kevin