Re: [PATCH v2] mm: Fix memblock_free_late() when using deferred struct page

[Benjamin Herrenschmidt <benh@kernel.crashing.org>]

> > +late_initcall(efi_free_boot_services_memory);

Why late btw ? Any particular reason ?

One very minor nit (but it kind of is annoying when you gather logs at
scale and some people do look at this :-) ) is that the memory isn't
accounted in the boot message:

Memory: 224440K/483372K available (16384K kernel code, 9440K rwdata,
11344K rodata, 3732K init, 6480K bss, 254088K reserved, 0K cma-
reserved)

I'm not going to cry about this, but it might be nice to have the
__initcall display how much extra if freed so it's just a log grep
away.

Cheers,
Ben.


> >  /*
> >   * A number of config table entries get remapped to virtual
> > addresses
> >   * after entering EFI virtual mode. However, the kexec kernel
> > requires
> > 
> > base-commit: 05f7e89ab9731565d8a62e3b5d1ec206485eeb0b
> > -- 
> > 2.51.0
> > 
> >  
> > > Cheers,
> > > Ben.
> > > 
> > 
>