Skip to content

gcd: Move mapping memory range message prior to adding memory space#1509

Merged
makubacki merged 1 commit into
OpenDevicePartnership:mainfrom
makubacki:gcd_move_map_mem_range_msg
May 14, 2026
Merged

gcd: Move mapping memory range message prior to adding memory space#1509
makubacki merged 1 commit into
OpenDevicePartnership:mainfrom
makubacki:gcd_move_map_mem_range_msg

Conversation

@makubacki
Copy link
Copy Markdown
Collaborator

Description

Allows the memory range details to be visible if the call to GCD.add_memory_space() fails.

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

How This Was Tested

  • cargo make all
  • Boot and examine when the message is printed.

Integration Instructions

  • N/A

Allows the memory range details to be visible if the call to
GCD.add_memory_space() fails.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@makubacki makubacki requested review from Javagedes, joschock and os-d May 13, 2026 22:23
@makubacki makubacki self-assigned this May 13, 2026
@github-actions github-actions Bot added the impact:non-functional Does not have a functional impact label May 13, 2026
@patina-automation
Copy link
Copy Markdown
Contributor

patina-automation Bot commented May 13, 2026

✅ QEMU Validation Passed

All QEMU validation jobs completed successfully.

Note: Q35 is only built on Windows hosts (QEMU boot is disabled due to a QEMU vfat issue).

Workflow run: https://github.com/OpenDevicePartnership/patina/actions/runs/25830468546

Boot Time to EFI Shell

Platform Elapsed
Q35 (Linux Host) 28.1s
SBSA (Linux Host) 38.6s

Dependencies

Repository Ref
patina 43b7ec9
patina-dxe-core-qemu c7e0191
patina-fw-patcher d656f1d
patina-qemu firmware v3.0.0
patina-qemu build script e0231b7

This comment was automatically generated by the Patina QEMU PR Validation Post workflow.

Comment thread patina_dxe_core/src/gcd.rs
@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@makubacki makubacki requested a review from vineelko May 14, 2026 20:31
@makubacki makubacki merged commit f6d632c into OpenDevicePartnership:main May 14, 2026
11 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants