On Sun, Jul 02, 2017 at 01:06:46PM +0300, Manos Pitsidianakis wrote:
> throttle_config() cancels the timers of the calling BlockBackend. This
> doesn't make sense because other BlockBackends in the group remain
> untouched. There's no need to cancel the timers in the one specific
> BlockBackend so let's not do that. Throttled requests will run as
> scheduled and future requests will follow the new configuration. This
> also allows a throttle group's configuration to be changed even when it
> has no members.
> 
> Signed-off-by: Manos Pitsidianakis <el13...@mail.ntua.gr>
> ---
>  block/throttle-groups.c     | 10 +---------
>  fsdev/qemu-fsdev-throttle.c |  2 +-
>  include/qemu/throttle.h     |  1 -
>  tests/test-throttle.c       |  4 ++--
>  util/throttle.c             | 14 --------------
>  5 files changed, 4 insertions(+), 27 deletions(-)

Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>

Attachment: signature.asc
Description: PGP signature

Reply via email to