summaryrefslogtreecommitdiffstats
path: root/src/drivers/mrc_cache/mrc_cache.c
diff options
context:
space:
mode:
authorEddie Vas <aeddiesharma@fb.com>2022-08-16 20:12:04 -0700
committerFelix Held <felix-coreboot@felixheld.de>2022-09-24 01:19:13 +0000
commit1df1cf994aa9ebdd9d51103d2d618f6aa66df2d6 (patch)
tree4b7a26680c02f136205b2cca3b80a5c45268aaac /src/drivers/mrc_cache/mrc_cache.c
parent1f5c5da812d5d71e9ee4f1b3ba47e5d323a8c05d (diff)
downloadcoreboot-1df1cf994aa9ebdd9d51103d2d618f6aa66df2d6.tar.gz
coreboot-1df1cf994aa9ebdd9d51103d2d618f6aa66df2d6.tar.bz2
coreboot-1df1cf994aa9ebdd9d51103d2d618f6aa66df2d6.zip
commonlib/fsp_relocate: add PE32 section support
Recently published Intel CedarIslandFSP binary contains PE images in FSP-M and FSP-S. This causes coreboot boot hang on DeltaLake servers. PI spec PI_Spec_1_7_final_Jan_2019 on uefi.org talks about FV files requiring to support SECTION_PE32 sections and FSP specification states that FSP images are created in alignment with PI specification. FSP images are relocated at build time and run time using the func fsp_component_relocate. That code only supported TE image relocation so far. The change required to add support for pe_relocate in fsp-relocate.c I had to move a few functions to top of file as they need to be used by pe_relocate but were placed below the te_relocate function. I chose to place pe_relocate function next to te_relocate. The code supports PE32 format, not PE32+, at this time. Links for PE and FSP specs are provided below for reference. Link= https://www.intel.com/content/dam/www/public/us/en/documents/technical-specifications/fsp-architecture-spec-v2.pdf Link= https://uefi.org/sites/default/files/resources/PI_Spec_1_7_final_Jan_2019.pdf TESTED= This code is tested with FSP version 33A for DeltaLake boot which has FSP-M and FSP-S as PE32 sections. This FSP version does not boot on DeltaLake without this change. Change-Id: I01e2c123d74f735a647b994aa66419c9796f193e Signed-off-by: Eddie Sharma <aeddiesharma@fb.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/66819 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Nathaniel L Desimone <nathaniel.l.desimone@intel.com>
Diffstat (limited to 'src/drivers/mrc_cache/mrc_cache.c')
0 files changed, 0 insertions, 0 deletions