[PATCH 48/48] tests/functional/test_aarch64_xlnx_versal: test the versal2 machine

Luc Michel posted 48 patches 4 months ago
Maintainers: Alistair Francis <alistair@alistair23.me>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Peter Maydell <peter.maydell@linaro.org>, Jason Wang <jasowang@redhat.com>
There is a newer version of this series
[PATCH 48/48] tests/functional/test_aarch64_xlnx_versal: test the versal2 machine
Posted by Luc Michel 4 months ago
Add a test for the amd-versal2-virt machine using the same command line,
kernel, initrd than the ones used for amd-versal-virt.

Signed-off-by: Luc Michel <luc.michel@amd.com>
---
 tests/functional/test_aarch64_xlnx_versal.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/tests/functional/test_aarch64_xlnx_versal.py b/tests/functional/test_aarch64_xlnx_versal.py
index 4b9c49e5d64..f32d904ef7e 100755
--- a/tests/functional/test_aarch64_xlnx_versal.py
+++ b/tests/functional/test_aarch64_xlnx_versal.py
@@ -18,12 +18,12 @@ class XlnxVersalVirtMachine(LinuxKernelTest):
         ('http://ports.ubuntu.com/ubuntu-ports/dists/bionic-updates/main/'
          'installer-arm64/20101020ubuntu543.19/images/netboot/'
          '/ubuntu-installer/arm64/initrd.gz'),
         'e7a5e716b6f516d8be315c06e7331aaf16994fe4222e0e7cfb34bc015698929e')
 
-    def test_aarch64_xlnx_versal_virt(self):
-        self.set_machine('xlnx-versal-virt')
+    def common_aarch64_amd_versal_virt(self, machine):
+        self.set_machine(machine)
         kernel_path = self.ASSET_KERNEL.fetch()
         initrd_path = self.ASSET_INITRD.fetch()
 
         self.vm.set_console()
         self.vm.add_args('-m', '2G',
@@ -31,7 +31,13 @@ def test_aarch64_xlnx_versal_virt(self):
                          '-kernel', kernel_path,
                          '-initrd', initrd_path)
         self.vm.launch()
         self.wait_for_console_pattern('Checked W+X mappings: passed')
 
+    def test_aarch64_amd_versal_virt(self):
+        self.common_aarch64_amd_versal_virt('amd-versal-virt')
+
+    def test_aarch64_amd_versal2_virt(self):
+        self.common_aarch64_amd_versal_virt('amd-versal2-virt')
+
 if __name__ == '__main__':
     LinuxKernelTest.main()
-- 
2.50.0
Re: [PATCH 48/48] tests/functional/test_aarch64_xlnx_versal: test the versal2 machine
Posted by Francisco Iglesias 3 months, 2 weeks ago
On Wed, Jul 16, 2025 at 11:54:30AM +0200, Luc Michel wrote:
> Add a test for the amd-versal2-virt machine using the same command line,
> kernel, initrd than the ones used for amd-versal-virt.
> 
> Signed-off-by: Luc Michel <luc.michel@amd.com>

Reviewed-by: Francisco Iglesias <francisco.iglesias@amd.com>

> ---
>  tests/functional/test_aarch64_xlnx_versal.py | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/functional/test_aarch64_xlnx_versal.py b/tests/functional/test_aarch64_xlnx_versal.py
> index 4b9c49e5d64..f32d904ef7e 100755
> --- a/tests/functional/test_aarch64_xlnx_versal.py
> +++ b/tests/functional/test_aarch64_xlnx_versal.py
> @@ -18,12 +18,12 @@ class XlnxVersalVirtMachine(LinuxKernelTest):
>          ('http://ports.ubuntu.com/ubuntu-ports/dists/bionic-updates/main/'
>           'installer-arm64/20101020ubuntu543.19/images/netboot/'
>           '/ubuntu-installer/arm64/initrd.gz'),
>          'e7a5e716b6f516d8be315c06e7331aaf16994fe4222e0e7cfb34bc015698929e')
>  
> -    def test_aarch64_xlnx_versal_virt(self):
> -        self.set_machine('xlnx-versal-virt')
> +    def common_aarch64_amd_versal_virt(self, machine):
> +        self.set_machine(machine)
>          kernel_path = self.ASSET_KERNEL.fetch()
>          initrd_path = self.ASSET_INITRD.fetch()
>  
>          self.vm.set_console()
>          self.vm.add_args('-m', '2G',
> @@ -31,7 +31,13 @@ def test_aarch64_xlnx_versal_virt(self):
>                           '-kernel', kernel_path,
>                           '-initrd', initrd_path)
>          self.vm.launch()
>          self.wait_for_console_pattern('Checked W+X mappings: passed')
>  
> +    def test_aarch64_amd_versal_virt(self):
> +        self.common_aarch64_amd_versal_virt('amd-versal-virt')
> +
> +    def test_aarch64_amd_versal2_virt(self):
> +        self.common_aarch64_amd_versal_virt('amd-versal2-virt')
> +
>  if __name__ == '__main__':
>      LinuxKernelTest.main()
> -- 
> 2.50.0
>