It means that the patch is fine but something else stops it from being
committed. In this case I wanted to go through the whole stack first

Le Mon, Aug 7, 2017 à 6:19 PM, Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
a écrit :

> On Mon, Aug 07, 2017 at 02:11:16PM +0000, Vladimir 'phcoder' Serbinenko
> wrote:
> > LGTM
>
> Is LGTM equivalant to Acked-by?
>
> >
> > Le Thu, Aug 3, 2017 à 12:09 PM, Leif Lindholm <leif.lindh...@linaro.org>
> a
> > écrit :
> >
> > > 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 <leif.lindh...@linaro.org>
> > > ---
> > >  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 c8fffe902..f1424f2e4 100644
> > > --- a/grub-core/kern/efi/mm.c
> > > +++ b/grub-core/kern/efi/mm.c
> > > @@ -448,7 +448,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);
> > >        if (! addr)
> > >         grub_fatal ("cannot allocate conventional memory %p with %u
> pages",
> > >                     (void *) ((grub_addr_t) start),
> > > --
> > > 2.11.0
> > >
> > >
> > > _______________________________________________
> > > Grub-devel mailing list
> > > Grub-devel@gnu.org
> > > https://lists.gnu.org/mailman/listinfo/grub-devel
> > >
>
> > _______________________________________________
> > Grub-devel mailing list
> > Grub-devel@gnu.org
> > https://lists.gnu.org/mailman/listinfo/grub-devel
>
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
>
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to