[PATCH] ARM: put memblock stuff into init sections

Russell King - ARM Linux linux at arm.linux.org.uk
Thu Dec 1 18:55:52 EST 2011


On Thu, Dec 01, 2011 at 11:24:50PM +0100, Uwe Kleine-König wrote:
> memblock is only used during early boot for ARM, so the function and
> data can safely go into the init sections.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>

NAK, for several reasons.  Tejun has a patch whose description starts
with this:

| 24aa07882b (memblock, x86: Replace memblock_x86_reserve/free_range()
| with generic ones) removed arch/x86/include/asm/memblock.h and dropped
| its inclusion from include/linux/memblock.h which breaks other
| architectures which depended on the generic memblock.h pulling in the
| arch specific one.

so, this definition in our asm/memblock.h won't have any effect for much
longer.

Second reason: defining this discards the memblock arrays.  Now look at
this:

#ifdef CONFIG_HAVE_ARCH_PFN_VALID
int pfn_valid(unsigned long pfn)
{
        return memblock_is_memory(__pfn_to_phys(pfn));
}
EXPORT_SYMBOL(pfn_valid);
#endif

Third reason: my comment about memblock only being used during early boot
was made because that's the case for using it to do _allocations_, because
once memblock has passed the memory and reserved areas to bootmem, using
memblock _from that point_ means that bootmem will not be updated - and
that in turn means that the kernel's page allocator will also not be
updated correctly.

During boot, memory information is passed thusly:

atags or dt -> memblock -> bootmem -> page allocator

Ideally, we should get rid of bootmem from that sequence, which will make
things a little less complicated - but I believe this is not possible at
the moment due to our use of sparsemem, and iirc sparsemem depends on
certain aspects of bootmem.



More information about the linux-arm-kernel mailing list