[PATCH v1] tests/avocado: Add LoongArch machine start test

[PATCH v1] tests/avocado: Add LoongArch machine start test

[Song Gao]

Add a new test in tests/avocado to check LoongArch virt machine start.

Signed-off-by: Song Gao <gaosong@loongson.cn>
---
 MAINTAINERS                              |  1 +
 tests/avocado/machine_loongarch_start.py | 68 ++++++++++++++++++++++++
 2 files changed, 69 insertions(+)
 create mode 100644 tests/avocado/machine_loongarch_start.py

diff --git a/MAINTAINERS b/MAINTAINERS
index f757369373..d3b11ac056 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -245,6 +245,7 @@ M: Xiaojuan Yang <yangxiaojuan@loongson.cn>
 S: Maintained
 F: target/loongarch/
 F: tests/tcg/loongarch64/
+F: tests/avocado/machine_loongarch_start.py
 
 M68K TCG CPUs
 M: Laurent Vivier <laurent@vivier.eu>
diff --git a/tests/avocado/machine_loongarch_start.py b/tests/avocado/machine_loongarch_start.py
new file mode 100644
index 0000000000..fa5dc5c49d
--- /dev/null
+++ b/tests/avocado/machine_loongarch_start.py
@@ -0,0 +1,68 @@
+# SPDX-License-Identifier: GPL-2.0-or-later
+#
+# LoongArch virt start test.
+#
+# Copyright (c) 2023 Loongson Technology Corporation Limited
+#
+
+import os
+import tempfile
+
+from avocado_qemu import QemuSystemTest
+from avocado_qemu import exec_command_and_wait_for_pattern
+from avocado_qemu import wait_for_console_pattern
+
+class LoongArchMachine(QemuSystemTest):
+    KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 '
+
+    timeout = 120
+
+    def wait_for_console_pattern(self, success_message, vm=None):
+        wait_for_console_pattern(self, success_message,
+                                 failure_message='Kernel panic - not syncing',
+                                 vm=vm)
+
+    dmesg_clear_count = 1
+    def clear_guest_dmesg(self):
+        exec_command_and_wait_for_pattern(self, 'dmesg -c > /dev/null; '
+                    'echo dm_clear\ ' + str(self.dmesg_clear_count),
+                    'dm_clear ' + str(self.dmesg_clear_count))
+        self.dmesg_clear_count += 1
+
+    def test_loongarch64_devices(self):
+
+        """
+        :avocado: tags=arch:loongarch64
+        :avocado: tags=machine:virt
+        """
+
+        kernel_url = ('https://github.com/yangxiaojuan-loongson/qemu-binary/'
+                      'releases/download/binary-files/vmlinuz.efi')
+        kernel_hash = '951b485b16e3788b6db03a3e1793c067009e31a2'
+        kernel_path = self.fetch_asset(kernel_url, asset_hash=kernel_hash)
+
+        initrd_url = ('https://github.com/yangxiaojuan-loongson/qemu-binary/'
+                      'releases/download/binary-files/ramdisk')
+        initrd_hash = 'c67658d9b2a447ce7db2f73ba3d373c9b2b90ab2'
+        initrd_path = self.fetch_asset(initrd_url, asset_hash=initrd_hash)
+
+        bios_url = ('https://github.com/yangxiaojuan-loongson/qemu-binary/'
+                    'releases/download/binary-files/QEMU_EFI.fd')
+        bios_hash = ('dfc1bfba4853cd763b9d392d0031827e8addbca8')
+        bios_path = self.fetch_asset(bios_url, asset_hash=bios_hash)
+
+        self.vm.set_console()
+        kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE +
+                               'root=/dev/ram rdinit=/sbin/init console=ttyS0,115200')
+        self.vm.add_args('-nographic',
+                         '-smp', '4',
+                         '-m', '1024',
+                         '-cpu', 'la464',
+                         '-kernel', kernel_path,
+                         '-initrd', initrd_path,
+                         '-bios', bios_path,
+                         '-append', kernel_command_line)
+        self.vm.launch()
+        self.wait_for_console_pattern('Run /sbin/init as init process')
+        exec_command_and_wait_for_pattern(self, 'cat /proc/cpuinfo',
+                                          'processor		: 3')
-- 
2.31.1

Re: [PATCH v1] tests/avocado: Add LoongArch machine start test

[Thomas Huth]

On 11/05/2023 14.21, Song Gao wrote:
> Add a new test in tests/avocado to check LoongArch virt machine start.
> 
> Signed-off-by: Song Gao <gaosong@loongson.cn>
> ---
>   MAINTAINERS                              |  1 +
>   tests/avocado/machine_loongarch_start.py | 68 ++++++++++++++++++++++++
>   2 files changed, 69 insertions(+)
>   create mode 100644 tests/avocado/machine_loongarch_start.py
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f757369373..d3b11ac056 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -245,6 +245,7 @@ M: Xiaojuan Yang <yangxiaojuan@loongson.cn>
>   S: Maintained
>   F: target/loongarch/
>   F: tests/tcg/loongarch64/
> +F: tests/avocado/machine_loongarch_start.py

I'd maybe just name it machine_loongarch.py, without the "start", in case 
you add some more tests later.

Anyway, thanks for tackling this!

Reviewed-by: Thomas Huth <thuth@redhat.com>