[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 08/21] tests: test-replication disable /replication/secondary/
From: |
Yonggang Luo |
Subject: |
[PATCH v2 08/21] tests: test-replication disable /replication/secondary/* on msys2/mingw. |
Date: |
Wed, 9 Sep 2020 17:46:04 +0800 |
They cause failure on msys2/mingw, that's because file-win32.c not implement
.bdrv_reopen_prepare/commit/abort yet.
> $ ./tests/test-replication.exe
> # random seed: R02S3f4d1c01af2b0a046990e0235c481faf
> 1..13
> # Start of replication tests
> # Start of primary tests
> ok 1 /replication/primary/read
> ok 2 /replication/primary/write
> ok 3 /replication/primary/start
> ok 4 /replication/primary/stop
> ok 5 /replication/primary/do_checkpoint
> ok 6 /replication/primary/get_error_all
> # End of primary tests
> # Start of secondary tests
> ok 7 /replication/secondary/read
> ok 8 /replication/secondary/write
> Unexpected error in bdrv_reopen_prepare() at ../block.c:4191:
> Block format 'file' used by node '#block4287' does not support reopening
> files
Can you try to find out what reopen this is?
I assume it's for switching between read-write and read-only. In this
case an implementation of .bdrv_reopen_prepare/commit/abort that can do
this switch is required.
This is more serious development work, so I can't propose a quick fix.
Alternatively, we could just declare replication unsupported on Windows
and let configure make sure that CONFIG_REPLICATION is never set for it.
<stefanha> luoyonggang: That might be missing functionality in
block/file-win32.c.
* davidgiluk yawns and looks up
<stefanha> luoyonggang: The block/file-posix.c block driver supports
.bdrv_reopen_*()
while block/file-win32.c does not. It's probably because no one has tried to
implement it.
<luoyonggang> OK, I got the direction,
<luoyonggang> Just need implement .bdrv_reopen_*() functions in file-win32.c
Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
---
tests/test-replication.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/tests/test-replication.c b/tests/test-replication.c
index e7cbd6b144..b067240add 100644
--- a/tests/test-replication.c
+++ b/tests/test-replication.c
@@ -392,6 +392,7 @@ static void test_secondary_write(void)
teardown_secondary();
}
+#ifndef _WIN32
static void test_secondary_start(void)
{
BlockBackend *top_blk, *local_blk;
@@ -546,6 +547,7 @@ static void test_secondary_get_error_all(void)
teardown_secondary();
}
+#endif
static void sigabrt_handler(int signo)
{
@@ -597,6 +599,7 @@ int main(int argc, char **argv)
/* Secondary */
g_test_add_func("/replication/secondary/read", test_secondary_read);
g_test_add_func("/replication/secondary/write", test_secondary_write);
+#ifndef _WIN32
g_test_add_func("/replication/secondary/start", test_secondary_start);
g_test_add_func("/replication/secondary/stop", test_secondary_stop);
g_test_add_func("/replication/secondary/continuous_replication",
@@ -605,6 +608,7 @@ int main(int argc, char **argv)
test_secondary_do_checkpoint);
g_test_add_func("/replication/secondary/get_error_all",
test_secondary_get_error_all);
+#endif
ret = g_test_run();
--
2.28.0.windows.1
- Re: [PATCH v2 03/21] configure: Fixes ncursesw detection under msys2/mingw and enable curses, (continued)
- [PATCH v2 04/21] curses: Fixes curses compiling errors., Yonggang Luo, 2020/09/09
- [PATCH v2 05/21] tests: disable /char/stdio/* tests in test-char.c on win32, Yonggang Luo, 2020/09/09
- [PATCH v2 06/21] ci: Enable msys2 ci in cirrus, Yonggang Luo, 2020/09/09
- [PATCH v2 07/21] tests: Trying fixes test-replication.c on msys2/mingw., Yonggang Luo, 2020/09/09
- [PATCH v2 08/21] tests: test-replication disable /replication/secondary/* on msys2/mingw.,
Yonggang Luo <=
- [PATCH v2 09/21] osdep: These function are only available on Non-Win32 system., Yonggang Luo, 2020/09/09
- [PATCH v2 10/21] meson: Use -b to ignore CR vs. CR-LF issues on Windows, Yonggang Luo, 2020/09/09
- [PATCH v2 11/21] meson: disable crypto tests are empty under win32, Yonggang Luo, 2020/09/09
- [PATCH v2 12/21] meson: remove empty else and duplicated gio deps, Yonggang Luo, 2020/09/09
- [PATCH v2 13/21] vmstate: Fixes test-vmstate.c on msys2/mingw, Yonggang Luo, 2020/09/09