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

Rename MM service group to SMM and update services #4

Closed
wants to merge 2 commits into from

Conversation

pathakraul
Copy link
Collaborator

This PR updates the MM_SERVICE service group to SECURE_MANAGEMENT_MODE service group and use prefix SMM_ for services and fields.

SMM service group is different from PI MM Mode which is implemented in secure execution environment. MM mode also happens to provide services and SMM service group messages are used to invoke the MM services.

.Management Services
=== Service Group - *SECURE_MANAGEMENT_MODE* (servicegroup_id: 0x0000A)
This secure management mode service group is used for software invocation of
Management Mode (MM) in a secure execution environment. PI Management Mode (MM)
Copy link
Collaborator

Choose a reason for hiding this comment

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

If first time use "PI", then put the full name "Platform Initialization (PI)".

@pathakraul
Copy link
Collaborator Author

Since SMM will become a new protocol instead of RPMI service group, lets hold this PR for final decision

@lftan
Copy link
Collaborator

lftan commented Mar 26, 2024

Okay

@pathakraul pathakraul closed this Apr 8, 2024
lftan pushed a commit to lftan/riscv-rpmi that referenced this pull request Apr 16, 2024
Add "CHAN_SIZE" parameter to WORD riscv-non-isa#4.

Signed-off-by: Ley Foon Tan <[email protected]>
@pathakraul pathakraul deleted the dev branch June 9, 2024 02:53
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