Re: [PATCH] arm/efi: fix ram base detection

[Daniel Kiper]

On Tue, Mar 30, 2021 at 10:01:23PM +0200, Vincent Stehlé via Grub-devel wrote:
> On Mon, Mar 22, 2021 at 06:28:51PM +0000, Leif Lindholm wrote:
> ..
> > > > diff --git a/grub-core/kern/efi/mm.c b/grub-core/kern/efi/mm.c
> > > > index 0cdb063bb..abf8772bc 100644
> > > > --- a/grub-core/kern/efi/mm.c
> > > > +++ b/grub-core/kern/efi/mm.c
> > > > @@ -677,7 +677,8 @@ grub_efi_get_ram_base(grub_addr_t *base_addr)
> > > >    for (desc = memory_map, *base_addr = GRUB_EFI_MAX_USABLE_ADDRESS;
> > > >         (grub_addr_t) desc < ((grub_addr_t) memory_map + 
> > > > memory_map_size);
> > > >         desc = NEXT_MEMORY_DESCRIPTOR (desc, desc_size))
> > > > -    if (desc->attribute & GRUB_EFI_MEMORY_WB)
> > > > +    if (desc->type == GRUB_EFI_CONVENTIONAL_MEMORY &&
> > > > +        desc->attribute & GRUB_EFI_MEMORY_WB)
> >
> > Can we safely assume we don't also need to check against
> > GRUB_EFI_PERSISTENT_MEMORY? If so, this is fine.
>
> Hi Leif,
>
> Thanks for the review.
>
> This is a good question about persistent memory; I don't know if we should
> check it or not.
>
> I am "fighting" with qemu to add an nvdimm above or below the first normal
> memory region to see how this behaves. I will let you know when I have
> succeeded.

Any updates about this?

Daniel