[PATCH for-9.0 v2 0/7] qtest/virtio-9p-test.c: fix slow tests

Daniel Henrique Barboza posted 7 patches 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240327093144.781701-1-dbarboza@ventanamicro.com
Maintainers: Greg Kurz <groug@kaod.org>, Christian Schoenebeck <qemu_oss@crudebyte.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
There is a newer version of this series
tests/qtest/virtio-9p-test.c | 228 +++++++++++++++--------------------
1 file changed, 96 insertions(+), 132 deletions(-)
[PATCH for-9.0 v2 0/7] qtest/virtio-9p-test.c: fix slow tests
Posted by Daniel Henrique Barboza 1 month ago
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