Skip to content

Conversation

@os-d
Copy link
Contributor

@os-d os-d commented Dec 2, 2025

Description

This PR adds stability tests to Patina DXE Core. It also adds timer based patina test dispatch.

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

How This Was Tested

Tested on Q35 and SBSA.

Integration Instructions

See docs.

@github-actions github-actions bot added impact:non-functional Does not have a functional impact impact:testing Affects testing type:documentation Improvements or additions to documentation labels Dec 2, 2025
@codecov
Copy link

codecov bot commented Dec 2, 2025

Codecov Report

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

Files with missing lines Patch % Lines
sdk/patina/src/test.rs 87.50% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@os-d os-d force-pushed the thrasher branch 3 times, most recently from afcbfd8 to 24c1d79 Compare December 4, 2025 17:01
os-d added 3 commits December 4, 2025 10:33
A bug was introduced to patina_test to run the tests twice in a
row. This fixes that to only run them once.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
This commit adds a new trigger type for patina_tests: timer.
Test writers can specify a timer interval in units of 100ns, i.e.
timer ticks, to have their test executed on.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
This commit adds a new module to patina_dxe_core: patina_test_support.
This module contains helper functions for executing patina_tests
relevant to patina_dxe_core. It is intended to have submodules for
different kind of tests being run. This commit only adds stability
tests, but future modules will be correctness, audit tests, etc.

The first set of stability tests are page table validation based,
particularly for large page splitting and unmapping.
@os-d os-d enabled auto-merge (rebase) December 4, 2025 18:33
@os-d os-d merged commit 7c220e8 into OpenDevicePartnership:main Dec 4, 2025
8 checks passed
@os-d os-d deleted the thrasher branch December 4, 2025 19:05
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 impact:testing Affects testing type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants