Skip to content

Conversation

@joschock
Copy link
Contributor

@joschock joschock commented Dec 1, 2025

Description

Adjusts wait_for_event semantics to better match historical behavior of core:

  • make out_index parameter optional
  • invoke CPU sleep function during idle loop.

Closes #791

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

How This Was Tested

Boot to OS on AARCH64 hardware.

Integration Instructions

N/A

@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

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

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

📢 Thoughts on this report? Let us know!

@joschock joschock force-pushed the wait_for_events_semantics branch 4 times, most recently from c5881d9 to 10cba41 Compare December 1, 2025 23:55
@joschock joschock force-pushed the wait_for_events_semantics branch from 10cba41 to 95b9a6f Compare December 2, 2025 18:19
@joschock joschock enabled auto-merge (squash) December 2, 2025 18:19
@joschock joschock merged commit 0733821 into OpenDevicePartnership:main Dec 2, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: events::wait_for_event needs to be updated to match EDK2 semantics.

4 participants