On Thu, Jul 27, 2017, 17:30 Daniel Kiper <[email protected]> wrote:
> On Mon, Jun 12, 2017 at 03:53:41PM +0100, Leif Lindholm wrote: > > With upcoming changes to EDK2, allocations of type EFI_LOADER_DATA may > > not return regions with execute ability. Since modules are loaded onto > > the heap, change the heap allocation type to GRUB_EFI_LOADER_CODE in > > order to permit execution on systems with this feature enabled. > > > > Closes: 50420 > > > > Signed-off-by: Leif Lindholm <[email protected]> > > --- > > grub-core/kern/efi/mm.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/grub-core/kern/efi/mm.c b/grub-core/kern/efi/mm.c > > index 7b1763bc5..f27a48e68 100644 > > --- a/grub-core/kern/efi/mm.c > > +++ b/grub-core/kern/efi/mm.c > > @@ -404,7 +404,9 @@ add_memory_regions (grub_efi_memory_descriptor_t > *memory_map, > > pages = required_pages; > > } > > > > - addr = grub_efi_allocate_pages (start, pages); > > + addr = grub_efi_allocate_pages_real (start, pages, > > + GRUB_EFI_ALLOCATE_ADDRESS, > > + GRUB_EFI_LOADER_CODE); > > Is it really needed? Is any module exectued in place or does it contain > code ready to run out of the box? > All the modules are loaded to heap > > Daniel > > _______________________________________________ > Grub-devel mailing list > [email protected] > https://lists.gnu.org/mailman/listinfo/grub-devel >
_______________________________________________ Grub-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/grub-devel
