Skip to content

Don't allow burst limits to be lower than the normal limits

Alberto Garcia qemu-devel 2 versions – Applied

Status from this message: Thanks, applied to my block tree:

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest2 patches 2 repliesApplied
  2. v12 patches 3 repliesSuperseded

Changes in v2 from the cover letter

- Simplify error message [Eric]

Patches in v2 Applied

#Subject and filesSizeRepliesApplied
1/2throttle: Don't allow burst limits to be lower than the normal limitsutil/throttle.c+5 -00 replies
2/2throttle: Test burst limits lower than the normal limitstests/test-throttle.c+8 -00 replies

Reviews trailers per patch

None of the 2 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchEric Blake
SeriesR-b
1/2
2/2

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [Qemu-devel] [PATCH for-2.7 v2 0/2] Don't allow burst limits to be lower than the normal limits

Hello,

Gu Nini found this problem and reported it in

https://bugzilla.redhat.com/show_bug.cgi?id=1355665

When setting the throttling configuration, the burst limits can be lower than the normal limits. This does not making any sense and behaves oddly, so let's forbid it.

Berto

v2:
- Simplify error message [Eric]

Alberto Garcia (2):
throttle: Don't allow burst limits to be lower than the normal limits
throttle: Test burst limits lower than the normal limits

 tests/test-throttle.c | 8 ++++++++
 util/throttle.c       | 5 +++++
 2 files changed, 13 insertions(+)
Signature · 1 line
2.8.1