Skip to content

[CHERRY-PICK] UefiTestingPkg: Correctly handle cached DMA audit ACPI tables - #941

Merged
Aaron (apop5) merged 1 commit into
microsoft:release/202608from
eeshanl:dmaprotectionacpi-release-202608
Sep 15, 2026
Merged

Aaron (apop5) merged 1 commit into
microsoft:release/202608from
eeshanl:dmaprotectionacpi-release-202608

Conversation

@eeshanl

@eeshanl Eeshan Londhe (eeshanl) commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Description

Cherry-pick a8b711b (#938) into release/202608. Related issue: #937.

Correctly handle cached DMA audit ACPI tables. Fixes #937

Tests call GetDmarAcpiTable(), but the Intel VT-d and IVRS
implementation caches the ACPI table and returns EFI_ALREADY_STARTED on
the second call without a reboot.

GetAcpiTable() will no longer return already started and will always
refresh the pointer with the data. On the caller side, the caching of
the global variable is handled so we don't call GetAcpiTable()
redundantly.

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

How This Was Tested

Tested on intel platform and all tests now pass.

Integration Instructions

N/A

…oft#938)

## Description

Correctly handle cached DMA audit ACPI tables. Fixes
microsoft#937

Tests call GetDmarAcpiTable(), but the Intel VT-d and IVRS
implementation caches the ACPI table and returns EFI_ALREADY_STARTED on
the second call without a reboot.

GetAcpiTable() will no longer return already started and will always
refresh the pointer with the data. On the caller side, the caching of
the global variable is handled so we don't call GetAcpiTable()
redundantly.

- [ ] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [ ] Includes tests?
- [ ] Includes documentation?

## How This Was Tested

Tested on Intel platform and all tests now pass.

## Integration Instructions

N/A

(cherry picked from commit a8b711b)
@mu-automation mu-automation Bot added the type:documentation Improvements or additions to documentation label Sep 15, 2026
@eeshanl Eeshan Londhe (eeshanl) changed the title [release/202608] Correctly handle cached DMA audit ACPI tables [CHERRY-PICK] Correctly handle cached DMA audit ACPI tables Sep 15, 2026
@eeshanl Eeshan Londhe (eeshanl) changed the title [CHERRY-PICK] Correctly handle cached DMA audit ACPI tables [CHERRY-PICK] UefiTestingPkg: Correctly handle cached DMA audit ACPI tables Sep 15, 2026
@eeshanl Eeshan Londhe (eeshanl) removed the type:documentation Improvements or additions to documentation label Sep 15, 2026
@mu-automation mu-automation Bot added the impact:non-functional Does not have a functional impact label Sep 15, 2026
@mu-automation

mu-automation Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

⏩ QEMU Validation Skipped

Validation was skipped (reason: wrong_target_branch).

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

@apop5
Aaron (apop5) merged commit 9c1822d into microsoft:release/202608 Sep 15, 2026
51 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