[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 10/19] 9p: darwin: Adjust assumption on virtio-9p-test
From: |
Christian Schoenebeck |
Subject: |
[PULL 10/19] 9p: darwin: Adjust assumption on virtio-9p-test |
Date: |
Fri, 04 Mar 2022 13:27:49 +0100 |
From: Will Cohen <wwcohen@gmail.com>
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>
Acked-by: Thomas Huth <thuth@redhat.com>
[Will Cohen: - Move this patch before 9p: darwin: meson
patch to avoid qtest breakage during
bisecting]
Signed-off-by: Will Cohen <wwcohen@gmail.com>
Reviewed-by: Greg Kurz <groug@kaod.org>
Message-Id: <20220227223522.91937-11-wwcohen@gmail.com>
Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.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 502e5ad0c7..01ca076afe 100644
--- a/tests/qtest/virtio-9p-test.c
+++ b/tests/qtest/virtio-9p-test.c
@@ -1253,7 +1253,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.20.1
- [PULL 01/19] 9p: linux: Fix a couple Linux assumptions, (continued)
- [PULL 01/19] 9p: linux: Fix a couple Linux assumptions, Christian Schoenebeck, 2022/03/04
- [PULL 17/19] 9pfs/9p-util.h: convert Doxygen -> kerneldoc format, Christian Schoenebeck, 2022/03/04
- [PULL 16/19] 9pfs/9p.c: convert Doxygen -> kerneldoc format, Christian Schoenebeck, 2022/03/04
- [PULL 11/19] 9p: darwin: meson: Allow VirtFS on Darwin, Christian Schoenebeck, 2022/03/04
- [PULL 04/19] 9p: darwin: Handle struct dirent differences, Christian Schoenebeck, 2022/03/04
- [PULL 08/19] 9p: darwin: Compatibility for f/l*xattr, Christian Schoenebeck, 2022/03/04
- [PULL 02/19] 9p: Rename 9p-util -> 9p-util-linux, Christian Schoenebeck, 2022/03/04
- [PULL 09/19] 9p: darwin: Implement compatibility for mknodat, Christian Schoenebeck, 2022/03/04
- [PULL 18/19] 9pfs/coth.h: drop Doxygen format on v9fs_co_run_in_worker(), Christian Schoenebeck, 2022/03/04
- [PULL 05/19] 9p: darwin: Ignore O_{NOATIME, DIRECT}, Christian Schoenebeck, 2022/03/04
- [PULL 10/19] 9p: darwin: Adjust assumption on virtio-9p-test,
Christian Schoenebeck <=
- [PULL 06/19] 9p: darwin: Move XATTR_SIZE_MAX->P9_XATTR_SIZE_MAX, Christian Schoenebeck, 2022/03/04
- [PULL 14/19] 9pfs/9p.h: convert Doxygen -> kerneldoc format, Christian Schoenebeck, 2022/03/04
- [PULL 03/19] 9p: darwin: Handle struct stat(fs) differences, Christian Schoenebeck, 2022/03/04
- [PULL 12/19] 9pfs: move qemu_dirent_dup() from osdep -> 9p-util, Christian Schoenebeck, 2022/03/04
- Re: [PULL 00/19] 9p queue 2022-03-04, Peter Maydell, 2022/03/04