On Mon, May 08, 2023 at 09:34:15AM +0200, Roger Pau Monné wrote:
> On Fri, Apr 28, 2023 at 04:15:24PM +0200, Daniel Kiper wrote:
> > On Thu, Apr 27, 2023 at 05:06:54PM +0200, Roger Pau Monne via Grub-devel
> > wrote:
> > > On failure to allocate from grub_relocator_firmware_alloc_region() in
> > > malloc_in_range() the function would stop enforcing the alignment, and
> > > the following was returned:
> > >
> > > lib/relocator.c:431: trying to allocate in 0x200000-0xffbf9fff aligned
> > > 0x200000 size 0x406000
> > > lib/relocator.c:1197: allocated: 0x74de2000+0x406000
> > > lib/relocator.c:1407: allocated 0x74de2000/0x74de2000
> > >
> > > Fix this by making sure that target always contains a suitably aligned
> > > address. After the change the return from the function is:
> > >
> > > lib/relocator.c:431: trying to allocate in 0x200000-0xffb87fff aligned
> > > 0x200000 size 0x478000
> > > lib/relocator.c:1204: allocated: 0x74c00000+0x478000
> > > lib/relocator.c:1414: allocated 0x74c00000/0x74c00000
> > >
> > > Fixes: 3a5768645c05 ('First version of allocation from firmware')
> > > Signed-off-by: Roger Pau Monné <roger....@citrix.com>
> >
> > LGTM but I would like to hear Vladimir's opinion too.
>
> Thanks Daniel, just a gentle ping to see if we can unblock this.
Sorry for delay but I was on vacation last week. I will check this with
Vladimir and get back to you.
Daniel
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel
