[PATCH 20/24] tests/functional: Move sh4/sh4eb tests into target-specific folders

Thomas Huth posted 24 patches 3 months, 2 weeks ago
There is a newer version of this series
[PATCH 20/24] tests/functional: Move sh4/sh4eb tests into target-specific folders
Posted by Thomas Huth 3 months, 2 weeks ago
From: Thomas Huth <thuth@redhat.com>

The tests/functional folder has become quite crowded, thus move the
sh4 tests into a target-specific subfolder.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MAINTAINERS                                       |  4 ++--
 tests/functional/meson.build                      | 15 ++-------------
 tests/functional/sh4/meson.build                  | 10 ++++++++++
 .../{test_sh4_r2d.py => sh4/test_r2d.py}          |  0
 .../{test_sh4_tuxrun.py => sh4/test_tuxrun.py}    |  0
 tests/functional/sh4eb/meson.build                |  5 +++++
 .../{test_sh4eb_r2d.py => sh4eb/test_r2d.py}      |  0
 7 files changed, 19 insertions(+), 15 deletions(-)
 create mode 100644 tests/functional/sh4/meson.build
 rename tests/functional/{test_sh4_r2d.py => sh4/test_r2d.py} (100%)
 rename tests/functional/{test_sh4_tuxrun.py => sh4/test_tuxrun.py} (100%)
 create mode 100644 tests/functional/sh4eb/meson.build
 rename tests/functional/{test_sh4eb_r2d.py => sh4eb/test_r2d.py} (100%)

diff --git a/MAINTAINERS b/MAINTAINERS
index 1f6b85ee132..a0174fca572 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1732,8 +1732,8 @@ F: hw/pci-host/sh_pci.c
 F: hw/timer/sh_timer.c
 F: include/hw/sh4/sh_intc.h
 F: include/hw/timer/tmu012.h
-F: tests/functional/test_sh4*_r2d.py
-F: tests/functional/test_sh4_tuxrun.py
+F: tests/functional/sh4*/test_r2d.py
+F: tests/functional/sh4/test_tuxrun.py
 
 SPARC Machines
 --------------
diff --git a/tests/functional/meson.build b/tests/functional/meson.build
index abaa4e00fca..ce713509e32 100644
--- a/tests/functional/meson.build
+++ b/tests/functional/meson.build
@@ -30,10 +30,8 @@ subdir('riscv32')
 subdir('riscv64')
 subdir('rx')
 subdir('s390x')
-
-test_sh4_timeouts = {
-  'sh4_tuxrun' : 240,
-}
+subdir('sh4')
+subdir('sh4eb')
 
 test_x86_64_timeouts = {
   'acpi_bits' : 420,
@@ -56,15 +54,6 @@ tests_generic_linuxuser = [
 tests_generic_bsduser = [
 ]
 
-tests_sh4_system_thorough = [
-  'sh4_r2d',
-  'sh4_tuxrun',
-]
-
-tests_sh4eb_system_thorough = [
-  'sh4eb_r2d',
-]
-
 tests_sparc_system_quick = [
   'sparc_migration',
 ]
diff --git a/tests/functional/sh4/meson.build b/tests/functional/sh4/meson.build
new file mode 100644
index 00000000000..56f824e1e71
--- /dev/null
+++ b/tests/functional/sh4/meson.build
@@ -0,0 +1,10 @@
+# SPDX-License-Identifier: GPL-2.0-or-later
+
+test_sh4_timeouts = {
+  'tuxrun' : 240,
+}
+
+tests_sh4_system_thorough = [
+  'r2d',
+  'tuxrun',
+]
diff --git a/tests/functional/test_sh4_r2d.py b/tests/functional/sh4/test_r2d.py
similarity index 100%
rename from tests/functional/test_sh4_r2d.py
rename to tests/functional/sh4/test_r2d.py
diff --git a/tests/functional/test_sh4_tuxrun.py b/tests/functional/sh4/test_tuxrun.py
similarity index 100%
rename from tests/functional/test_sh4_tuxrun.py
rename to tests/functional/sh4/test_tuxrun.py
diff --git a/tests/functional/sh4eb/meson.build b/tests/functional/sh4eb/meson.build
new file mode 100644
index 00000000000..25e9a6e4041
--- /dev/null
+++ b/tests/functional/sh4eb/meson.build
@@ -0,0 +1,5 @@
+# SPDX-License-Identifier: GPL-2.0-or-later
+
+tests_sh4eb_system_thorough = [
+  'r2d',
+]
diff --git a/tests/functional/test_sh4eb_r2d.py b/tests/functional/sh4eb/test_r2d.py
similarity index 100%
rename from tests/functional/test_sh4eb_r2d.py
rename to tests/functional/sh4eb/test_r2d.py
-- 
2.50.1
Re: [PATCH 20/24] tests/functional: Move sh4/sh4eb tests into target-specific folders
Posted by Philippe Mathieu-Daudé 3 months, 1 week ago
On 1/8/25 17:12, Thomas Huth wrote:
> From: Thomas Huth <thuth@redhat.com>
> 
> The tests/functional folder has become quite crowded, thus move the
> sh4 tests into a target-specific subfolder.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   MAINTAINERS                                       |  4 ++--
>   tests/functional/meson.build                      | 15 ++-------------
>   tests/functional/sh4/meson.build                  | 10 ++++++++++
>   .../{test_sh4_r2d.py => sh4/test_r2d.py}          |  0
>   .../{test_sh4_tuxrun.py => sh4/test_tuxrun.py}    |  0
>   tests/functional/sh4eb/meson.build                |  5 +++++
>   .../{test_sh4eb_r2d.py => sh4eb/test_r2d.py}      |  0
>   7 files changed, 19 insertions(+), 15 deletions(-)
>   create mode 100644 tests/functional/sh4/meson.build
>   rename tests/functional/{test_sh4_r2d.py => sh4/test_r2d.py} (100%)
>   rename tests/functional/{test_sh4_tuxrun.py => sh4/test_tuxrun.py} (100%)
>   create mode 100644 tests/functional/sh4eb/meson.build
>   rename tests/functional/{test_sh4eb_r2d.py => sh4eb/test_r2d.py} (100%)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1f6b85ee132..a0174fca572 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1732,8 +1732,8 @@ F: hw/pci-host/sh_pci.c
>   F: hw/timer/sh_timer.c
>   F: include/hw/sh4/sh_intc.h
>   F: include/hw/timer/tmu012.h
> -F: tests/functional/test_sh4*_r2d.py
> -F: tests/functional/test_sh4_tuxrun.py
> +F: tests/functional/sh4*/test_r2d.py
> +F: tests/functional/sh4/test_tuxrun.py
>   
>   SPARC Machines
>   --------------
> diff --git a/tests/functional/meson.build b/tests/functional/meson.build
> index abaa4e00fca..ce713509e32 100644
> --- a/tests/functional/meson.build
> +++ b/tests/functional/meson.build
> @@ -30,10 +30,8 @@ subdir('riscv32')
>   subdir('riscv64')
>   subdir('rx')
>   subdir('s390x')
> -
> -test_sh4_timeouts = {
> -  'sh4_tuxrun' : 240,
> -}
> +subdir('sh4')
> +subdir('sh4eb')

Here also I'd keep a single sh4/ directory (it is the same architecture
with an endianness variant we're going to remove).

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>