[PATCH v3 8/9] tests/function/aspeed: Add functional test for AST2700FC

Steven Lee via posted 9 patches 6 months, 2 weeks ago
There is a newer version of this series
[PATCH v3 8/9] tests/function/aspeed: Add functional test for AST2700FC
Posted by Steven Lee via 6 months, 2 weeks ago
Add functional test for AST2700-fc machine.

Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
Change-Id: Ieced249cf471515a33f8f5f5386a2f58d431f2f9
---
 tests/functional/test_aarch64_ast2700fc.py | 137 +++++++++++++++++++++
 1 file changed, 137 insertions(+)
 create mode 100755 tests/functional/test_aarch64_ast2700fc.py

diff --git a/tests/functional/test_aarch64_ast2700fc.py b/tests/functional/test_aarch64_ast2700fc.py
new file mode 100755
index 0000000000..f7b7907fde
--- /dev/null
+++ b/tests/functional/test_aarch64_ast2700fc.py
@@ -0,0 +1,137 @@
+#!/usr/bin/env python3
+#
+# Functional test that boots the AST2700 multi-SoCs with firmware
+#
+# Copyright (C) 2025 ASPEED Technology Inc
+#
+# SPDX-License-Identifier: GPL-2.0-or-later
+
+import os
+
+from qemu_test import QemuSystemTest, Asset
+from qemu_test import wait_for_console_pattern
+from qemu_test import exec_command_and_wait_for_pattern
+
+
+class AST2700fcMachineSDK(QemuSystemTest):
+    ASSET_SDK_V905_AST2700 = Asset(
+            'https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.05/ast2700-default-obmc.tar.gz',
+            'c1f4496aec06743c812a6e9a1a18d032f34d62f3ddb6956e924fef62aa2046a5')
+
+    def do_test_aarch64_ast2700fc_ca35_start(self, image):
+        self.require_netdev('user')
+        self.vm.set_console()
+        self.vm.add_args('-drive', 'file=' + image + ',if=mtd,format=raw',
+                         '-net', 'nic', '-net', 'user', '-snapshot')
+
+        self.vm.launch()
+
+        wait_for_console_pattern(self, 'U-Boot 2023.10')
+        wait_for_console_pattern(self, '## Loading kernel from FIT Image')
+        wait_for_console_pattern(self, 'Starting kernel ...')
+
+    def do_test_aarch64_ast2700fc_ssp_start(self):
+
+        self.vm.shutdown()
+        self.vm.set_console(console_index=1)
+        self.vm.launch()
+
+    def do_test_aarch64_ast2700fc_tsp_start(self):
+        self.vm.shutdown()
+        self.vm.set_console(console_index=2)
+        self.vm.launch()
+
+    def start_ast2700fc_test(self, name):
+        ca35_core = 4
+        uboot_size = os.path.getsize(self.scratch_file(name,
+                                                       'u-boot-nodtb.bin'))
+        uboot_dtb_load_addr = hex(0x400000000 + uboot_size)
+
+        load_images_list = [
+            {
+                'addr': '0x400000000',
+                'file': self.scratch_file(name,
+                                          'u-boot-nodtb.bin')
+            },
+            {
+                'addr': str(uboot_dtb_load_addr),
+                'file': self.scratch_file(name, 'u-boot.dtb')
+            },
+            {
+                'addr': '0x430000000',
+                'file': self.scratch_file(name, 'bl31.bin')
+            },
+            {
+                'addr': '0x430080000',
+                'file': self.scratch_file(name, 'optee',
+                                          'tee-raw.bin')
+            }
+        ]
+
+        for load_image in load_images_list:
+            addr = load_image['addr']
+            file = load_image['file']
+            self.vm.add_args('-device',
+                             f'loader,force-raw=on,addr={addr},file={file}')
+
+        for i in range(ca35_core):
+            self.vm.add_args('-device',
+                             f'loader,addr=0x430000000,cpu-num={i}')
+
+        load_elf_list = {
+            'ssp': self.scratch_file(name, 'ast2700-ssp.elf'),
+            'tsp': self.scratch_file(name, 'ast2700-tsp.elf')
+        }
+
+        for cpu_num, key in enumerate(load_elf_list, start=4):
+            file = load_elf_list[key]
+            self.vm.add_args('-device',
+                             f'loader,file={file},cpu-num={cpu_num}')
+
+        self.vm.add_args('-device',
+                         'tmp105,bus=aspeed.i2c.bus.1,address=0x4d,id=tmp-test')
+        self.do_test_aarch64_ast2700fc_ca35_start(
+            self.scratch_file(name, 'image-bmc'))
+
+        wait_for_console_pattern(self, f'{name} login:')
+
+        exec_command_and_wait_for_pattern(self, 'root', 'Password:')
+        exec_command_and_wait_for_pattern(self, '0penBmc', f'root@{name}:~#')
+
+        exec_command_and_wait_for_pattern(
+                self,
+                'echo lm75 0x4d > /sys/class/i2c-dev/i2c-1/device/new_device ',
+                'i2c i2c-1: new_device: Instantiated device lm75 at 0x4d')
+        exec_command_and_wait_for_pattern(
+                self,
+                'cat /sys/class/hwmon/hwmon*/temp1_input', '0')
+        self.vm.cmd('qom-set', path='/machine/peripheral/tmp-test',
+                    property='temperature', value=18000)
+        exec_command_and_wait_for_pattern(
+                self,
+                'cat /sys/class/hwmon/hwmon*/temp1_input', '18000')
+
+        self.do_test_aarch64_ast2700fc_ssp_start()
+
+        exec_command_and_wait_for_pattern(self, '\012', 'ssp:~$')
+        exec_command_and_wait_for_pattern(self, 'version',
+                                          'Zephyr version 3.7.1')
+        exec_command_and_wait_for_pattern(self, 'md 72c02000 1',
+                                          '[72c02000] 06010103')
+
+        self.do_test_aarch64_ast2700fc_tsp_start()
+
+        exec_command_and_wait_for_pattern(self, '\012', 'tsp:~$')
+        exec_command_and_wait_for_pattern(self, 'version',
+                                          'Zephyr version 3.7.1')
+        exec_command_and_wait_for_pattern(self, 'md 72c02000 1',
+                                          '[72c02000] 06010103')
+
+    def test_aarch64_ast2700fc_sdk_v09_05(self):
+        self.set_machine('ast2700fc')
+        self.archive_extract(self.ASSET_SDK_V905_AST2700)
+        self.start_ast2700fc_test('ast2700-default')
+
+
+if __name__ == '__main__':
+    QemuSystemTest.main()
-- 
2.34.1
Re: [PATCH v3 8/9] tests/function/aspeed: Add functional test for AST2700FC
Posted by Cédric Le Goater 6 months, 2 weeks ago
On 4/29/25 11:18, Steven Lee wrote:
> Add functional test for AST2700-fc machine.
> 
> Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
> Change-Id: Ieced249cf471515a33f8f5f5386a2f58d431f2f9
> --->   tests/functional/test_aarch64_ast2700fc.py | 137 +++++++++++++++++++++
>   1 file changed, 137 insertions(+)
>   create mode 100755 tests/functional/test_aarch64_ast2700fc.py
> 
> diff --git a/tests/functional/test_aarch64_ast2700fc.py b/tests/functional/test_aarch64_ast2700fc.py
> new file mode 100755
> index 0000000000..f7b7907fde
> --- /dev/null
> +++ b/tests/functional/test_aarch64_ast2700fc.py
> @@ -0,0 +1,137 @@
> +#!/usr/bin/env python3
> +#
> +# Functional test that boots the AST2700 multi-SoCs with firmware
> +#
> +# Copyright (C) 2025 ASPEED Technology Inc
> +#
> +# SPDX-License-Identifier: GPL-2.0-or-later
> +
> +import os
> +
> +from qemu_test import QemuSystemTest, Asset
> +from qemu_test import wait_for_console_pattern
> +from qemu_test import exec_command_and_wait_for_pattern
> +
> +
> +class AST2700fcMachineSDK(QemuSystemTest):
> +    ASSET_SDK_V905_AST2700 = Asset(
> +            'https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.05/ast2700-default-obmc.tar.gz',
> +            'c1f4496aec06743c812a6e9a1a18d032f34d62f3ddb6956e924fef62aa2046a5')
> +
> +    def do_test_aarch64_ast2700fc_ca35_start(self, image):
> +        self.require_netdev('user')
> +        self.vm.set_console()
> +        self.vm.add_args('-drive', 'file=' + image + ',if=mtd,format=raw',
> +                         '-net', 'nic', '-net', 'user', '-snapshot')
> +
> +        self.vm.launch()
> +
> +        wait_for_console_pattern(self, 'U-Boot 2023.10')
> +        wait_for_console_pattern(self, '## Loading kernel from FIT Image')
> +        wait_for_console_pattern(self, 'Starting kernel ...')
> +
> +    def do_test_aarch64_ast2700fc_ssp_start(self):
> +
> +        self.vm.shutdown()
> +        self.vm.set_console(console_index=1)
> +        self.vm.launch()
> +
> +    def do_test_aarch64_ast2700fc_tsp_start(self):
> +        self.vm.shutdown()
> +        self.vm.set_console(console_index=2)
> +        self.vm.launch()
> +
> +    def start_ast2700fc_test(self, name):
> +        ca35_core = 4
> +        uboot_size = os.path.getsize(self.scratch_file(name,
> +                                                       'u-boot-nodtb.bin'))
> +        uboot_dtb_load_addr = hex(0x400000000 + uboot_size)
> +
> +        load_images_list = [
> +            {
> +                'addr': '0x400000000',
> +                'file': self.scratch_file(name,
> +                                          'u-boot-nodtb.bin')
> +            },
> +            {
> +                'addr': str(uboot_dtb_load_addr),
> +                'file': self.scratch_file(name, 'u-boot.dtb')
> +            },
> +            {
> +                'addr': '0x430000000',
> +                'file': self.scratch_file(name, 'bl31.bin')
> +            },
> +            {
> +                'addr': '0x430080000',
> +                'file': self.scratch_file(name, 'optee',
> +                                          'tee-raw.bin')
> +            }
> +        ]


Could this test be simplified with the recent addition of vbootrom support ?

If so, you can use this branch :

   https://github.com/legoater/qemu/commits/aspeed-next/

Anyhow,

Reviewed-by: Cédric Le Goater <clg@redhat.com>

Thanks,

C.


> +        for load_image in load_images_list:
> +            addr = load_image['addr']
> +            file = load_image['file']
> +            self.vm.add_args('-device',
> +                             f'loader,force-raw=on,addr={addr},file={file}')
> +
> +        for i in range(ca35_core):
> +            self.vm.add_args('-device',
> +                             f'loader,addr=0x430000000,cpu-num={i}')
> +
> +        load_elf_list = {
> +            'ssp': self.scratch_file(name, 'ast2700-ssp.elf'),
> +            'tsp': self.scratch_file(name, 'ast2700-tsp.elf')
> +        }
> +
> +        for cpu_num, key in enumerate(load_elf_list, start=4):
> +            file = load_elf_list[key]
> +            self.vm.add_args('-device',
> +                             f'loader,file={file},cpu-num={cpu_num}')
> +
> +        self.vm.add_args('-device',
> +                         'tmp105,bus=aspeed.i2c.bus.1,address=0x4d,id=tmp-test')
> +        self.do_test_aarch64_ast2700fc_ca35_start(
> +            self.scratch_file(name, 'image-bmc'))
> +
> +        wait_for_console_pattern(self, f'{name} login:')
> +
> +        exec_command_and_wait_for_pattern(self, 'root', 'Password:')
> +        exec_command_and_wait_for_pattern(self, '0penBmc', f'root@{name}:~#')
> +
> +        exec_command_and_wait_for_pattern(
> +                self,
> +                'echo lm75 0x4d > /sys/class/i2c-dev/i2c-1/device/new_device ',
> +                'i2c i2c-1: new_device: Instantiated device lm75 at 0x4d')
> +        exec_command_and_wait_for_pattern(
> +                self,
> +                'cat /sys/class/hwmon/hwmon*/temp1_input', '0')
> +        self.vm.cmd('qom-set', path='/machine/peripheral/tmp-test',
> +                    property='temperature', value=18000)
> +        exec_command_and_wait_for_pattern(
> +                self,
> +                'cat /sys/class/hwmon/hwmon*/temp1_input', '18000')
> +
> +        self.do_test_aarch64_ast2700fc_ssp_start()
> +
> +        exec_command_and_wait_for_pattern(self, '\012', 'ssp:~$')
> +        exec_command_and_wait_for_pattern(self, 'version',
> +                                          'Zephyr version 3.7.1')
> +        exec_command_and_wait_for_pattern(self, 'md 72c02000 1',
> +                                          '[72c02000] 06010103')
> +
> +        self.do_test_aarch64_ast2700fc_tsp_start()
> +
> +        exec_command_and_wait_for_pattern(self, '\012', 'tsp:~$')
> +        exec_command_and_wait_for_pattern(self, 'version',
> +                                          'Zephyr version 3.7.1')
> +        exec_command_and_wait_for_pattern(self, 'md 72c02000 1',
> +                                          '[72c02000] 06010103')
> +
> +    def test_aarch64_ast2700fc_sdk_v09_05(self):
> +        self.set_machine('ast2700fc')
> +        self.archive_extract(self.ASSET_SDK_V905_AST2700)
> +        self.start_ast2700fc_test('ast2700-default')
> +
> +
> +if __name__ == '__main__':
> +    QemuSystemTest.main()


RE: [PATCH v3 8/9] tests/function/aspeed: Add functional test for AST2700FC
Posted by Steven Lee 6 months, 2 weeks ago
Hi Cédric,

> -----Original Message-----
> From: Cédric Le Goater <clg@kaod.org>
> Sent: Wednesday, April 30, 2025 5:35 PM
> To: Steven Lee <steven_lee@aspeedtech.com>; Peter Maydell
> <peter.maydell@linaro.org>; Troy Lee <leetroy@gmail.com>; Jamin Lin
> <jamin_lin@aspeedtech.com>; Andrew Jeffery
> <andrew@codeconstruct.com.au>; Joel Stanley <joel@jms.id.au>; open
> list:ASPEED BMCs <qemu-arm@nongnu.org>; open list:All patches CC here
> <qemu-devel@nongnu.org>
> Cc: Troy Lee <troy_lee@aspeedtech.com>; longzl2@lenovo.com; Yunlin Tang
> <yunlin.tang@aspeedtech.com>
> Subject: Re: [PATCH v3 8/9] tests/function/aspeed: Add functional test for
> AST2700FC
> 
> On 4/29/25 11:18, Steven Lee wrote:
> > Add functional test for AST2700-fc machine.
> >
> > Signed-off-by: Steven Lee <steven_lee@aspeedtech.com>
> > Change-Id: Ieced249cf471515a33f8f5f5386a2f58d431f2f9
> > --->   tests/functional/test_aarch64_ast2700fc.py | 137
> > ---> +++++++++++++++++++++
> >   1 file changed, 137 insertions(+)
> >   create mode 100755 tests/functional/test_aarch64_ast2700fc.py
> >
> > diff --git a/tests/functional/test_aarch64_ast2700fc.py
> > b/tests/functional/test_aarch64_ast2700fc.py
> > new file mode 100755
> > index 0000000000..f7b7907fde
> > --- /dev/null
> > +++ b/tests/functional/test_aarch64_ast2700fc.py
> > @@ -0,0 +1,137 @@
> > +#!/usr/bin/env python3
> > +#
> > +# Functional test that boots the AST2700 multi-SoCs with firmware # #
> > +Copyright (C) 2025 ASPEED Technology Inc # # SPDX-License-Identifier:
> > +GPL-2.0-or-later
> > +
> > +import os
> > +
> > +from qemu_test import QemuSystemTest, Asset from qemu_test import
> > +wait_for_console_pattern from qemu_test import
> > +exec_command_and_wait_for_pattern
> > +
> > +
> > +class AST2700fcMachineSDK(QemuSystemTest):
> > +    ASSET_SDK_V905_AST2700 = Asset(
> > +
> 'https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.05/ast
> 2700-default-obmc.tar.gz',
> > +
> >
> +'c1f4496aec06743c812a6e9a1a18d032f34d62f3ddb6956e924fef62aa2046a5'
> )
> > +
> > +    def do_test_aarch64_ast2700fc_ca35_start(self, image):
> > +        self.require_netdev('user')
> > +        self.vm.set_console()
> > +        self.vm.add_args('-drive', 'file=' + image + ',if=mtd,format=raw',
> > +                         '-net', 'nic', '-net', 'user', '-snapshot')
> > +
> > +        self.vm.launch()
> > +
> > +        wait_for_console_pattern(self, 'U-Boot 2023.10')
> > +        wait_for_console_pattern(self, '## Loading kernel from FIT
> Image')
> > +        wait_for_console_pattern(self, 'Starting kernel ...')
> > +
> > +    def do_test_aarch64_ast2700fc_ssp_start(self):
> > +
> > +        self.vm.shutdown()
> > +        self.vm.set_console(console_index=1)
> > +        self.vm.launch()
> > +
> > +    def do_test_aarch64_ast2700fc_tsp_start(self):
> > +        self.vm.shutdown()
> > +        self.vm.set_console(console_index=2)
> > +        self.vm.launch()
> > +
> > +    def start_ast2700fc_test(self, name):
> > +        ca35_core = 4
> > +        uboot_size = os.path.getsize(self.scratch_file(name,
> > +
> 'u-boot-nodtb.bin'))
> > +        uboot_dtb_load_addr = hex(0x400000000 + uboot_size)
> > +
> > +        load_images_list = [
> > +            {
> > +                'addr': '0x400000000',
> > +                'file': self.scratch_file(name,
> > +                                          'u-boot-nodtb.bin')
> > +            },
> > +            {
> > +                'addr': str(uboot_dtb_load_addr),
> > +                'file': self.scratch_file(name, 'u-boot.dtb')
> > +            },
> > +            {
> > +                'addr': '0x430000000',
> > +                'file': self.scratch_file(name, 'bl31.bin')
> > +            },
> > +            {
> > +                'addr': '0x430080000',
> > +                'file': self.scratch_file(name, 'optee',
> > +                                          'tee-raw.bin')
> > +            }
> > +        ]
> 
> 
> Could this test be simplified with the recent addition of vbootrom support ?
> 
> If so, you can use this branch :
> 
>    https://github.com/legoater/qemu/commits/aspeed-next/
> 
> Anyhow,
> 

Yes, I will simplify the test in the v4 patch series.

Regards,
Steven