[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 11/11] 9p: darwin: Adjust assumption on virtio-9p-test
|
From: |
Will Cohen |
|
Subject: |
[PATCH v4 11/11] 9p: darwin: Adjust assumption on virtio-9p-test |
|
Date: |
Sun, 6 Feb 2022 15:07:19 -0500 |
The previous test depended on the assumption that P9_DOTL_AT_REMOVEDIR
and AT_REMOVEDIR have the same value.
While this is true on Linux, it is not true everywhere, and leads to an
incorrect test failure on unlink_at, noticed when adding 9p to darwin:
Received response 7 (RLERROR) instead of 77 (RUNLINKAT)
Rlerror has errno 22 (Invalid argument)
**
ERROR:../tests/qtest/virtio-9p-test.c:305:v9fs_req_recv: assertion
failed (hdr.id == id): (7 == 77) Bail out!
ERROR:../tests/qtest/virtio-9p-test.c:305:v9fs_req_recv: assertion
failed (hdr.id == id): (7 == 77)
Signed-off-by: Fabian Franz <fabianfranz.oss@gmail.com>
[Will Cohen: - Add explanation of patch and description
of pre-patch test failure]
Signed-off-by: Will Cohen <wwcohen@gmail.com>
---
tests/qtest/virtio-9p-test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/qtest/virtio-9p-test.c b/tests/qtest/virtio-9p-test.c
index 41fed41de1..6bcf89f0f8 100644
--- a/tests/qtest/virtio-9p-test.c
+++ b/tests/qtest/virtio-9p-test.c
@@ -1270,7 +1270,7 @@ static void fs_unlinkat_dir(void *obj, void *data,
QGuestAllocator *t_alloc)
/* ... and is actually a directory */
g_assert((st.st_mode & S_IFMT) == S_IFDIR);
- do_unlinkat(v9p, "/", "02", AT_REMOVEDIR);
+ do_unlinkat(v9p, "/", "02", P9_DOTL_AT_REMOVEDIR);
/* directory should be gone now */
g_assert(stat(new_dir, &st) != 0);
--
2.32.0 (Apple Git-132)
- [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, (continued)
- [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Will Cohen, 2022/02/06
- Re: [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Philippe Mathieu-Daudé, 2022/02/06
- Re: [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Will Cohen, 2022/02/06
- Re: [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Christian Schoenebeck, 2022/02/07
- Re: [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Will Cohen, 2022/02/07
- Re: [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Greg Kurz, 2022/02/07
- Re: [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Christian Schoenebeck, 2022/02/07
Re: [PATCH v4 10/11] 9p: darwin: meson: Allow VirtFS on Darwin, Christian Schoenebeck, 2022/02/07
[PATCH v4 11/11] 9p: darwin: Adjust assumption on virtio-9p-test,
Will Cohen <=
Re: [PATCH v4 00/11] 9p: Add support for darwin, Christian Schoenebeck, 2022/02/07