Skip to content

Conversation

@cfernald
Copy link
Contributor

@cfernald cfernald commented Dec 4, 2025

Description

patina_dxe_core: Add support for reloading the core from the debugger

This commit implements a new feature that allows the Patina core to be
reloaded from the debugger without needing a full system reboot or reflash.
This is particularly useful for development and debugging, as it enables
rapid iteration on core changes. This feature is intended to be coordinated
from the UefiExt debugger extension. This feature is experimental and
is likely to evolve and change.

patina_debugger: Send nack packet on debugger reset

When a error occurs, it may because a packet was a packet was corrupted
or partially received. In this case, it is better to send a NACK packet
to the host to request a retransmission, rather than sending a stop code.

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

How This Was Tested

  • Q35
  • SBSA
  • Intel Platform

Integration Instructions

Platforms may add the debugger_reload feature from patina_dxe_core to enable this feature.

@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@cfernald
Copy link
Contributor Author

cfernald commented Dec 4, 2025

Related: microsoft/uefi_debug_tools#12

@makubacki makubacki added the type:feature-request A new feature proposal label Dec 4, 2025
@os-d os-d changed the title Implement reloading the core through the debugger [REBASE & FF] Implement reloading the core through the debugger Dec 4, 2025
@cfernald cfernald force-pushed the dbg_restart branch 6 times, most recently from e836ac9 to 223fe68 Compare December 6, 2025 01:03
@github-actions github-actions bot added the type:documentation Improvements or additions to documentation label Dec 6, 2025
When a error occurs, it may because a packet was a packet was corrupted
or partially received. In this case, it is better to send a NACK packet
to the host to request a retransmission, rather than sending a stop code.
This commit implements a new feature that allows the Patina core to be
reloaded from the debugger without needing a full system reboot or reflash.
This is particularly useful for development and debugging, as it enables
rapid iteration on core changes. This feature is intended to be coordinated
from the UefiExt debugger extension. This feature is experimental and
is likely to evolve and change.
@cfernald cfernald enabled auto-merge (rebase) December 8, 2025 18:15
@cfernald cfernald merged commit cd2e774 into OpenDevicePartnership:main Dec 8, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:documentation Improvements or additions to documentation type:feature-request A new feature proposal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants