On Thu, Aug 24, 2017 at 04:24:42PM +0300, Alberto Garcia wrote:
> Hi,
>
> here's the new version of my patch series with misc fixes for the
> throttling code.
>
> Stefan, once this is reviewed, can you please remove the "Make
> LeakyBucket.avg and LeakyBucket.max integer types" commit (id
> 218f470639117a8e39) from your block-next branch? This series contains
> a new version of that patch that replaces the one that you have.
>
> v2:
> - Patch 3: Make the code a bit more concise
> - Patch 5: LeakyBucket.avg and LeakyBucket.max are now unsigned
> - Patch 6: Make burst_length 64bit and check its range
> - Patch 7: Add unit tests
>
> v1: https://lists.gnu.org/archive/html/qemu-block/2017-08/msg00753.html
> - Initial version
>
> Output of backport-diff against v1:
>
> Key:
> [----] : patches are identical
> [####] : number of functional differences between upstream/downstream patch
> [down] : patch is downstream-only
> The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively
>
> 001/7:[----] [--] 'throttle: Fix wrong variable name in the header documentation'
> 002/7:[----] [--] 'throttle: Update the throttle_fix_bucket() documentation'
> 003/7:[down] 'throttle: Make throttle_is_valid() a bit less verbose'
> 004/7:[----] [--] 'throttle: Remove throttle_fix_bucket() / throttle_unfix_bucket()'
> 005/7:[0010] [FC] 'throttle: Make LeakyBucket.avg and LeakyBucket.max integer types'
> 006/7:[down] 'throttle: Make burst_length 64bit and add range checks'
> 007/7:[down] 'throttle: Test the valid range of config values'
>
> Alberto Garcia (7):
> throttle: Fix wrong variable name in the header documentation
> throttle: Update the throttle_fix_bucket() documentation
> throttle: Make throttle_is_valid() a bit less verbose
> throttle: Remove throttle_fix_bucket() / throttle_unfix_bucket()
> throttle: Make LeakyBucket.avg and LeakyBucket.max integer types
> throttle: Make burst_length 64bit and add range checks
> throttle: Test the valid range of config values
>
> include/qemu/throttle.h | 8 ++---
> tests/test-throttle.c | 80 ++++++++++++++++++++++++++++++++++++++++++++-
> util/throttle.c | 86 +++++++++++++++++++------------------------------
> 3 files changed, 117 insertions(+), 57 deletions(-)
>
> --
> 2.11.0
>
>
I have dropped the previous "Make LeakyBucket.avg and LeakyBucket.max
integer types" commit and merged this series instead.
Thanks, applied to my block-next tree:
https://github.com/stefanha/qemu/commits/block-next
Stefan