Re: [Qemu-block] [PATCH 3/7] block: move restarting of throttled reqs to block/throttle-groups.c

2016-03-29 Thread Paolo Bonzini
On 29/03/2016 16:14, Alberto Garcia wrote: > On Thu 24 Mar 2016 05:39:22 PM CET, Paolo Bonzini wrote: >> @@ -335,6 +346,11 @@ void throttle_group_config(BlockDriverState *bs, >> ThrottleConfig *cfg) >> } >> throttle_config(ts, tt, cfg); >> qemu_mutex_unlock(&tg->lock); >> + >> +

Re: [Qemu-block] [PATCH 3/7] block: move restarting of throttled reqs to block/throttle-groups.c

2016-03-29 Thread Alberto Garcia
On Thu 24 Mar 2016 05:39:22 PM CET, Paolo Bonzini wrote: > @@ -335,6 +346,11 @@ void throttle_group_config(BlockDriverState *bs, > ThrottleConfig *cfg) > } > throttle_config(ts, tt, cfg); > qemu_mutex_unlock(&tg->lock); > + > +aio_context_acquire(bdrv_get_aio_context(bs)); > +

[Qemu-block] [PATCH 3/7] block: move restarting of throttled reqs to block/throttle-groups.c

2016-03-24 Thread Paolo Bonzini
We want to remove throttled_reqs from block/io.c. This is the easy part---hide the handling of throttled_reqs during disable/enable of throttling within throttle-groups.c. Signed-off-by: Paolo Bonzini --- v1->v2: only restart first request after throttle_group_config --- block/io.c