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

attempt at listing some system rules with examples #401

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

tariqkurd-repo
Copy link
Collaborator

@tariqkurd-repo tariqkurd-repo commented Oct 2, 2024

Something which came up in RVI discussions is how to build CHERI systems, so this section adds some system guidelines.


. Update any data bytes without also writing the tag
.. This implies clearing the tag if a non-CHERI aware bus master overwrites a capability in memory
. Set the tag without also writing the data.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
. Set the tag without also writing the data.
. Read a tagged value with mismatched (stale or newer) data
. Set the tag without also writing the data.

This is implicit in the "accessed atomically" but might be worth calling out explicitly?

. Update any data bytes without also writing the tag
.. This implies clearing the tag if a non-CHERI aware bus master overwrites a capability in memory
. Set the tag without also writing the data.
.. Clearing tags in memory does not require updating the data
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe this works better as a note?

Suggested change
.. Clearing tags in memory does not require updating the data
NOTE: Clearing tags in memory does not necessarily require updating the associated data.

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.

2 participants