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

From: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
To: qemu-devel@nongnu.org
Cc: thuth@redhat.com, alistair.francis@wdc.com, groug@kaod.org,
	peter.maydell@linaro.org, qemu_oss@crudebyte.com,
	Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Subject: [PATCH for-9.0 v2 0/7] qtest/virtio-9p-test.c: fix slow tests
Date: Wed, 27 Mar 2024 06:31:37 -0300	[thread overview]
Message-ID: <20240327093144.781701-1-dbarboza@ventanamicro.com> (raw)

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