search

AMDESE / ovmf

EDK2 / OVMF related projects
BSD 2-Clause "Simplified" License
6 stars 21 forks source link

Compilation fails with smm #1

Open jirislaby opened 5 years ago

jirislaby commented 5 years ago

With the current patchset, OVMF build fails for the ovmf-x86_64-smm flavor (which is by default built in SUSE). The errors are like:

OvmfPkg/QemuFlashFvbServicesRuntimeDxe/QemuFlash.c:14:10: fatal error: Library/VmgExitLib.h: No such file or directory
#include <Library/VmgExitLib.h>
         ^~~~~~~~~~~~~~~~~~~~~~
...
build.py...
 : error F002: Failed to build module
       OvmfPkg/QemuFlashFvbServicesRuntimeDxe/FvbServicesSmm.inf [X64, GCC5, DEBUG]

Another one is:

/tmp/cc7KG5f7.ltrans0.ltrans.o: In function `CommonExceptionHandlerWorker':
UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiDxeSmmCpuException.c:95: undefined reference to `DoVcException'

It seems to be expected as e.g. UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptionHandlerLib.inf does not include any of the implementations of DoVcException. So is SMM ignored in this submission?

tlendacky commented 5 years ago

On 11/26/19 6:45 AM, Jiri Slaby wrote:

With the current patchset, OVMF build fails for the ovmf-x86_64-smm flavor (which is by default built in SUSE). The errors are like:

OvmfPkg/QemuFlashFvbServicesRuntimeDxe/QemuFlash.c:14:10: fatal error: Library/VmgExitLib.h: No such file or directory
#include <Library/VmgExitLib.h>
         ^~~~~~~~~~~~~~~~~~~~~~
...
build.py...
 : error F002: Failed to build module
       OvmfPkg/QemuFlashFvbServicesRuntimeDxe/FvbServicesSmm.inf [X64, GCC5, DEBUG]

Another one is:

/tmp/cc7KG5f7.ltrans0.ltrans.o: In function `CommonExceptionHandlerWorker':
UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiDxeSmmCpuException.c:95: undefined reference to `DoVcException'

It seems to be expected as e.g. UefiCpuPkg/Library/CpuExceptionHandlerLib/SmmCpuExceptionHandlerLib.inf does not include any of the implementations of DoVcException. So is SMM ignored in this submission?

SMM is not supported in (this release of) SEV-ES... but, it shouldn't cause a build failure. Thanks for reporting it, I'll take care of it.

Thanks, Tom