Skip to content
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

[patch-axel-87a] support qemu-riscv-virt platform test #12

Closed
wants to merge 13 commits into from

Conversation

axel-h
Copy link
Member

@axel-h axel-h commented Mar 29, 2022

Test with: seL4/seL4_tools#135

@axel-h axel-h force-pushed the patch-axel-87a branch 10 times, most recently from 65d7e5f to 18fc171 Compare April 3, 2022 22:57
@axel-h axel-h force-pushed the patch-axel-87a branch 2 times, most recently from 5d1c85a to 33a710c Compare April 11, 2022 19:25
@axel-h axel-h force-pushed the patch-axel-87a branch 12 times, most recently from cf0c7f2 to ea0fa5d Compare April 25, 2022 19:37
@axel-h axel-h force-pushed the patch-axel-87a branch from 9d277bb to 1d90bce Compare May 2, 2022 14:46
elmankku and others added 3 commits June 3, 2022 08:46
Sometimes device trees may have differences between vanilla Linux
and SoC vendor, and the changes can be tedious to override with a
device tree overlay.

This change allows overriding the platform default dts and overlays
with a custom dts.

Signed-off-by: Markku Ahvenjärvi <markkux@ssrc.tii.ae>
Generated from maaxboard-dcss-hdmi.dts (Avnet/linux-imx) for more
comprehensive device support. Updated overlay to align.
Signed-off-by: Mark Jenkinson <mark.jenkinson@capgemini.com>
There are cases that reasonably require non-ASCII characters in the
source code, hence tools/bitfield_gen.py must be able to handle them
correctly. This commit makes sure the bitfield generator consistently
opens all files with UTF-8 encoding.

Co-authored-by: Gerwin Klein <gerwin.klein@proofcraft.systems>
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Xaphiosis and others added 10 commits June 6, 2022 10:10
This commit introduces `addressTranslateS1` to be used on Arm platforms
with hypervisor enabled for stage 1 (vaddr to IPA) translation. On
AArch32 this is a rename from `addressTranslateS1CPR`, and on AArch64 it
wraps `ats1e1r`. This changes the ABI on AArch64 to report faulting
address as IPA.

Reasoning:

With hypervisor enabled, AArch64 defined `addressTranslateS1CPR` to do
nothing, while AArch32 defined it to do stage 1 translation. This
delivered VM faults to the user with the faulting address being either
an IPA or a vaddr depending on mode. This inconsistency is undesireable.

This commit proposes adjusting the inconsistency to match AArch32
behaviour, as it is one of the verified platforms.

Signed-off-by: Rafal Kolanski <rafal.kolanski@proofcraft.systems>
For a VM fault in a hypervisor context, 32-bit Arm translated the IP
address into an IPA, while 64-bit Arm did not. The previous commit
made these consistent by performing the translation on both.

After investigation and discussion, the 32-bit Arm behaviour was
declared a bug: reporting an IPA (instead of a VA) to the VMM is not
very useful and can cause issues when the fault message is not sent
immediately (SELFOUR-1602). This commit, therefore, removes all stage 1
translation from Arch_setMRs_fault on Arm platforms.

Signed-off-by: Rafal Kolanski <rafal.kolanski@proofcraft.systems>
Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
Also reject any unknown CPU.

Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
- makes setting more generic an allow overriding parameters
- save QEMU parameters in DTS as comment

Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
Signed-off-by: Axel Heider <axel.heider@hensoldt.net>
@axel-h axel-h closed this Jun 7, 2022
@axel-h axel-h deleted the patch-axel-87a branch June 7, 2022 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants