[U-Boot] [u-boot PATCH] arm, lib/bootm.c: Exclude HIGHMEM from being used by u-boot

[Albert ARIBAUD <albert.u.boot@aribaud.net>]

Hi David,

Le 19/08/2011 19:57, David Long a ?crit :
>
> Reserve any memory above 768MB to prevent u-boot from relocating fdt or initrd
> data into memory that Linux cannot reference during early boot. Code taken (with
> modifications) from the powerpc bootm.c.

What exactly prevents ARM Linux from booting when FDT or initrd are 
above 768MB? Can this limitation not be lifter on the Linux side?

Also:

> Signed-off-by: David A. Long<dave.long@linaro.org>
> ---
>   arch/arm/lib/bootm.c |   24 +++++++++++++++++++++++-
>   1 files changed, 23 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c
> index 802e833..437ef35 100644
> --- a/arch/arm/lib/bootm.c
> +++ b/arch/arm/lib/bootm.c
> @@ -32,6 +32,10 @@
>
>   DECLARE_GLOBAL_DATA_PTR;
>
> +#ifndef CONFIG_SYS_LINUX_LOWMEM_MAX_SIZE
> +#define CONFIG_SYS_LINUX_LOWMEM_MAX_SIZE	(768*1024*1024)
> +#endif

I'd tend to think that if no max size was defined, then we should not limit.

> +
>   #if defined (CONFIG_SETUP_MEMORY_TAGS) || \
>       defined (CONFIG_CMDLINE_TAG) || \
>       defined (CONFIG_INITRD_TAG) || \
> @@ -60,7 +64,25 @@ static int bootm_linux_fdt(int machid, bootm_headers_t *images);
>
>   void arch_lmb_reserve(struct lmb *lmb)
>   {
> -	ulong sp;
> +	phys_size_t bootm_size;
> +	ulong size, sp, bootmap_base;
> +
> +	bootmap_base = getenv_bootm_low();
> +	bootm_size = getenv_bootm_size();
> +
> +#ifdef DEBUG
> +	if (((u64)bootmap_base + bootm_size)>
> +	    (CONFIG_SYS_SDRAM_BASE + (u64) gd->ram_size))
> +		puts("WARNING: bootm_low + bootm_size exceed total memory\n");
> +#endif
> +
> +	size = min(bootm_size, CONFIG_SYS_LINUX_LOWMEM_MAX_SIZE);
> +
> +	if (size<  bootm_size) {
> +		ulong base = bootmap_base + size;
> +		printf("WARNING: adjusting available memory to %lx\n", size);
> +		lmb_reserve(lmb, base, bootm_size - size);
> +	}
>
>   	/*
>   	 * Booting a (Linux) kernel image

Amicalement,
-- 
Albert.