Hi, > - SetDxeMemoryProtectionSettings (&DxeSettings, > DxeMemoryProtectionSettingsPcd); > - SetMmMemoryProtectionSettings (&MmSettings, MmMemoryProtectionSettingsPcd); > + Status = QemuFwCfgParseString (DXE_MEMORY_PROTECTION_PROFILE_FWCFG_FILE, > &StringSize, String); > + if (!EFI_ERROR (Status)) { > + DEBUG ((DEBUG_INFO, "Setting DXE Memory Protection Profile: %a\n", > String)); > + if (AsciiStriCmp (String, "debug") == 0) { > + DxeSettings = > DxeMemoryProtectionProfiles[DxeMemoryProtectionSettingsDebug].Settings;
I'd suggest to just loop over DxeMemoryProtectionProfiles and compare String with .Name, so we don't have to touch this in case we add or remove profiles. > + DxeSettings = > DxeMemoryProtectionProfiles[DxeMemoryProtectionSettingsDebug].Settings; I'd prefer to use DxeMemoryProtectionSettingsPcd by default. take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#108089): https://edk2.groups.io/g/devel/message/108089 Mute This Topic: https://groups.io/mt/100830924/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-