Skip to content

guide: add instructions on specifying a UEFI firmware file#2874

Merged
mattkur merged 4 commits intomicrosoft:mainfrom
mattkur:minor-guide-updates
Mar 5, 2026
Merged

guide: add instructions on specifying a UEFI firmware file#2874
mattkur merged 4 commits intomicrosoft:mainfrom
mattkur:minor-guide-updates

Conversation

@mattkur
Copy link
Contributor

@mattkur mattkur commented Mar 3, 2026

I found myself confused when running openvmm to boot from a VHD with a UEFI formware. So, add a note in the guide.

@mattkur mattkur requested a review from a team as a code owner March 3, 2026 05:01
Copilot AI review requested due to automatic review settings March 3, 2026 05:01
@github-actions github-actions bot added the Guide label Mar 3, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation clarifying how to provide a UEFI firmware image when running OpenVMM outside of cargo run, which otherwise sets firmware-related environment variables automatically via .cargo/config.toml.

Changes:

  • Add a new “UEFI firmware required when running outside cargo” tip block to the “Running OpenVMM” guide with example invocation and common firmware paths.
  • Update the CLI reference to document --uefi-firmware and its relationship to environment-variable defaults.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
Guide/src/user_guide/openvmm/run.md Adds a tip explaining why --uefi-firmware is needed outside cargo run, with example usage and firmware locations.
Guide/src/reference/openvmm/management/cli.md Documents the --uefi-firmware flag and its defaulting behavior via environment variables.

mattkur and others added 2 commits March 2, 2026 21:10
@mattkur mattkur merged commit f102fc3 into microsoft:main Mar 5, 2026
56 checks passed
@mattkur mattkur deleted the minor-guide-updates branch March 5, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants