Skip to content

Comments

Fix AEM for Intel TXT on UEFI platforms#30

Merged
krystian-hebel merged 8 commits intotb-devfrom
intel-txt-aem-efi-fixes
Mar 25, 2025
Merged

Fix AEM for Intel TXT on UEFI platforms#30
krystian-hebel merged 8 commits intotb-devfrom
intel-txt-aem-efi-fixes

Conversation

@SergiiDmytruk
Copy link
Member

See commit messages for explanation of the changes.

This fixes TrenchBoot/trenchboot-issues#56

Out of grub_vbe_enable_mtrr() in video/i386/pc/vbe.c

Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
Instead of an abstract memory range.

Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
In this case MTRRs masks should be limited to MAXPHYSADDR bits as
reported by CPUID instead of to a fixed width of 36 bits.

Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
Boot params are set only for Linux boot process and it being NULL won't
stop the code from adding policy entries that can confuse DLME.

Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
Just in case.

Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
The correct offset is a requirement for TXT and layout of sections in
memory can be very different from the layout in a file.

Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
* Add missing newlines.
* Use grub_dprintf() to provide more context, each next grub_error()
  discards the message of the previous one unless error stack is pushed.

Signed-off-by: Sergii Dmytruk <sergii.dmytruk@3mdeb.com>
@SergiiDmytruk SergiiDmytruk force-pushed the intel-txt-aem-efi-fixes branch from c14b586 to b551778 Compare March 24, 2025 19:43
@krystian-hebel krystian-hebel merged commit b551778 into tb-dev Mar 25, 2025
1 check passed
@krystian-hebel krystian-hebel deleted the intel-txt-aem-efi-fixes branch March 25, 2025 13:27
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.

2 participants