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

Update capability tag reading via ptrace() for async revocation #2076

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

rwatson
Copy link
Member

@rwatson rwatson commented Mar 25, 2024

I got a panic: cheri revoke does not support foreign maps (yet) running GDB on chericat. This fix merges the tweak from ebc253e into proc_read_cheri_tags_page(), which seems to resolve the issue here. But definitely wants review by @markjdb. When ready, this change will want to go into the demo branch also.

@rwatson rwatson requested a review from markjdb March 25, 2024 20:41
@jrtc27 jrtc27 changed the base branch from main to dev March 25, 2024 20:42
Copy link
Contributor

@markjdb markjdb left a comment

Choose a reason for hiding this comment

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

Yes this is fine. Sorry for the speedbump, I'm not sure how I missed this when looking at the original panic. I thought I had audited all of the vm_fault() callers in the tree.. :(

@bsdjhb bsdjhb merged commit 454b567 into dev Mar 26, 2024
27 of 28 checks passed
@bsdjhb bsdjhb deleted the cap_tag_read_revoke_async_fix branch March 26, 2024 03:23
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.

3 participants