Re: [Qemu-devel] [PATCH 1/6] throttle: Extract timers from ThrottleState into a separate structure

[Stefan Hajnoczi]

On Tue, Mar 10, 2015 at 05:30:45PM +0200, Alberto Garcia wrote:
> From: Benoît Canet <address@hidden>
> 
> Group throttling will share ThrottleState between multiple bs.
> As a consequence the ThrottleState will be accessed by multiple aio
> context.
> 
> Timers are tied to their aio context so they must go out of the
> ThrottleState structure.
> 
> This commit paves the way for each bs of a common ThrottleState to
> have its own timer.
> 
> Signed-off-by: Benoit Canet <address@hidden>
> Signed-off-by: Alberto Garcia <address@hidden>
> ---
>  block.c                   | 35 ++++++++++++--------
>  include/block/block_int.h |  1 +
>  include/qemu/throttle.h   | 38 ++++++++++++++--------
>  tests/test-throttle.c     | 82 
> ++++++++++++++++++++++++++---------------------
>  util/throttle.c           | 73 ++++++++++++++++++++++++-----------------
>  5 files changed, 135 insertions(+), 94 deletions(-)

Reviewed-by: Stefan Hajnoczi <address@hidden>

pgpn_zhFJpHJu.pgp
Description: PGP signature