[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 09/18] tests/acceptance: Re-enable the microblaze test
From: |
Philippe Mathieu-Daudé |
Subject: |
[PULL 09/18] tests/acceptance: Re-enable the microblaze test |
Date: |
Mon, 8 Feb 2021 14:37:02 +0100 |
From: Thomas Huth <thuth@redhat.com>
The microblaze kernel sometimes gets stuck during boot (ca. 1 out of 200
times), so we disabled the corresponding acceptance tests some months
ago. However, it's likely better to check that the kernel is still
starting than to not testing it at all anymore. Move the test to
a separate file, enable it again and check for an earlier console
message that should always appear.
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
Message-Id: <20210128152815.585478-1-thuth@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
MAINTAINERS | 1 +
tests/acceptance/boot_linux_console.py | 9 -------
tests/acceptance/machine_microblaze.py | 35 ++++++++++++++++++++++++++
3 files changed, 36 insertions(+), 9 deletions(-)
create mode 100644 tests/acceptance/machine_microblaze.py
diff --git a/MAINTAINERS b/MAINTAINERS
index 9543d7c44a2..99621abf8d7 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1119,6 +1119,7 @@ M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
S: Maintained
F: hw/microblaze/petalogix_s3adsp1800_mmu.c
F: include/hw/char/xilinx_uartlite.h
+F: tests/acceptance/machine_microblaze.py
petalogix_ml605
M: Edgar E. Iglesias <edgar.iglesias@gmail.com>
diff --git a/tests/acceptance/boot_linux_console.py
b/tests/acceptance/boot_linux_console.py
index c34075d5378..eb012867997 100644
--- a/tests/acceptance/boot_linux_console.py
+++ b/tests/acceptance/boot_linux_console.py
@@ -1056,15 +1056,6 @@ def test_m68k_mcf5208evb(self):
tar_hash = 'ac688fd00561a2b6ce1359f9ff6aa2b98c9a570c'
self.do_test_advcal_2018('07', tar_hash, 'sanity-clause.elf')
- @skip("Test currently broken") # Console stuck as of 5.2-rc1
- def test_microblaze_s3adsp1800(self):
- """
- :avocado: tags=arch:microblaze
- :avocado: tags=machine:petalogix-s3adsp1800
- """
- tar_hash = '08bf3e3bfb6b6c7ce1e54ab65d54e189f2caf13f'
- self.do_test_advcal_2018('17', tar_hash, 'ballerina.bin')
-
def test_or1k_sim(self):
"""
:avocado: tags=arch:or1k
diff --git a/tests/acceptance/machine_microblaze.py
b/tests/acceptance/machine_microblaze.py
new file mode 100644
index 00000000000..7f6d18495d8
--- /dev/null
+++ b/tests/acceptance/machine_microblaze.py
@@ -0,0 +1,35 @@
+# Functional test that boots a microblaze Linux kernel and checks the console
+#
+# Copyright (c) 2018, 2021 Red Hat, Inc.
+#
+# This work is licensed under the terms of the GNU GPL, version 2 or
+# later. See the COPYING file in the top-level directory.
+
+from avocado_qemu import Test
+from avocado_qemu import wait_for_console_pattern
+from avocado.utils import archive
+
+class MicroblazeMachine(Test):
+
+ timeout = 90
+
+ def test_microblaze_s3adsp1800(self):
+ """
+ :avocado: tags=arch:microblaze
+ :avocado: tags=machine:petalogix-s3adsp1800
+ """
+
+ tar_url = ('https://www.qemu-advent-calendar.org'
+ '/2018/download/day17.tar.xz')
+ tar_hash = '08bf3e3bfb6b6c7ce1e54ab65d54e189f2caf13f'
+ file_path = self.fetch_asset(tar_url, asset_hash=tar_hash)
+ archive.extract(file_path, self.workdir)
+ self.vm.set_console()
+ self.vm.add_args('-kernel', self.workdir + '/day17/ballerina.bin')
+ self.vm.launch()
+ wait_for_console_pattern(self, 'This architecture does not have '
+ 'kernel memory protection')
+ # Note:
+ # The kernel sometimes gets stuck after the "This architecture ..."
+ # message, that's why we don't test for a later string here. This
+ # needs some investigation by a microblaze wizard one day...
--
2.26.2
- [PULL 01/18] tests/acceptance: Extract tesseract_available() helper in new namespace, (continued)
- [PULL 01/18] tests/acceptance: Extract tesseract_available() helper in new namespace, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 03/18] tests/acceptance: Extract do_test_arm_orangepi_armbian_uboot() method, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 07/18] tests/acceptance: Add a test for the virtex-ml507 ppc machine, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 05/18] tests/acceptance: Move the pseries test to a separate file, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 06/18] tests/acceptance: Test the mpc8544ds machine, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 11/18] tests/acceptance/boot_linux: rename misleading cloudinit method, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 12/18] tests/acceptance/virtiofs_submounts: use workdir property, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 14/18] tests/acceptance/virtiofs_submounts: use a virtio-net device instead, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 15/18] tests/acceptance/virtiofs_submounts: standardize port as integer, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 04/18] tests/acceptance: Test U-Boot/Linux from Armbian 20.08 on Orange Pi PC, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 09/18] tests/acceptance: Re-enable the microblaze test,
Philippe Mathieu-Daudé <=
- [PULL 13/18] tests/acceptance/virtiofs_submounts: do not ask for ssh key password, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 16/18] tests/acceptance/virtiofs_submounts: required space between IP and port, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 10/18] tests/acceptance/boot_linux: fix typo on cloudinit error message, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 18/18] Acceptance Tests: remove unnecessary tag from documentation example, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 08/18] tests/acceptance: Increase the timeout in the replay tests, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 17/18] Acceptance tests: clarify ssh connection failure reason, Philippe Mathieu-Daudé, 2021/02/08
- Re: [PULL 00/18] Integration testing patches for 2021-02-08, Peter Maydell, 2021/02/08