Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pcr bank change fix rebased #165

Merged
merged 2 commits into from
Sep 16, 2024
Merged

Pcr bank change fix rebased #165

merged 2 commits into from
Sep 16, 2024

Conversation

miczyg1
Copy link
Contributor

@miczyg1 miczyg1 commented Sep 16, 2024

No description provided.

Tcg2PhysicalPresenceLibSubmitRequestToPreOSFunction writes to TPM2
physical presence PPI provided by coreboot (a memory region preserved
across reboots). CPU caches must be explicitly flushed prior to platform
reboot or request written to PPI will be lost.

Signed-off-by: Artur Kowalski <[email protected]>
Flush cache not only when placing request in PPI, but also after
clearing old request from PPI.

Signed-off-by: Artur Kowalski <[email protected]>
@miczyg1
Copy link
Contributor Author

miczyg1 commented Sep 16, 2024

Changes cherry-picked from #164

@miczyg1
Copy link
Contributor Author

miczyg1 commented Sep 16, 2024

Tested on VP4670. Since it was already review by me and merged to older EDK2 branch, merging it here as well.

@miczyg1 miczyg1 merged commit 4d1eaf5 into rebased Sep 16, 2024
1 check passed
@miczyg1 miczyg1 deleted the pcr-bank-change-fix-rebased branch September 16, 2024 11:28
@mkopec
Copy link
Member

mkopec commented Oct 2, 2024

Doesn't seem to help on NovaCustom V56 (SLB9672)

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.

3 participants