diff options
author | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2015-10-02 10:59:13 -0700 |
---|---|---|
committer | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2015-10-07 03:08:58 +0000 |
commit | 72bb66eb9cecf94b66a4aca3586165d5495fcfdb (patch) | |
tree | 181f29ae1450f1a3596c4ba70c40688e35671137 /src/cpu | |
parent | 3f4a997529dfedaa4eefd00dc7d5ab32aa8e7b77 (diff) |
x86/bootblock: Use LDFLAGS_bootblock to enable garbage collection
The x86 bootblock linking is a mess. The bootblock is treated in
a very special manner, and never received the update to link-time
garbage collection.
On newer x86 platforms, the boot media is no longer memory-mapped.
That means we need to do a lot more setup in the bootblock. ROMCC is
unsuitable for this task, and walkcbfs only works on memory-mapped
CBFS. We need to revise the x86 bootflow for this new case.
The approach this patch series takes is to perform CAR setup in the
bootblock, and load the following stage (either romstage or verstage)
from the boot media. This approach is not new, but has been done on
our ARM ports for years.
Since we will be adding .c files to the bootblock, it is prudent to
use link-time garbage collection. This is also consistent to how we
do things on other architectures. Unification FTW!
Change-Id: I16b78456df56e0053984a9aca9367e2542adfdc9
Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Reviewed-on: http://review.coreboot.org/11781
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/cpu')
-rw-r--r-- | src/cpu/dmp/vortex86ex/biosdata.ld | 22 | ||||
-rw-r--r-- | src/cpu/dmp/vortex86ex/biosdata_ex.ld | 6 | ||||
-rw-r--r-- | src/cpu/intel/fit/fit.ld | 2 | ||||
-rw-r--r-- | src/cpu/x86/16bit/reset16.ld | 2 |
4 files changed, 16 insertions, 16 deletions
diff --git a/src/cpu/dmp/vortex86ex/biosdata.ld b/src/cpu/dmp/vortex86ex/biosdata.ld index fd2bb339e5..c1f50c4637 100644 --- a/src/cpu/dmp/vortex86ex/biosdata.ld +++ b/src/cpu/dmp/vortex86ex/biosdata.ld @@ -29,56 +29,56 @@ SECTIONS { . = 0xffffbc00; .dmp_reserved (.): { - *(.dmp_reserved) + KEEP(*(.dmp_reserved)) } . = 0xffffc000; .dmp_kbd_fw_part2 (.): { - *(.dmp_kbd_fw_part2) + KEEP(*(.dmp_kbd_fw_part2)) } . = 0xffffd000; .dmp_mtbf_low_cnt (.): { - *(.dmp_mtbf_low_cnt) + KEEP(*(.dmp_mtbf_low_cnt)) } . = 0xffffe000; .dmp_kbd_fw_part1 (.): { - *(.dmp_kbd_fw_part1) + KEEP(*(.dmp_kbd_fw_part1)) } . = 0xfffff000; .dmp_spi_flash_disk_driver (.): { - *(.dmp_spi_flash_disk_driver) + KEEP(*(.dmp_spi_flash_disk_driver)) } . = 0xfffff800; .dmp_frontdoor (.): { - *(.dmp_frontdoor) + KEEP(*(.dmp_frontdoor)) } . = 0xfffffe00; .dmp_isoinfo (.): { - *(.dmp_isoinfo) + KEEP(*(.dmp_isoinfo)) } . = 0xffffffa0; .dmp_isodata_checksum (.): { - *(.dmp_isodata_checksum) + KEEP(*(.dmp_isodata_checksum)) } . = 0xffffffb0; .dmp_mac (.): { - *(.dmp_mac) + KEEP(*(.dmp_mac)) } . = 0xffffffb8; .dmp_mtbf_limit (.): { - *(.dmp_mtbf_limit) + KEEP(*(.dmp_mtbf_limit)) } . = 0xffffffc0; .dmp_isodata (.): { - *(.dmp_isodata) + KEEP(*(.dmp_isodata)) } } diff --git a/src/cpu/dmp/vortex86ex/biosdata_ex.ld b/src/cpu/dmp/vortex86ex/biosdata_ex.ld index 4d8441c748..8930d80240 100644 --- a/src/cpu/dmp/vortex86ex/biosdata_ex.ld +++ b/src/cpu/dmp/vortex86ex/biosdata_ex.ld @@ -20,16 +20,16 @@ SECTIONS { . = 0xfffffd00; .a9123_crossbar_config (.): { - *(.a9123_crossbar_config) + KEEP(*(.a9123_crossbar_config)) } . = 0xffffffb6; .a9123_strap_1 (.): { - *(.a9123_strap_1) + KEEP(*(.a9123_strap_1)) } . = 0xffffffbb; .a9123_strap_2 (.): { - *(.a9123_strap_2) + KEEP(*(.a9123_strap_2)) } } diff --git a/src/cpu/intel/fit/fit.ld b/src/cpu/intel/fit/fit.ld index 9ccfe82c5f..5817e1ded2 100644 --- a/src/cpu/intel/fit/fit.ld +++ b/src/cpu/intel/fit/fit.ld @@ -1,6 +1,6 @@ SECTIONS { . = 0xffffffc0; .fit_pointer (.): { - *(.fit_pointer) + KEEP(*(.fit_pointer)) } } diff --git a/src/cpu/x86/16bit/reset16.ld b/src/cpu/x86/16bit/reset16.ld index a31a580d12..1730992434 100644 --- a/src/cpu/x86/16bit/reset16.ld +++ b/src/cpu/x86/16bit/reset16.ld @@ -9,7 +9,7 @@ SECTIONS { _ROMTOP = 0xfffffff0; . = _ROMTOP; .reset . : { - *(.reset) + *(.reset); . = 15 ; BYTE(0x00); } |