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>
signature.asc
Description: PGP signature