From 83bd46e5e5df0176af1208c7feec98d64273875b Mon Sep 17 00:00:00 2001 From: "Ronald G. Minnich" Date: Sun, 16 Sep 2018 09:59:54 -0700 Subject: selfboot: remove bounce buffers Bounce buffers used to be used in those cases where the payload might overlap coreboot. Bounce buffers are a problem for rampayloads as they need malloc. They are also an artifact of our x86 past before we had relocatable ramstage; only x86, out of the 5 architectures we support, needs them; currently they only seem to matter on the following chipsets: src/northbridge/amd/amdfam10/Kconfig src/northbridge/amd/lx/Kconfig src/northbridge/via/vx900/Kconfig src/soc/intel/fsp_baytrail/Kconfig src/soc/intel/fsp_broadwell_de/Kconfig The first three are obsolete or at least could be changed to avoid the need to have bounce buffers. The last two should change to no longer need them. In any event they can be fixed or pegged to a release which supports them. For these five chipsets we change CONFIG_RAMBASE from 0x100000 (the value needed in 1999 for the 32-bit Linux kernel, the original ramstage) to 0xe00000 (14 Mib) which will put the non-relocatable x86 ramstage out of the way of any reasonable payload until we can get rid of it for good. 14 MiB was chosen after some discussion, but it does fit well: o Fits in the 16 MiB cacheable range coreboot sets up by default o Most small payloads are well under 14 MiB (even kernels!) o Most large payloads get loaded at 16 MiB (especially kernels!) With this change in place coreboot correctly still loads a bzImage payload. Werner reports that the 0xe00000 setting works on his broadwell systems. Change-Id: I602feb32f35e8af1d0dc4ea9f25464872c9b824c Signed-off-by: Ronald G. Minnich Reviewed-on: https://review.coreboot.org/28647 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- src/arch/arm64/boot.c | 5 ----- 1 file changed, 5 deletions(-) (limited to 'src/arch/arm64') diff --git a/src/arch/arm64/boot.c b/src/arch/arm64/boot.c index 2357912eae..d8a4630750 100644 --- a/src/arch/arm64/boot.c +++ b/src/arch/arm64/boot.c @@ -55,11 +55,6 @@ void arch_prog_run(struct prog *prog) doit(prog_entry_arg(prog)); } -int arch_supports_bounce_buffer(void) -{ - return 0; -} - /* Generic stage entry point. Can be overridden by board/SoC if needed. */ __weak void stage_entry(void) { -- cgit v1.2.3