[v3] Misc fixes for throttle

[PATCH v3 0/6] Misc fixes for throttle

zhenwei pi posted 6 patches 10 months, 1 week ago

Diff against v1 v2 v4 v5
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230713064111.558652-1-pizhenwei@bytedance.com

Maintainers: "Gonglei (Arei)" <arei.gonglei@huawei.com>, Zhenwei Pi <pizhenwei@bytedance.com>, Alberto Garcia <berto@igalia.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Greg Kurz <groug@kaod.org>, Christian Schoenebeck <qemu_oss@crudebyte.com>

There is a newer version of this series

backends/cryptodev.c        | 12 +++---
block/throttle-groups.c     |  6 ++-
fsdev/qemu-fsdev-throttle.c |  8 ++--
include/qemu/throttle.h     | 15 +++++---
tests/unit/test-throttle.c  | 76 ++++++++++++++++++++++++++++++++++---
util/throttle.c             | 64 +++++++++++++++++++------------
6 files changed, 136 insertions(+), 45 deletions(-)

Expand all Fold all

[PATCH v3 0/6] Misc fixes for throttle

Posted by zhenwei pi 10 months, 1 week ago

v2 -> v3:
- patch 1 -> patch 5 are already reviewed by Alberto
- append patch 6: throttle: use enum ThrottleType instead of bool is_write

v1 -> v2:
- rename 'ThrottleTimerType' to 'ThrottleType'
- add assertion to throttle_schedule_timer

v1:
- introduce enum ThrottleTimerType instead of timers[0], timer[1]...
- support read-only and write-only for throttle
- adapt related test codes
- cryptodev uses a write-only throttle timer

Zhenwei Pi (6):
  throttle: introduce enum ThrottleType
  test-throttle: use enum ThrottleType
  throttle: support read-only and write-only
  test-throttle: test read only and write only
  cryptodev: use NULL throttle timer cb for read direction
  throttle: use enum ThrottleType instead of bool is_write

 backends/cryptodev.c        | 12 +++---
 block/throttle-groups.c     |  6 ++-
 fsdev/qemu-fsdev-throttle.c |  8 ++--
 include/qemu/throttle.h     | 15 +++++---
 tests/unit/test-throttle.c  | 76 ++++++++++++++++++++++++++++++++++---
 util/throttle.c             | 64 +++++++++++++++++++------------
 6 files changed, 136 insertions(+), 45 deletions(-)

-- 
2.34.1

PING: [PATCH v3 0/6] Misc fixes for throttle

Posted by zhenwei pi 9 months, 4 weeks ago

Hi Kevin, Hanna,

Patch 1 -> patch 5 of this series are already reviewed by Alberto(these 
affects throttle framework only), the patch 6 affects qemu block layer, 
would you please review this(in the further step, merge this series if 
this is acceptable)?

On 7/13/23 14:41, zhenwei pi wrote:
> v2 -> v3:
> - patch 1 -> patch 5 are already reviewed by Alberto
> - append patch 6: throttle: use enum ThrottleType instead of bool is_write
> 
> v1 -> v2:
> - rename 'ThrottleTimerType' to 'ThrottleType'
> - add assertion to throttle_schedule_timer
> 
> v1:
> - introduce enum ThrottleTimerType instead of timers[0], timer[1]...
> - support read-only and write-only for throttle
> - adapt related test codes
> - cryptodev uses a write-only throttle timer
> 
> Zhenwei Pi (6):
>    throttle: introduce enum ThrottleType
>    test-throttle: use enum ThrottleType
>    throttle: support read-only and write-only
>    test-throttle: test read only and write only
>    cryptodev: use NULL throttle timer cb for read direction
>    throttle: use enum ThrottleType instead of bool is_write
> 
>   backends/cryptodev.c        | 12 +++---
>   block/throttle-groups.c     |  6 ++-
>   fsdev/qemu-fsdev-throttle.c |  8 ++--
>   include/qemu/throttle.h     | 15 +++++---
>   tests/unit/test-throttle.c  | 76 ++++++++++++++++++++++++++++++++++---
>   util/throttle.c             | 64 +++++++++++++++++++------------
>   6 files changed, 136 insertions(+), 45 deletions(-)
> 

-- 
zhenwei pi