[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL for-2.10 08/15] throttle: Test the valid range of con
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PULL for-2.10 08/15] throttle: Test the valid range of config values |
Date: |
Thu, 31 Aug 2017 09:22:03 +0100 |
From: Alberto Garcia <address@hidden>
Signed-off-by: Alberto Garcia <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
tests/test-throttle.c | 77 +++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 77 insertions(+)
diff --git a/tests/test-throttle.c b/tests/test-throttle.c
index 41c0dd2529..bf7a5a648a 100644
--- a/tests/test-throttle.c
+++ b/tests/test-throttle.c
@@ -378,6 +378,82 @@ static void test_is_valid(void)
test_is_valid_for_value(1, true);
}
+static void test_ranges(void)
+{
+ int i;
+
+ for (i = 0; i < BUCKETS_COUNT; i++) {
+ LeakyBucket *b = &cfg.buckets[i];
+ throttle_config_init(&cfg);
+
+ /* avg = 0 means throttling is disabled, but the config is valid */
+ b->avg = 0;
+ g_assert(throttle_is_valid(&cfg, NULL));
+ g_assert(!throttle_enabled(&cfg));
+
+ /* These are valid configurations (values <= THROTTLE_VALUE_MAX) */
+ b->avg = 1;
+ g_assert(throttle_is_valid(&cfg, NULL));
+
+ b->avg = THROTTLE_VALUE_MAX;
+ g_assert(throttle_is_valid(&cfg, NULL));
+
+ b->avg = THROTTLE_VALUE_MAX;
+ b->max = THROTTLE_VALUE_MAX;
+ g_assert(throttle_is_valid(&cfg, NULL));
+
+ /* Values over THROTTLE_VALUE_MAX are not allowed */
+ b->avg = THROTTLE_VALUE_MAX + 1;
+ g_assert(!throttle_is_valid(&cfg, NULL));
+
+ b->avg = THROTTLE_VALUE_MAX;
+ b->max = THROTTLE_VALUE_MAX + 1;
+ g_assert(!throttle_is_valid(&cfg, NULL));
+
+ /* burst_length must be between 1 and THROTTLE_VALUE_MAX */
+ b->avg = 1;
+ b->max = 1;
+ b->burst_length = 0;
+ g_assert(!throttle_is_valid(&cfg, NULL));
+
+ b->avg = 1;
+ b->max = 1;
+ b->burst_length = 1;
+ g_assert(throttle_is_valid(&cfg, NULL));
+
+ b->avg = 1;
+ b->max = 1;
+ b->burst_length = THROTTLE_VALUE_MAX;
+ g_assert(throttle_is_valid(&cfg, NULL));
+
+ b->avg = 1;
+ b->max = 1;
+ b->burst_length = THROTTLE_VALUE_MAX + 1;
+ g_assert(!throttle_is_valid(&cfg, NULL));
+
+ /* burst_length * max cannot exceed THROTTLE_VALUE_MAX */
+ b->avg = 1;
+ b->max = 2;
+ b->burst_length = THROTTLE_VALUE_MAX / 2;
+ g_assert(throttle_is_valid(&cfg, NULL));
+
+ b->avg = 1;
+ b->max = 3;
+ b->burst_length = THROTTLE_VALUE_MAX / 2;
+ g_assert(!throttle_is_valid(&cfg, NULL));
+
+ b->avg = 1;
+ b->max = THROTTLE_VALUE_MAX;
+ b->burst_length = 1;
+ g_assert(throttle_is_valid(&cfg, NULL));
+
+ b->avg = 1;
+ b->max = THROTTLE_VALUE_MAX;
+ b->burst_length = 2;
+ g_assert(!throttle_is_valid(&cfg, NULL));
+ }
+}
+
static void test_max_is_missing_limit(void)
{
int i;
@@ -669,6 +745,7 @@ int main(int argc, char **argv)
g_test_add_func("/throttle/config/enabled", test_enabled);
g_test_add_func("/throttle/config/conflicting", test_conflicting_config);
g_test_add_func("/throttle/config/is_valid", test_is_valid);
+ g_test_add_func("/throttle/config/ranges", test_ranges);
g_test_add_func("/throttle/config/max", test_max_is_missing_limit);
g_test_add_func("/throttle/config/iops_size",
test_iops_size_is_missing_limit);
--
2.13.5
- [Qemu-devel] [PULL for-2.10 00/15] Block patches, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 01/15] nvme: Fix get/set number of queues feature, again, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 02/15] throttle: Fix wrong variable name in the header documentation, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 03/15] throttle: Update the throttle_fix_bucket() documentation, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 04/15] throttle: Make throttle_is_valid() a bit less verbose, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 05/15] throttle: Remove throttle_fix_bucket() / throttle_unfix_bucket(), Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 06/15] throttle: Make LeakyBucket.avg and LeakyBucket.max integer types, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 07/15] throttle: Make burst_length 64bit and add range checks, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 08/15] throttle: Test the valid range of config values,
Stefan Hajnoczi <=
- [Qemu-devel] [PULL for-2.10 09/15] oslib-posix: Print errors before aborting on qemu_alloc_stack(), Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 10/15] misc: Remove unused Error variables, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 12/15] docker.py: Python 2.6 argparse compatibility, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 13/15] tests: migration/guestperf Python 2.6 argparse compatibility, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 14/15] qemu-doc: Add UUID support in initiator name, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 11/15] scripts: add argparse module for Python 2.6 compatibility, Stefan Hajnoczi, 2017/08/31
- [Qemu-devel] [PULL for-2.10 15/15] qcow2: allocate cluster_cache/cluster_data on demand, Stefan Hajnoczi, 2017/08/31
- Re: [Qemu-devel] [PULL for-2.10 00/15] Block patches, no-reply, 2017/08/31
- Re: [Qemu-devel] [PULL for-2.10 00/15] Block patches, Eric Blake, 2017/08/31
- Re: [Qemu-devel] [PULL for-2.10 00/15] Block patches, Peter Maydell, 2017/08/31