Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-10-05 Thread Gerd Hoffmann
On Thu, Oct 05, 2023 at 10:23:25AM +0200, Laszlo Ersek wrote: > On 10/5/23 08:31, Nhi Pham via groups.io wrote: > > Hi Ard, Oliver, > > > > I'm investigating the crash on grub2/shim loader due to the added > > EFI_MEMORY_ATTRIBUTE_PROTOCOL when rebasing. I found this interesting > > patch and went

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-10-05 Thread Laszlo Ersek
On 10/5/23 08:31, Nhi Pham via groups.io wrote: > Hi Ard, Oliver, > > I'm investigating the crash on grub2/shim loader due to the added > EFI_MEMORY_ATTRIBUTE_PROTOCOL when rebasing. I found this interesting > patch and went through on the discussion, I am still not sure the > conclusion on this p

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-10-04 Thread Nhi Pham via groups.io
Hi Ard, Oliver, I'm investigating the crash on grub2/shim loader due to the added EFI_MEMORY_ATTRIBUTE_PROTOCOL when rebasing. I found this interesting patch and went through on the discussion, I am still not sure the conclusion on this patch. This issue impacts many platforms, and any downs

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-23 Thread Sean
I think that is an interesting idea but I would expect some push back from OS loader maintainers. I would expect they don't want to be constrained by the lowest common capabilities of the platforms they still support/run on in the ecosystem.   Not to mention the challenges around servicing and/

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-23 Thread Ard Biesheuvel
On Tue, 20 Jun 2023 at 19:07, Sean Brogan wrote: > > I don't think a MemoryAttributes2Protocol with a single API would have > avoided the errors. > > The programming pattern that triggered this would still need multiple calls > to any API and in the future where all memory is allocated as NX thi

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-20 Thread Sean
I don't think a MemoryAttributes2Protocol with a single API would have avoided the errors. The programming pattern that triggered this would still need multiple calls to any API and in the future where all memory is allocated as NX this possibility would still exist. A short term effort to m

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-20 Thread Gerd Hoffmann
On Tue, Jun 20, 2023 at 04:16:40PM +0300, Ard Biesheuvel wrote: > On Tue, Jun 20, 2023, 12:33 Gerd Hoffmann wrote: > > > On Mon, Jun 19, 2023 at 10:32:25PM +0200, Oliver Steffen wrote: > > > Recent versions of shim (15.6 and 15.7) crash when the newly added > > > EFI_MEMORY_ATTRIBUTE_PROTOCOL is

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-20 Thread Michael Kubacki
We recently encountered this issue as well. For reference, some details are in this issue: https://github.com/microsoft/mu_silicon_arm_tiano/issues/124 Specifically, this comment: https://github.com/microsoft/mu_silicon_arm_tiano/issues/124#issuecomment-1593550704 On 6/20/2023 9:16 AM, Ard Bies

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-20 Thread Ard Biesheuvel
On Tue, Jun 20, 2023, 12:33 Gerd Hoffmann wrote: > On Mon, Jun 19, 2023 at 10:32:25PM +0200, Oliver Steffen wrote: > > Recent versions of shim (15.6 and 15.7) crash when the newly added > > EFI_MEMORY_ATTRIBUTE_PROTOCOL is provided by the firmware. To allow > > existing installations to boot, pr

Re: [edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-20 Thread Gerd Hoffmann
On Mon, Jun 19, 2023 at 10:32:25PM +0200, Oliver Steffen wrote: > Recent versions of shim (15.6 and 15.7) crash when the newly added > EFI_MEMORY_ATTRIBUTE_PROTOCOL is provided by the firmware. To allow > existing installations to boot, provide a workaround in form of a Pcd > that allows tuning it

[edk2-devel] [PATCH 1/1] ArmPkg: Add Pcd to disable EFI_MEMORY_ATTRIBUTE_PROTOCOL

2023-06-19 Thread Oliver Steffen
Recent versions of shim (15.6 and 15.7) crash when the newly added EFI_MEMORY_ATTRIBUTE_PROTOCOL is provided by the firmware. To allow existing installations to boot, provide a workaround in form of a Pcd that allows tuning it off at build time (defaults to 'enabled'). Additionally, check the ret