Re: [Qemu-block] [PATCH v2 05/13] block: Move throttling fields from BDS to BB

2016-05-11 Thread Kevin Wolf
Am 09.05.2016 um 14:43 hat Stefan Hajnoczi geschrieben: > On Fri, Apr 22, 2016 at 07:42:34PM +0200, Kevin Wolf wrote: > > @@ -1527,7 +1525,7 @@ static int bdrv_open_inherit(BlockDriverState **pbs, > > const char *filename, > > return -ENODEV; > > } > > > > -if (bs->

Re: [Qemu-block] [PATCH v2 05/13] block: Move throttling fields from BDS to BB

2016-05-10 Thread Alberto Garcia
On Fri 22 Apr 2016 07:42:34 PM CEST, Kevin Wolf wrote: > typedef struct BlockBackendPublic { > -/* I/O throttling */ > +/* I/O throttling. > + * throttle_state tells us if this BDS has I/O limits configured. > + * io_limits_disabled tells us if they are currently being enforced */

Re: [Qemu-block] [PATCH v2 05/13] block: Move throttling fields from BDS to BB

2016-05-09 Thread Stefan Hajnoczi
On Fri, Apr 22, 2016 at 07:42:34PM +0200, Kevin Wolf wrote: > @@ -1527,7 +1525,7 @@ static int bdrv_open_inherit(BlockDriverState **pbs, > const char *filename, > return -ENODEV; > } > > -if (bs->throttle_state) { > +if (blk_get_public(bs->blk)->throttle_sta

Re: [Qemu-block] [PATCH v2 05/13] block: Move throttling fields from BDS to BB

2016-05-04 Thread Alberto Garcia
On Fri 22 Apr 2016 07:42:34 PM CEST, Kevin Wolf wrote: > This patch changes where the throttling state is stored (used to be the > BlockDriverState, now it is the BlockBackend), but it doesn't actually > make it a BB level feature yet. For example, throttling is still > disabled when the BDS is det

[Qemu-block] [PATCH v2 05/13] block: Move throttling fields from BDS to BB

2016-04-22 Thread Kevin Wolf
This patch changes where the throttling state is stored (used to be the BlockDriverState, now it is the BlockBackend), but it doesn't actually make it a BB level feature yet. For example, throttling is still disabled when the BDS is detached from the BB. Signed-off-by: Kevin Wolf --- block.c