linux-kernel - Re: [PATCH] x86 boot: Pass E820 memory map entries more than 128 via linked list of setup data

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 23 Jun 2008 13:54:01 +0800
From:	"Huang, Ying" <ying.huang@...el.com>
To:	Paul Jackson <pj@....com>
Cc:	Ingo Molnar <mingo@...e.hu>, "H. Peter Anvin" <hpa@...or.com>,
	andi@...stfloor.org, mingo@...hat.com, tglx@...utronix.de,
	linux-kernel@...r.kernel.org, Yinghai Lu <yhlu.kernel@...il.com>
Subject: Re: [PATCH] x86 boot: Pass E820 memory map entries more than 128
	via linked list of setup data

Hi, Paul,

This patch can be used for machine whose E820 memory map has more than
128 entries. So it is not necessary to use EFI memory map for this (EFI
memory map is used for EFI runtime services only).

With this patch, your previous patch:

x86 boot: add code to add BIOS provided EFI memory entries to kernel

is not necessary, so can be reversed.

What do you think about that?

Best Regards,
Huang Ying

On Wed, 2008-06-18 at 13:45 +0200, Ingo Molnar wrote:
> * Huang, Ying <ying.huang@...el.com> wrote:
> 
> > Because of the size limits of struct boot_params (zero page), the 
> > maximum number of E820 memory map entries can be passed to kernel is 
> > 128. As pointed by Paul Jackson, there is some machine produced by SGI 
> > with so many nodes that the number of E820 memory map entries is more 
> > than 128. To enabling Linux kernel on these system, a new setup data 
> > type named SETUP_E820_EXT is defined to pass additional memory map 
> > entries to Linux kernel.
> > 
> > This patch is based on x86/auto-latest branch of git-x86 tree and has 
> > been tested on x86_64 and i386 platform.
> 
> applied to tip/x86/mpparse, thanks.
> 
> this bit collided with a recent change:
> 
> > -	do {
> > -		u64 start = biosmap->addr;
> > -		u64 size = biosmap->size;
> > -		u64 end = start + size;
> > -		u32 type = biosmap->type;
> > -
> > -		/* Overflow in 64 bits? Ignore the memory map. */
> > -		if (start > end)
> > -			return -1;
> > -
> > -		add_memory_region(start, size, type);
> > -	} while (biosmap++, --nr_map);
> > -	return 0;
> > +	return __copy_e820_map(biosmap, nr_map);
> 
> because add_memory_region() got renamed to e820_add_region(). I did the 
> obvious fixup.
> 
> 	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/