On Tue, Dec 14, 2021 at 09:41:24PM +0800, Min Xu wrote:
> RFC: https://bugzilla.tianocore.org/show_bug.cgi?id=3429
>
> Tdvf Config-B skip PEI phase to reduce attack surface. So instead of
> jumping to SecStartupPhase2 (), TdxStartup () is called. This function
> brings up Tdx guest from SEC phase to DXE phase.
> + #ifdef INTEL_TDX_FULL_FEATURE
> + if (SecTdxIsEnabled ()) {
> + TdxStartup (&SecCoreData);
> +
> + //
> + // Never arrived here
> + //
> + ASSERT (FALSE);
> + CpuDeadLoop ();
> + }
> +
> + #endif
Oh, wow. So you compile in PEI, then decide at runtime whenever you
use it or not?
No. Please don't. That's just silly. If you don't want use PEI, ok,
fine, but please go the way then, remove PEI from the build and take
the PEI-less code path in all cases.
take care,
Gerd
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84892): https://edk2.groups.io/g/devel/message/84892
Mute This Topic: https://groups.io/mt/87720802/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
