On 3/17/25 4:21 PM, Nikunj A. Dadhania wrote:
Hi Vaishali,
Thanks for testing the patches.
On 3/17/2025 8:37 PM, Tom Lendacky wrote:
On 3/17/25 07:52, Vaishali Thakkar wrote:
On 3/17/25 6:23 AM, Nikunj A Dadhania wrote:
Hi Nikunj,
I've been trying to test this patchset with the above QEMU command line
and with the OVMF built from upstream master. But I'm encountering
following errors:
" !!!!!!!! Image Section Alignment(0x40) does not match Required Alignment
(0x1000) !!!!!!!!
ProtectUefiImage failed to create image properties record "
I bisected EDK2/OVMF and found that the above messages started appearing
with commit 37f63deeefa8 ("MdeModulePkg: MemoryProtection: Use
ImageRecordPropertiesLib")
It doesn't appear to cause any issues while booting as I'm able to
progress to the grub menu and boot the OS. Is it failing for you?
Thanks,
Tom
I briefly looked at this[1] branch as well but it appears to be no longer
actively maintained as I ran into some build errors which are fixed in
upstream.
The build command I'm using to build the OVMF is as follows:
build -a X64 -b DEBUG -t GCC5 -D DEBUG_VERBOSE -p OvmfPkg/OvmfPkgX64.dsc
So, I was wondering if you've some extra patches on top of upstream OVMF
to test SecureTSC or are there any modifications required in my build
command?
No, I do not have any Secure TSC related modification in OVMF. If it can boot
SNP, that should work for Secure TSC guests as well.
Thanks both for confirming that those OVMF messages were a red herring. It
helped
me uncover an issue unrelated to this patchset. I have been able to test this
patchset successfully now. For the whole patchset, please feel free to add:
Tested-by: Vaishali Thakkar <vaishali.thakkar@xxxxxxxx>
Regards
Nikunj
Thank you!
[1] https://github.com/AMDESE/ovmf/tree/snp-latest
