Repository navigation
doc: define the single-slot host filesystem boundary - #500
Merged
Pedro Henrique Penna (ppenna) merged 2 commits intoOct 12, 2026
Merged
Pedro Henrique Penna (ppenna) merged 2 commits into
Pedro Henrique Penna (ppenna) merged 2 commits into
Conversation
Describe directory and aggregate host exports within the fixed filesystem slot, and clarify the dormant-slot restore exception. Keep detailed syntax and policy contracts in the linked filesystem ABI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot started reviewing on behalf of
Pedro Henrique Penna (ppenna)
October 11, 2026 23:11
View session
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The restore wording omits the snapshot capability required to bind a new filesystem attachment.
1 open finding
What changed in this PR
Defines the single-slot host filesystem configuration boundary and restore behavior.
Changes:
- Documents single-directory and aggregate HostFs exports.
- Clarifies dormant-slot restore attachment behavior.
| File | Description |
|---|---|
doc/design/configuration-boundary.md |
Documents HostFs slot configuration and restore semantics. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Clarify the conditions and constraints for restore operations compared to cold boot. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot started reviewing on behalf of
Pedro Henrique Penna (ppenna)
October 12, 2026 11:27
View session
Pedro Henrique Penna (ppenna)
deleted the
doc-sync/filesystem-configuration-boundary
branch
October 12, 2026 11:45
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
Fill the host-filesystem configuration gap in the configuration boundary. The chapter was last edited at NVX commit
5f5bd9f3d, before the aggregate and regular-file export changes reached the OpenVMM pin.Keep this to one architecture-level topic in one document: explain how multiple host paths fit the fixed device inventory, and make the dormant-slot restore exception explicit. Link to the existing filesystem ABI instead of repeating its option syntax, export-policy details, or compatibility rules. No implementation or gitlink changes.
Base:
origin/devata5df986e5e606a82add90646fb6a3f9de0a2ec69.Changes
fs:microvm0slot, with either--mountfor a directory or--mount-aggregateand--mount-childfor named directory or regular-file children. Evidence: fixed filesystem inventory, CLI arguments, and file-child construction tests.OpenVMM revision
2130f24b5a51561cc6224fe5a0aff93cb121ac32(unchanged).The previous pin when this chapter was last edited was
3796991d52ce07d88b5dba25e4d1e8deaff6fd95; the intervening pinned commits were inspected, and the claims above were checked against the current source and tests, not the OpenVMM Guide or PR descriptions.Follow-ups
doc/in a later run. Conservatively defer that relocation because its inbound links are in the CI and setup guides, which open PRs test-microvm: pause and resume a microVM through OpenVMM state control #405 and Adopt bind-once microVM image slots #406 modify. All files changed by open PRs were left untouched.Validation
python -m unittest scripts.test_nvx_tools.CliTests scripts.test_performance.PerformanceTests- PASS, 117 tests,OK (skipped=1).doc/design.mdlinks all 16 design chapters.git diff --check- PASS.python scripts/nvx.py verify- PASS, both during setup and after editing; the OpenVMM checkout matches the recorded gitlink.No builds or hypervisor runs were needed for this documentation-only change.