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

Add linear privilege level measurement value type #354

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

Conversation

deeglaze
Copy link
Collaborator

As a means of representing a CPL or VMPL, this provides some extra semantic understanding. Leaving condition-matching to a later PR.

@yogeshbdeshpande
Copy link
Collaborator

@deeglaze Instead of directly making a change request, can we please have an issue, which describes a requirement.

What is a Privilege level of an Environment ?

Why it is needed? What uses cases are relevant when this feature gains relevance ?

@@ -1104,6 +1104,19 @@ are acceptable states.
Integrity Registers can be used to model the PCRs in a TPM or vTPM, in which case the identifier is the register index, or other kinds of vendor-specific measured objects.


Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@andrew-draper requests this PR to contain a matching algorithm.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Also example diags.

@yogeshbdeshpande
Copy link
Collaborator

Add matching rules to the PR

Add examples to the PR, in CDDL

Copy link
Collaborator

@yogeshbdeshpande yogeshbdeshpande left a comment

Choose a reason for hiding this comment

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

Please continue good work on the PR

@yogeshbdeshpande
Copy link
Collaborator

@deeglaze I assume, there is still work to be done on this PR, so we would not take this up during CoRIM Meeting on 29th Jan 2025 ?

As a means of representing a CPL or VMPL, this provides some extra
semantic understanding. Leaving condition-matching to a later PR.
This adds taggs 564 and 565 (563 proposed by PR ietf-rats-wg#297).
The -Infinity is valid diagnostic notation, but our tooling does not
recognize it.
Copy link
Collaborator

@nedmsmith nedmsmith left a comment

Choose a reason for hiding this comment

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

See inline comments

draft-ietf-rats-corim.md Outdated Show resolved Hide resolved
deeglaze and others added 2 commits January 29, 2025 21:34
Co-authored-by: Ned Smith <ned.smith@intel.com>
Copy link
Collaborator

@nedmsmith nedmsmith left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@nedmsmith nedmsmith left a comment

Choose a reason for hiding this comment

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

LGTM

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