Hi,
> >
> > Add Intel Tdx helper library. The library provides the routines to:
> > -  set or clear Shared bit for a given memory region.
> > -  query whether TDX is enabled.
> 
> Hmm, patch 22 adds functions to set the shared bit too.
> Looks like duplicate functionality on a first glance.
> 
Ah yes, patch 22 (setting the shared bit) is duplicated with 
BaseMemEncryptTdxLib. And setting shared bit for Mmio space can be moved to 
TdxDxe driver (It is more reasonable). Then MdeModulePkg/Core/DxeIplPeim can be 
un-touched. To be honest I am a little nervous to touch the core code.
Thanks much for the reminder.

Min


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#84496): https://edk2.groups.io/g/devel/message/84496
Mute This Topic: https://groups.io/mt/86739894/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to