[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 3/8] test-write-threshold: rewrite test_threshold_(not_)trigge
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[PATCH v3 3/8] test-write-threshold: rewrite test_threshold_(not_)trigger tests |
Date: |
Thu, 6 May 2021 12:06:16 +0300 |
These tests use bdrv_write_threshold_exceeded() API, which is used only
for test (since pre-previous commit). Better is testing real API, which
is used in block.c as well.
So, let's call bdrv_write_threshold_check_write(), and check is
bs->write_threshold_offset cleared or not (it's cleared iff threshold
triggered).
Also we get rid of BdrvTrackedRequest use here. Note, that paranoiac
bdrv_check_request() calls were added in 8b1170012b1 to protect
BdrvTrackedRequest. Drop them now.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
---
tests/unit/test-write-threshold.c | 22 ++++------------------
1 file changed, 4 insertions(+), 18 deletions(-)
diff --git a/tests/unit/test-write-threshold.c
b/tests/unit/test-write-threshold.c
index fc1c45a2eb..fd40a815b8 100644
--- a/tests/unit/test-write-threshold.c
+++ b/tests/unit/test-write-threshold.c
@@ -55,41 +55,27 @@ static void test_threshold_multi_set_get(void)
static void test_threshold_not_trigger(void)
{
- uint64_t amount = 0;
uint64_t threshold = 4 * 1024 * 1024;
BlockDriverState bs;
- BdrvTrackedRequest req;
memset(&bs, 0, sizeof(bs));
- memset(&req, 0, sizeof(req));
- req.offset = 1024;
- req.bytes = 1024;
-
- bdrv_check_request(req.offset, req.bytes, &error_abort);
bdrv_write_threshold_set(&bs, threshold);
- amount = bdrv_write_threshold_exceeded(&bs, &req);
- g_assert_cmpuint(amount, ==, 0);
+ bdrv_write_threshold_check_write(&bs, 1024, 1024);
+ g_assert_cmpuint(bdrv_write_threshold_get(&bs), ==, threshold);
}
static void test_threshold_trigger(void)
{
- uint64_t amount = 0;
uint64_t threshold = 4 * 1024 * 1024;
BlockDriverState bs;
- BdrvTrackedRequest req;
memset(&bs, 0, sizeof(bs));
- memset(&req, 0, sizeof(req));
- req.offset = (4 * 1024 * 1024) - 1024;
- req.bytes = 2 * 1024;
-
- bdrv_check_request(req.offset, req.bytes, &error_abort);
bdrv_write_threshold_set(&bs, threshold);
- amount = bdrv_write_threshold_exceeded(&bs, &req);
- g_assert_cmpuint(amount, >=, 1024);
+ bdrv_write_threshold_check_write(&bs, threshold - 1024, 2 * 1024);
+ g_assert_cmpuint(bdrv_write_threshold_get(&bs), ==, 0);
}
typedef struct TestStruct {
--
2.29.2
- [PATCH v3 0/8] block: refactor write threshold, Vladimir Sementsov-Ogievskiy, 2021/05/06
- [PATCH v3 2/8] block: drop write notifiers, Vladimir Sementsov-Ogievskiy, 2021/05/06
- [PATCH v3 5/8] block/write-threshold: don't use aio context lock, Vladimir Sementsov-Ogievskiy, 2021/05/06
- [PATCH v3 1/8] block/write-threshold: don't use write notifiers, Vladimir Sementsov-Ogievskiy, 2021/05/06
- [PATCH v3 3/8] test-write-threshold: rewrite test_threshold_(not_)trigger tests,
Vladimir Sementsov-Ogievskiy <=
- [PATCH v3 8/8] write-threshold: deal with includes, Vladimir Sementsov-Ogievskiy, 2021/05/06
- [PATCH v3 4/8] block/write-threshold: drop extra APIs, Vladimir Sementsov-Ogievskiy, 2021/05/06
- [PATCH v3 6/8] test-write-threshold: drop extra tests, Vladimir Sementsov-Ogievskiy, 2021/05/06
- [PATCH v3 7/8] test-write-threshold: drop extra TestStruct structure, Vladimir Sementsov-Ogievskiy, 2021/05/06
- Re: [PATCH v3 0/8] block: refactor write threshold, Stefan Hajnoczi, 2021/05/12
- Re: [PATCH v3 0/8] block: refactor write threshold, Max Reitz, 2021/05/12