Skip to content

boards: arm: mps3: fix intermittent ci failures #90685

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 28, 2025

Conversation

wearyzen
Copy link
Collaborator

@wearyzen wearyzen commented May 27, 2025

FVP's are functionally accurate but not cycle accurate which is
different from QEMU which prefers speed over accuracy.
This sometimes leads to intermittent test failures with FVP which
would otherwise pass with QEMU.
The intermittent failed tests are successful on retries however,
adding a timeout_multiplier would reduce the chance of failures so
we set the timeout_multiplier to a value which will avoid the issue.
Fixes #90654

@github-actions github-actions bot added the platform: ARM Arm Limited label May 27, 2025
@github-actions github-actions bot requested a review from ithinuel May 27, 2025 16:48
@wearyzen wearyzen added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label May 28, 2025
Copy link
Member

@fabiobaltieri fabiobaltieri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a comment as to why this needs to be changed for this board? I imagine it takes some time to boot or something like that?

FVP's are functionally accurate but not cycle accurate which is
different from QEMU which prefers speed over accuracy.
This sometimes leads to intermittent test failures with FVP which
would otherwise pass with QEMU.
The intermittent failed tests are successful on retries however,
adding a timeout_multiplier would reduce the chance of failures so
we set the timeout_multiplier to a value which will avoid the issue.

Fixes zephyrproject-rtos#90654

Signed-off-by: Sudan Landge <sudan.landge@arm.com>
@wearyzen wearyzen force-pushed the mps3_test_timeout branch from adaab14 to ecaedda Compare May 28, 2025 13:11
Copy link

@wearyzen wearyzen requested a review from fabiobaltieri May 28, 2025 13:32
@kartben kartben merged commit 6d038d3 into zephyrproject-rtos:main May 28, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: ARM Arm Limited Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mps3: kernel.memory_protection.syscalls.kyield: FAILED Timeout
4 participants