Re: [PATCH] MIPS: DEC: Restore bootmem reservation for firmware working memory area
From: Maciej W. Rozycki
Date: Wed Oct 14 2020 - 16:54:04 EST
Hello Serge,
> > @@ -146,6 +150,9 @@ void __init plat_mem_setup(void)
> >
> > ioport_resource.start = ~0UL;
> > ioport_resource.end = 0UL;
> > +
> > + /* Stay away from the firmware working memory area for now. */
>
> > + memblock_reserve(PHYS_OFFSET, __pa_symbol(&_text));
>
> Shouldn't that be:
> + memblock_reserve(PHYS_OFFSET, __pa_symbol(&_text) - PHYS_OFFSET);
> instead?
Good point: unlike `free_init_pages' which uses start/end this function
uses start/size as its arguments. For DEC effectively it does not matter
as PHYS_OFFSET is 0 anyway, but code has to be semantically correct.
I'll post an update.
Maciej