Skip to content

[RFC] Rust supervisor proposal#1170

Merged
kuqin12 merged 54 commits intoOpenDevicePartnership:mainfrom
kuqin12:rust_supv_rfc
Apr 3, 2026
Merged

[RFC] Rust supervisor proposal#1170
kuqin12 merged 54 commits intoOpenDevicePartnership:mainfrom
kuqin12:rust_supv_rfc

Conversation

@kuqin12
Copy link
Copy Markdown
Contributor

@kuqin12 kuqin12 commented Dec 8, 2025

Description

This proposal intends to provide a supervised Standalone MM solution with Rust.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

RFC only.

Integration Instructions

N/A

@kuqin12 kuqin12 self-assigned this Dec 8, 2025
@kuqin12 kuqin12 added the rfc A request-for-change proposal label Dec 8, 2025
@github-actions github-actions Bot added impact:non-functional Does not have a functional impact impact:testing Affects testing labels Dec 8, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
@cfernald cfernald self-requested a review December 8, 2025 18:56
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
@kuqin12 kuqin12 requested a review from makubacki December 10, 2025 00:11
@joschock
Copy link
Copy Markdown
Contributor

General feedback: since this is a Patina RFC, I think it would be good to have a section that helps to better understand how this RFC would impact and connect with existing Patina code. For example, here are some questions I think you could answer to clarify the Patina/MM relationship:

  1. Will the MM core be a separate repo? Or do you plan to build an MM core lib crate in parallel with patina_dxe_core?
  2. How do you anticipate the MM consuming patina modules (like allocator/events/protocol_db)? Or do you plan to use patina modules as reference and just fork new separate MM implementations?
  3. Are there any significant architectural changes required to existing patina need to accommodate/enable the MM design?

kuqin12 and others added 9 commits January 9, 2026 13:20
Co-authored-by: Oliver Smith-Denny <osde@microsoft.com>
Co-authored-by: Oliver Smith-Denny <osde@microsoft.com>
Co-authored-by: Oliver Smith-Denny <osde@microsoft.com>
Co-authored-by: Oliver Smith-Denny <osde@microsoft.com>
Co-authored-by: Oliver Smith-Denny <osde@microsoft.com>
Co-authored-by: Oliver Smith-Denny <osde@microsoft.com>
@kuqin12
Copy link
Copy Markdown
Contributor Author

kuqin12 commented Jan 10, 2026

General feedback: since this is a Patina RFC, I think it would be good to have a section that helps to better understand how this RFC would impact and connect with existing Patina code. For example, here are some questions I think you could answer to clarify the Patina/MM relationship:

  1. Will the MM core be a separate repo? Or do you plan to build an MM core lib crate in parallel with patina_dxe_core?
  2. How do you anticipate the MM consuming patina modules (like allocator/events/protocol_db)? Or do you plan to use patina modules as reference and just fork new separate MM implementations?
  3. Are there any significant architectural changes required to existing patina need to accommodate/enable the MM design?

Thanks, I updated the RFC to include 2 nutrient content tables to reflect such. But the short summary is that I would like to make the new supevisor sitting in parallel with dxe_core amd move some common functional modules out from patina_core_dxe so that they can be shared.

Then for the newly authored code, such as syscall dispatcher and privilege manager, I intend to produce them as their own crate. What if we want to use them in DXE core one day..? 😈

Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0026-mm-supv.md
Copy link
Copy Markdown
Contributor

@os-d os-d left a comment

Choose a reason for hiding this comment

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

I'm sorry for all the grammar nits but I can't help it... :)

Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Copy link
Copy Markdown
Contributor

@os-d os-d left a comment

Choose a reason for hiding this comment

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

I'm sorry for all the grammar nits but I can't help it... :)

Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0026-mm-supv.md
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0026-mm-supv.md
Comment thread docs/src/rfc/text/0026-mm-supv.md
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
Comment thread docs/src/rfc/text/0000-mm-supv.md Outdated
@github-actions github-actions Bot added the type:documentation Improvements or additions to documentation label Apr 3, 2026
@kuqin12 kuqin12 enabled auto-merge (squash) April 3, 2026 00:35
@kuqin12 kuqin12 merged commit 539e18e into OpenDevicePartnership:main Apr 3, 2026
11 checks passed
@kuqin12 kuqin12 deleted the rust_supv_rfc branch April 3, 2026 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:non-functional Does not have a functional impact impact:testing Affects testing rfc A request-for-change proposal type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants