Re: [Qemu-devel] [PATCH] block: fix initialization in bdrv_io_limits_enable()

2013-01-15 Thread Stefan Hajnoczi
On Fri, Jan 11, 2013 at 01:29:55PM +0100, Peter Lieven wrote: > bdrv_io_limits_enable() starts a new slice, but does not set io_base > correctly for that slice. > > Here is how io_base is used: > >bytes_base = bs->nr_bytes[is_write] - bs->io_base.bytes[is_write]; >bytes_res = (unsigned

Re: [Qemu-devel] [PATCH] block: fix initialization in bdrv_io_limits_enable()

2013-01-11 Thread Paolo Bonzini
Il 11/01/2013 13:29, Peter Lieven ha scritto: > bdrv_io_limits_enable() starts a new slice, but does not set io_base > correctly for that slice. > > Here is how io_base is used: > >bytes_base = bs->nr_bytes[is_write] - bs->io_base.bytes[is_write]; >bytes_res = (unsigned) nb_sectors * B

[Qemu-devel] [PATCH] block: fix initialization in bdrv_io_limits_enable()

2013-01-11 Thread Peter Lieven
bdrv_io_limits_enable() starts a new slice, but does not set io_base correctly for that slice. Here is how io_base is used: bytes_base = bs->nr_bytes[is_write] - bs->io_base.bytes[is_write]; bytes_res = (unsigned) nb_sectors * BDRV_SECTOR_SIZE; if (bytes_base + bytes_res <= bytes_li