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

PTE bits updates #101

Open
atishp04 opened this issue Sep 16, 2022 · 1 comment
Open

PTE bits updates #101

atishp04 opened this issue Sep 16, 2022 · 1 comment
Labels
kvm Needed for the AP-TEE KVM port.

Comments

@atishp04
Copy link
Contributor

Currently, Salus doesn't support PTE bit modification. All the pages set as RWX. VMM should be able to pass that information for page add SBI functions. @atulkharerivos confirmed that this is on the TODO list. Just creating the issue to track it so that KVM implementation can be changed once it is supported in Salus.

@abrestic-rivos
Copy link
Collaborator

Aside from TVM migration (which will likely need its own set of APIs), what's the use case for wanting to set the G-stage PTE bits to anything other than RWX for confidential memory? If we were to allow it, I'd imagine we'd have to measure it as part of the TVM configuration.

For shared memory I guess it doesn't really matter, though IMO we should ban shared pages being mapped with X permissions.

@abrestic-rivos abrestic-rivos added the kvm Needed for the AP-TEE KVM port. label Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kvm Needed for the AP-TEE KVM port.
Projects
None yet
Development

No branches or pull requests

2 participants