Re: [edk2-devel] [PATCH v4 00/11] ArmPkg: implement EFI memory attributes protocol

2023-03-13 Thread Ard Biesheuvel
On Wed, 8 Mar 2023 at 18:24, Taylor Beebe wrote: > > My mistake - the DEBUG_PROPERTY_DEBUG_CLEAR_MEMORY_ENABLED feature is > why FreePagesWithProtectionAttributesTestCase might fail. > > To make the clear memory feature more compatible with the memory > attribute protocol, can you add a check to

Re: [edk2-devel] [PATCH v4 00/11] ArmPkg: implement EFI memory attributes protocol

2023-03-08 Thread Taylor Beebe
My mistake - the DEBUG_PROPERTY_DEBUG_CLEAR_MEMORY_ENABLED feature is why FreePagesWithProtectionAttributesTestCase might fail. To make the clear memory feature more compatible with the memory attribute protocol, can you add a check to DebugClearMemoryEnabled() on free and clear EFI_MEMORY_RP

Re: [edk2-devel] [PATCH v4 00/11] ArmPkg: implement EFI memory attributes protocol

2023-03-01 Thread Ard Biesheuvel
On Wed, 1 Mar 2023 at 21:43, Taylor Beebe wrote: > > > > On 2/11/2023 2:05 AM, Ard Biesheuvel wrote: > > On Sat, 11 Feb 2023 at 01:56, Taylor Beebe wrote: > >> > >> Hey Ard, > >> > >> Once the Memory Attribute Protocol is made available, Windows will have > >> some expectations about its

Re: [edk2-devel] [PATCH v4 00/11] ArmPkg: implement EFI memory attributes protocol

2023-03-01 Thread Taylor Beebe
On 2/11/2023 2:05 AM, Ard Biesheuvel wrote: On Sat, 11 Feb 2023 at 01:56, Taylor Beebe wrote: Hey Ard, Once the Memory Attribute Protocol is made available, Windows will have some expectations about its functionality. Can you run this test app created by me and Jiewen to ensure it meets

Re: [edk2-devel] [PATCH v4 00/11] ArmPkg: implement EFI memory attributes protocol

2023-02-11 Thread Ard Biesheuvel
On Sat, 11 Feb 2023 at 01:56, Taylor Beebe wrote: > > Hey Ard, > > Once the Memory Attribute Protocol is made available, Windows will have > some expectations about its functionality. Can you run this test app > created by me and Jiewen to ensure it meets the Windows requirements? > Part of the

Re: [edk2-devel] [PATCH v4 00/11] ArmPkg: implement EFI memory attributes protocol

2023-02-10 Thread Taylor Beebe
Hey Ard, Once the Memory Attribute Protocol is made available, Windows will have some expectations about its functionality. Can you run this test app created by me and Jiewen to ensure it meets the Windows requirements? Part of the test needed an AARCH64 implementation which I just added -

[edk2-devel] [PATCH v4 00/11] ArmPkg: implement EFI memory attributes protocol

2023-02-09 Thread Ard Biesheuvel
v4: - major cleanup of the 32-bit ARM code - add support for EFI_MEMORY_RP using the access flag - enable stack guard in ArmVirtPkg (which uses EFI_MEMORY_RP) - incorporate optimization from other series [0] to avoid splitting block entries unnecessarily v3: - fix ARM32 bug in attribute