Skip to content

Conversation

@vineelko
Copy link
Contributor

@vineelko vineelko commented Dec 8, 2025

Description

  • Including the image size helps when inspecting memory in the debugger.
  • This is especially useful when loading symbols on demand in lldb.
    --
  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

booted q35 to shell

Integration Instructions

NA

@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Dec 8, 2025
@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
patina_dxe_core/src/image.rs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@vineelko vineelko force-pushed the users/vineelko/minor_fix_print_loaded_size_1208 branch from 30c20c9 to 1347e9b Compare December 11, 2025 20:25
- Including the image size helps when inspecting memory in the debugger.

Signed-off-by: Vineel Kovvuri[MSFT] <vineelko@microsoft.com>
@vineelko vineelko force-pushed the users/vineelko/minor_fix_print_loaded_size_1208 branch from 1347e9b to c6a55a7 Compare December 11, 2025 20:26
@vineelko vineelko merged commit 0aeeb58 into OpenDevicePartnership:main Dec 11, 2025
7 of 8 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.

4 participants