On Wed, Jan 18, 2023 at 05:56:20PM +0800, Jiaxin Wu wrote: > This patch is to avoid configure SMBASE if SmBase relocation has been > done. If gSmmBaseHobGuid found, means SmBase info has been relocated > and recorded in the SmBase array. No need to do the relocation in > SmmCpuFeaturesInitializeProcessor().
Still not answered: Who produces the gSmmBaseHobGuid? If you intend to submit the producer code to edk2 please do so as part of this patch series, so it is possible to see the whole picture instead of only some pieces of the puzzle when reviewing the code. If you don't submit the producer code it is pointless to touch OVMF. You are only adding dead code. That would be a rather questionable decision though and add a big question mark to intels open source commitment. take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#98800): https://edk2.groups.io/g/devel/message/98800 Mute This Topic: https://groups.io/mt/96350764/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
