]> bbs.cooldavid.org Git - net-next-2.6.git/commit
x86: move dma32_reserve_bootmem() after reserve_crashkernel()
authorBernhard Walle <bwalle@suse.de>
Fri, 18 Jul 2008 17:07:53 +0000 (19:07 +0200)
committerIngo Molnar <mingo@elte.hu>
Fri, 18 Jul 2008 21:35:46 +0000 (23:35 +0200)
commit91467bdf6e53058af13fd255375d6634ba0c70e0
tree267f33c8455bb2bd2c2fdbe39259fafd0e1d567b
parent5b664cb235e97afbf34db9c4d77f08ebd725335e
x86: move dma32_reserve_bootmem() after reserve_crashkernel()

On a x86-64 machine (nothing special I could encounter) I had the problem that
crashkernel reservation with the usual "64M@16M" failed. While debugging that,
I encountered that dma32_reserve_bootmem() reserves a memory region which is in
that area.

Because dma32_reserve_bootmem() does not rely on a specific offset but
crashkernel does, it makes sense to move the dma32_reserve_bootmem()
reservation down a bit. I tested that patch and it works without problems. I
don't see any negative effects of that move, but maybe I oversaw something ...

While we strictly don't need that patch in 2.6.27 because we have the
automatic, dynamic offset detection, it makes sense to also include it here
because:

  - it's easier to get it in -stable then,
  - many people are still used to the 'crashkernel=...@16M' syntax,
  - not everybody may be using a reloatable kernel.

Signed-off-by: Bernhard Walle <bwalle@suse.de>
Cc: kexec@lists.infradead.org
Cc: vgoyal@redhat.com
Cc: akpm@linux-foundation.org
Cc: Bernhard Walle <bwalle@suse.de>
Cc: yhlu.kernel@gmail.com
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/x86/kernel/setup.c