x86: Allow 1MB of slack between the e820 map and SRAT, not 4GB
authorYinghai Lu <yinghai@kernel.org>
Fri, 8 May 2009 07:37:34 +0000 (00:37 -0700)
committerIngo Molnar <mingo@elte.hu>
Mon, 11 May 2009 09:38:21 +0000 (11:38 +0200)
commit0964b0562bb9c93194e852b47bab2397b9e11c18
tree89a82d56a9f262607be41153b2a3da37cf01d37f
parentb37ab91907e9002925f4217e3bbd496aa12c2fa3
x86: Allow 1MB of slack between the e820 map and SRAT, not 4GB

It is expected that there might be slight differences between the e820
map and the SRAT table and the intention was that 1MB of slack be allowed.

The calculation comparing e820ram and pxmram assumes the units are bytes,
when they are in fact pages. This means 4GB of slack is being allowed,
not 1MB. This patch makes the correct comparison.

comment is from Mel.

[ Impact: don't accept buggy SRATs that could dump up to 4G of RAM ]

Signed-off-by: Yinghai Lu <yinghai@kernel.org>
Acked-by: Mel Gorman <mel@csn.ul.ie>
Cc: Andrew Morton <akpm@linux-foundation.org>
LKML-Reference: <4A03E13E.6050107@kernel.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/mm/srat_64.c