Re: [PATCH] Increase number of e820 entries hard limit from 32 to 128

From: Andi Kleen
Date: Sat Apr 23 2005 - 10:12:27 EST


On Fri, Apr 22, 2005 at 07:32:50PM -0700, Venkatesh Pallipadi wrote:
> On Fri, Apr 22, 2005 at 06:51:59PM -0700, Linus Torvalds wrote:
> > On Fri, 22 Apr 2005, Venkatesh Pallipadi wrote:
> > > The specifications that talk about E820 map doesn't have an upper limit
> > > on the number of E820 entries. But, today's kernel has a hard limit of 32.
> > > With increase in memory size, we are seeing the number of E820 entries
> > > reaching close to 32. Patch below bumps the number upto 128.
> >
> > Hmm. Anything that changes setup.S tends to have bootloader dependencies.
> > I worry whether this one does too..
> >
>
> The setup.S change in this patch should be OK. As it is adding to the
> existing zero-page and keeping it within one page. I tested it on systems
> with grub, adding some dummy E820 entries and it worked fine.

The last time I tried to extend the zero page (with a longer command line)
it broke lilo on systems with EDID support and CONFIG_EDID enabled.
Make sure you test that case.

-Andi

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/