diff options
author | Joe Thornber <ejt@redhat.com> | 2019-10-29 10:33:31 +0000 |
---|---|---|
committer | Joe Thornber <ejt@redhat.com> | 2020-01-16 14:58:12 +0000 |
commit | 056eb0a8809aee3ca888aff4fe8e22bf8884a0cc (patch) | |
tree | 2650012654bf0a84670c697ca5cb0afc23437073 | |
parent | babde3da5530e4bbda5fb474fb5ea32a55adf16f (diff) | |
download | lvm2-056eb0a8809aee3ca888aff4fe8e22bf8884a0cc.tar.gz |
[bcache] add unit test
abort-forces-read
-rw-r--r-- | test/unit/bcache_t.c | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/test/unit/bcache_t.c b/test/unit/bcache_t.c index 668d24d77..2a8f931e4 100644 --- a/test/unit/bcache_t.c +++ b/test/unit/bcache_t.c @@ -837,6 +837,29 @@ static void test_abort_single_block(void *context) T_ASSERT(bcache_flush(cache)); } +static void test_abort_forces_reread(void *context) +{ + struct fixture *f = context; + struct mock_engine *me = f->me; + struct bcache *cache = f->cache; + struct block *b; + int fd = 17; + + _expect_read(me, fd, 0); + _expect(me, E_WAIT); + T_ASSERT(bcache_get(cache, fd, 0, GF_DIRTY, &b)); + bcache_put(b); + + bcache_abort_fd(cache, fd); + T_ASSERT(bcache_flush(cache)); + + // Check the block is re-read + _expect_read(me, fd, 0); + _expect(me, E_WAIT); + T_ASSERT(bcache_get(cache, fd, 0, 0, &b)); + bcache_put(b); +} + static void test_abort_only_specific_fd(void *context) { struct fixture *f = context; @@ -960,6 +983,7 @@ static struct test_suite *_small_tests(void) T("abort-with-no-blocks", "you can call abort, even if there are no blocks in the cache", test_abort_no_blocks); T("abort-single-block", "single block get silently discarded", test_abort_single_block); + T("abort-forces-read", "if a block has been discarded then another read is necc.", test_abort_forces_reread); T("abort-specific-fd", "abort doesn't effect other fds", test_abort_only_specific_fd); T("concurrent-reads-after-invalidate", "prefetch should still issue concurrent reads after invalidate", |