tests/functional: add boot error detection for RME tests

[PATCH] tests/functional: add boot error detection for RME tests

Pierrick Bouvier posted 1 patch 11 months, 1 week ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250303185745.2504842-1-pierrick.bouvier@linaro.org

Maintainers: Radoslaw Biernacki <rad@semihalf.com>, Peter Maydell <peter.maydell@linaro.org>, Leif Lindholm <leif.lindholm@oss.qualcomm.com>

tests/functional/test_aarch64_rme_sbsaref.py | 3 ++-
tests/functional/test_aarch64_rme_virt.py    | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)

Expand all Fold all

[PATCH] tests/functional: add boot error detection for RME tests

Posted by Pierrick Bouvier 11 months, 1 week ago

It was identified that those tests randomly fail with a synchronous
exception at boot (reported by EDK2).
While we solve this problem, report failure immediately so tests don't
timeout in CI.

Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
---
 tests/functional/test_aarch64_rme_sbsaref.py | 3 ++-
 tests/functional/test_aarch64_rme_virt.py    | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/tests/functional/test_aarch64_rme_sbsaref.py b/tests/functional/test_aarch64_rme_sbsaref.py
index 93bb5283386..ddcc9493a6c 100755
--- a/tests/functional/test_aarch64_rme_sbsaref.py
+++ b/tests/functional/test_aarch64_rme_sbsaref.py
@@ -60,7 +60,8 @@ def test_aarch64_rme_sbsaref(self):
 
         self.vm.launch()
         # Wait for host VM boot to complete.
-        wait_for_console_pattern(self, 'Welcome to Buildroot')
+        wait_for_console_pattern(self, 'Welcome to Buildroot',
+                                 failure_message='Synchronous Exception at')
         exec_command_and_wait_for_pattern(self, 'root', '#')
 
         test_realms_guest(self)
diff --git a/tests/functional/test_aarch64_rme_virt.py b/tests/functional/test_aarch64_rme_virt.py
index 42b9229b4cb..38e01721a4e 100755
--- a/tests/functional/test_aarch64_rme_virt.py
+++ b/tests/functional/test_aarch64_rme_virt.py
@@ -89,7 +89,8 @@ def test_aarch64_rme_virt(self):
 
         self.vm.launch()
         # Wait for host VM boot to complete.
-        wait_for_console_pattern(self, 'Welcome to Buildroot')
+        wait_for_console_pattern(self, 'Welcome to Buildroot',
+                                 failure_message='Synchronous Exception at')
         exec_command_and_wait_for_pattern(self, 'root', '#')
 
         test_realms_guest(self)
-- 
2.39.5

Re: [PATCH] tests/functional: add boot error detection for RME tests

Posted by Alex Bennée 11 months, 1 week ago

Pierrick Bouvier <pierrick.bouvier@linaro.org> writes:

> It was identified that those tests randomly fail with a synchronous
> exception at boot (reported by EDK2).
> While we solve this problem, report failure immediately so tests don't
> timeout in CI.
>
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>

Queued to maintainer/for-10.0-softfreeze, thanks.

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro

Re: [PATCH] tests/functional: add boot error detection for RME tests

Posted by Philippe Mathieu-Daudé 11 months, 1 week ago

On 3/3/25 19:57, Pierrick Bouvier wrote:
> It was identified that those tests randomly fail with a synchronous
> exception at boot (reported by EDK2).
> While we solve this problem, report failure immediately so tests don't
> timeout in CI.
> 
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> ---
>   tests/functional/test_aarch64_rme_sbsaref.py | 3 ++-
>   tests/functional/test_aarch64_rme_virt.py    | 3 ++-
>   2 files changed, 4 insertions(+), 2 deletions(-)

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