Re: [edk2-devel] [PATCH 1/1] BaseTools: FMMT GuidTool Auto Select Config file

2023-06-09 Thread Rebecca Cran
That won't work on Unix, where the separator is ':', not ';'. -- Rebecca Cran On 6/5/23 02:12, Yuwei Chen wrote: Currently, Python FMMT tool does not support automatically select FMMTConf.ini file which saves GuidTool settings. This patch support this features. Cc: Rebecca Cran Cc: Liming

[edk2-devel] [PATCH v1 3/3] MdeModulePkg: Add Phase-Specific MemoryProtectionHobLib Implementations

2023-06-09 Thread Taylor Beebe
Add DXE, SMM, and STANDALONE MM implementations of the MemoryProtectionHobLib. Signed-off-by: Taylor Beebe Cc: Jian J Wang Cc: Liming Gao Cc: Dandan Bi --- MdeModulePkg/Library/MemoryProtectionHobLib/DxeMemoryProtectionHobLib.c | 182

[edk2-devel] [PATCH v1 2/3] MdeModulePkg: Add MemoryProtectionHobLib Definitions and NULL Libs

2023-06-09 Thread Taylor Beebe
DxeMemoryProtectionHobLib and MmMemoryProtectionHobLib will fetch the memory protection settings HOB entry for their respective phase, validate the settings, and populate a global for access. Memory protection settings are currently dictated via FixedAtBuild PCDs where the settings needed to be

[edk2-devel] [PATCH v1 1/3] MdeModulePkg: Add DXE and MM Memory Protection Settings HOB Definitions

2023-06-09 Thread Taylor Beebe
These headers provide settings definitions for memory protections, settings profiles for easily enabling memory protections, and the GUIDs used for producing the memory protection HOB. The settings options are functionally 1:1 with the existing PCD bitfield definitions. Instead of setting a fixed

[edk2-devel] [PATCH v1 0/3] MdeModulePkg: Adding Dynamic Memory Protection Settings Libraries

2023-06-09 Thread Taylor Beebe
Memory protection settings are currently configured via FixedAtBuild PCDs which resulted in a build-time configuration of memory mitigations. To improve flexibility and compatibility, this patchset adds HOB definitions and libraries required for configuring memory protection settings at runtime.

Re: [edk2-devel] [PATCH 1/1] MdeModulePkg: Cache device path during LoadImage calls

2023-06-09 Thread Nolan Hergert
Any feedback? From: devel@edk2.groups.io on behalf of Nolan Hergert Sent: Wednesday, May 24, 2023 12:07 PM To: devel@edk2.groups.io Cc: Bi, Dandan ; Gao, Liming Subject: [edk2-devel] [PATCH 1/1] MdeModulePkg: Cache device path during LoadImage calls  

[edk2-devel] [PATCH v2 0/2] MdeModulePkg/Bus/Ata/AtaAtapiPassThru: Fix

2023-06-09 Thread Ranbir Singh
v1 -> v2: - Retain outer cast - Add error check instead of Status storage removal Ranbir Singh (2): MdeModulePkg/Bus/Ata/AtaAtapiPassThru: Fix SIGN_EXTENSION Coverity issue MdeModulePkg/Bus/Ata/AtaAtapiPassThru: Fix UNUSED_VALUE Coverity issue

[edk2-devel] [PATCH] Get HobListPtr before calling the Multiphase FSPS

2023-06-09 Thread kurugodx
REF : https://bugzilla.tianocore.org/show_bug.cgi?id=4480 In the FspsWrapperPeim, before calling FspWrapperVariableRequestHandler and FspWrapperMultiPhaseHandler ,FspHobListPtr should be available so that BL will be able to get the correct FspHobListPtr value Signed-off-by: kurugodx Cc: Chasel

[edk2-devel] [PATCH v2 2/2] MdeModulePkg/Bus/Ata/AtaAtapiPassThru: Fix UNUSED_VALUE Coverity issue

2023-06-09 Thread Ranbir Singh
From: Ranbir Singh The return value stored in Status after call to SetDriveParameters is not made of any use thereafter and hence it remains as UNUSED. Add error check as is done after calls to SetDeviceTransferMode. Cc: Hao A Wu Cc: Ray Ni REF:

[edk2-devel] [PATCH v2 1/2] MdeModulePkg/Bus/Ata/AtaAtapiPassThru: Fix SIGN_EXTENSION Coverity issue

2023-06-09 Thread Ranbir Singh
From: Ranbir Singh Line number 1348 does contain a typecast with UINT32, but it is after all the operations (16-bit left shift followed by OR'ing) are over. To avoid any SIGN_EXTENSION, typecast the intermediate result after 16-bit left shift operation immediately with UINT32. Cc: Hao A Wu Cc:

[edk2-devel] [PATCH] Get HobListPtr before calling the Multiphase FSPS

2023-06-09 Thread kurugodx
REF : https://bugzilla.tianocore.org/show_bug.cgi?id=4480 In the FspsWrapperPeim, before calling FspWrapperVariableRequestHandler and FspWrapperMultiPhaseHandler ,FspHobListPtr should be available so that BL will be able to get the correct FspHobListPtr value Signed-off-by: kurugodx Cc: Chasel

Re: [edk2-devel] [PATCH] Get HobListPtr before calling the Multiphase FSPS

2023-06-09 Thread Chiu, Chasel
Please update the subject per format requirement - PackageName: title With this update, Reviewed-by: Chasel Chiu Thanks, Chasel > -Original Message- > From: Aishwarya, KurugoduMelmatamX > > Sent: Friday, June 9, 2023 6:40 AM > To: devel@edk2.groups.io > Cc: Aishwarya,

Re: [edk2-devel] [PATCH V2 6/6] MdeModulePkg/SmmCorePerformanceLib: Disable perf-logging at runtime

2023-06-09 Thread Wu, Jiaxin
Reviewed-by: Jiaxin Wu > -Original Message- > From: Ni, Ray > Sent: Wednesday, May 31, 2023 7:35 PM > To: devel@edk2.groups.io > Cc: Wu, Jiaxin ; Bi, Dandan ; > Gao, Liming ; Wang, Jian J > > Subject: [PATCH V2 6/6] MdeModulePkg/SmmCorePerformanceLib: Disable > perf-logging at

Re: [edk2-devel] [PATCH V2 5/6] MdeModulePkg/SmmPerformanceLib: Disable perf-logging after ExitBS

2023-06-09 Thread Wu, Jiaxin
Reviewed-by: Jiaxin Wu > -Original Message- > From: Ni, Ray > Sent: Wednesday, May 31, 2023 7:35 PM > To: devel@edk2.groups.io > Cc: Wu, Jiaxin ; Bi, Dandan ; > Gao, Liming ; Wang, Jian J > > Subject: [PATCH V2 5/6] MdeModulePkg/SmmPerformanceLib: Disable perf- > logging after ExitBS

Re: [edk2-devel] [PATCH V2 4/6] MdeModulePkg/SmmCore: Add perf-logging for SmmDriverDispatchHandler

2023-06-09 Thread Wu, Jiaxin
Reviewed-by: Jiaxin Wu > -Original Message- > From: Ni, Ray > Sent: Wednesday, May 31, 2023 7:35 PM > To: devel@edk2.groups.io > Cc: Wang, Jian J ; Gao, Liming > ; Wu, Jiaxin > Subject: [PATCH V2 4/6] MdeModulePkg/SmmCore: Add perf-logging for > SmmDriverDispatchHandler > >

Re: [edk2-devel] [PATCH V2 3/6] MdeModulePkg/SmmCore: Add perf-logging for time-consuming procedures

2023-06-09 Thread Wu, Jiaxin
Could we also make below perf-logged? PlatformHookBeforeSmmDispatch & PlatformHookAfterSmmDispatch Others good to me. Reviewed-by: Jiaxin Wu Thanks, Jiaxin > -Original Message- > From: Ni, Ray > Sent: Wednesday, May 31, 2023 7:35 PM > To: devel@edk2.groups.io > Cc: Wang, Jian J

Re: [edk2-devel] [PATCH V2 2/6] UefiCpuPkg/CpuSmm: Add perf-logging for MP procedures

2023-06-09 Thread Wu, Jiaxin
> +// > > +// The list of all MP procedures that need to be perf-logged. > > +// > > +#define SMM_MP_PERF_PROCEDURE_LIST(_) \ > > + _(SmmInitHandler), \ > > + _(SmmRendezvousEntry), \ > > + _(PlatformValidSmi), \ > > + _(SmmRendezvousExit), \ > > + _(SmmMpProcedureMax) // Add new

Re: [edk2-devel] [PATCH] Get HobListPtr before calling the Multiphase FSPS

2023-06-09 Thread Ashraf Ali S
Reviewed-by: Ashraf Ali S -Original Message- From: Aishwarya, KurugoduMelmatamX Sent: Friday, June 9, 2023 7:10 PM To: devel@edk2.groups.io Cc: Aishwarya, KurugoduMelmatamX ; Chiu, Chasel ; Desimone, Nathaniel L ; Duggapu, Chinni B ; Chen, Gang C ; Zeng, Star ; Kuo, Ted ; S, Ashraf

Re: [edk2-devel] [PATCH V2 1/6] UefiCpuPkg/CpuSmm: Add perf-logging for time-consuming BSP procedures

2023-06-09 Thread Wu, Jiaxin
> > The patch adds perf-logging for the following potential > time-consuming BSP procedures: > * PiCpuSmmEntry > - SmmRelocateBases > * ExecuteFirstSmiInit > ExecuteFirstSmiInit is not belong to the SmmRelocateBases. Others good to me. Reviewed-by: Jiaxin Wu > * BSPHandler > -

[edk2-devel] [Patch V6 02/14] MdeModulePkg: Remove other attribute protection in UnsetGuardPage

2023-06-09 Thread duntan
In UnsetGuardPage(), before SmmReadyToLock, remove NX and RO memory attribute protection for guarded page since EfiConventionalMemory in SMRAM is RW and executable before SmmReadyToLock. If UnsetGuardPage() happens after SmmReadyToLock, then apply EFI_MEMORY_XP to the guarded page to make sure

[edk2-devel] [Patch V6 01/14] OvmfPkg:Remove code that apply AddressEncMask to non-leaf entry

2023-06-09 Thread duntan
Remove code that apply AddressEncMask to non-leaf entry when split smm page table by MemEncryptSevLib. In FvbServicesSmm driver, it calls MemEncryptSevClearMmioPageEncMask to clear AddressEncMask bit in page table for a specific range. In AMD SEV feature, this AddressEncMask bit in page table is

[edk2-devel] [Patch V6 00/14] Use CpuPageTableLib to create and update smm page table

2023-06-09 Thread duntan
In the V6 patch set: In 'MdeModulePkg: Remove other attribute protection in UnsetGuardPage', add code to apply EFI_MEMORY_XP to the guarded page to be freed in UnsetGuardPage() if it happens after SmmReadyToLock. Only resend the patches in MdeModulePkg and OvmfPkg for further review. Other

Re: [edk2-devel] [Patch V5 02/14] MdeModulePkg: Remove RO and NX protection when unset guard page

2023-06-09 Thread duntan
Hi Ard, Thanks for your question. This patch does cause a difference that NX protections maybe removed for some EfiConventionalMemory in SMRAM after SmmReadyToLock. Before SmmReadyToLock, EfiConventionalMemory in SMRAM is always RW and executable. When SmmReadyToLock, SetMemMapAttributes() in

Re: [edk2-devel] [PATCH v3 1/3] MdeModulePkg/Bus/Usb/UsbNetwork/UsbRndis: Add USB RNDIS devices support

2023-06-09 Thread RichardHo [何明忠] via groups . io
Hi Hao Wu, I will provide V4 for below next. #1 Modify DSC file in patch 3/3 #2 add Acked-by: Hao A Wu #3 modify DXE_DRIVER to UEFI_DRIVER Thanks, Richard -Original Message- From: Wu, Hao A Sent: 2023年6月8日 2:23 PM To: devel@edk2.groups.io; Richard Ho (何明忠) Cc: Andrew Fish ; Leif