-
Notifications
You must be signed in to change notification settings - Fork 7
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
base: main
Are you sure you want to change the base?
Conversation
595848e
to
879c4b2
Compare
@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. | |||
|
|||
|
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also example diags.
Add matching rules to the PR Add examples to the PR, in CDDL |
There was a problem hiding this 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
@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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See inline comments
Co-authored-by: Ned Smith <ned.smith@intel.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
As a means of representing a CPL or VMPL, this provides some extra semantic understanding. Leaving condition-matching to a later PR.