xilinx_qemu: CI fixes #919
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
--skip-meson is listed twice that's why remove one occurence. Signed-off-by: Michal Simek <[email protected]>
Update to 8.2.7 QEMU which is aligned with Xilinx v2025.1 tools. The main reason for this update is that this version is supporting Versal Gen 2 and associated models. Signed-off-by: Michal Simek <[email protected]>
As has been described by Stephanos in the commit message tests/kernel/context is broken when these two patches are not applied that's why get them back to have clean CI loop. Issue can be replicated by running west twister -v -p qemu_cortex_r5 -O /tmp/twister -T tests/kernel/context/ Signed-off-by: Michal Simek <[email protected]>
Model available in Xilinx v2025.1 version is missing CPU timers which can be used by Zephyr that's why wire them. And also enable EMACLITE network IP to have model for it available. Signed-off-by: Michal Simek <[email protected]>
Likely Qemu patch is correct and Zephyr tests/misc/llext-edk/ should be updated not to cause unaligned accesses. But for now it is better to update Qemu not to break CI to have time to update test. Signed-off-by: Michal Simek <[email protected]>
nashif
approved these changes
May 19, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@nashif reported me 3 issues which are reported by CI.
Two of them are related to patches which have been removed but they were the part of previous QEMU version.
Both of them are visible by running
west twister -v -p qemu_cortex_r5 -O /tmp/twister -T tests/kernel/context/
https://github.com/zephyrproject-rtos/zephyr-testing/actions/runs/15058691212/job/42329628698
https://github.com/zephyrproject-rtos/zephyr-testing/actions/runs/15058691212/job/42329629012
The last one is reproducible by running
west twister -v -p qemu_cortex_r5 -O /tmp/twister -T tests/misc/llext-edk/
https://github.com/zephyrproject-rtos/zephyr-testing/actions/runs/15058691212/job/42329629235
And it is tricky because it is coming from upstream QEMU commit which is likely correct but still would be better to revert this functionality which wasn't the part of QEMU v6.0.0 but it gets merged to v6.1 development tree.
The reason is to provide time to find out the reason why unaligned accesses are triggered by Qemu test.
In the meantime Xilinx also released new Qemu version based on 8.2.7 which was used for testing above issues that's why better to directly use this version instead of rely on older version.