Hi, In this new version I added a handful of patches to make it easier to review, as suggested by Greg in v1.
Thomas, the changes made aren't negligible so I didn't add your Tested-by. You're more than welcome to give this version a try :) The first 2 patches are needed to ease the burden of individual test changes. It was really annoying to deal with changing the dir name every time a test was being removed. Parametrizing the dir name (and anything that has the dir in it) makes it trivial to change the dir name when needed. I kept the slow revert patch (7) because this series is passing the Gitlab CI with it: https://gitlab.com/danielhb/qemu/-/pipelines/1228813691 I'll leave to the 9p maintainers to decide whether we should go ahead with it or not. Changes from v1: - patches 1 and 2 from v1: dropped - patches 1 and 2 (new): - parametrize 'new_dir' name - patches 3, 4, 5, 6 (new): - individual test consolidation - v1 link: https://mail.gnu.org/archive/html/qemu-devel/2024-03/msg06146.html Daniel Henrique Barboza (7): qtest/virtio-9p-test.c: add '_path' in path vars qtest/virtio-9p-test.c: parametrize 'dir' name in local tests qtest/virtio-9p-test.c: consolidate create dir tests qtest/virtio-9p-test.c: consolidate create file tests qtest/virtio-9p-test.c: consolidate create symlink tests qtest/virtio-9p-test.c: consolidate hardlink tests qtest/virtio-9p-test.c: remove g_test_slow() gate tests/qtest/virtio-9p-test.c | 228 +++++++++++++++-------------------- 1 file changed, 96 insertions(+), 132 deletions(-) -- 2.44.0