From 177a402b6eb554cf3ad1dce12bc558582877cba3 Mon Sep 17 00:00:00 2001 From: Nikolai Vyssotski Date: Fri, 4 Jun 2021 10:35:15 -0500 Subject: [PATCH] soc/amd/common/fsp/pci: Add size field to PCIe interrupt routing HOB EDK2 mandates HOB to be in increments of qword (8). This HOB has 13 elements which causes it be padded with 4 bytes of garbage. This results in coreboot failing intermittently with invalid data. Add "number of entries" field to specify the number of valid entries in the table. BUG=b:190153208 Cq-depend: chrome-internal:3889619 TEST=verify HOB is present and correct size (13) is reported Change-Id: Iaafae304f04a5f26d75a41a6d6fcb4ee69954d20 Signed-off-by: Nikolai Vyssotski Reviewed-on: https://review.coreboot.org/c/coreboot/+/55237 Reviewed-by: Raul Rangel Tested-by: build bot (Jenkins) --- src/soc/amd/common/fsp/pci/pci_routing_info.c | 14 +++++++++----- src/vendorcode/amd/fsp/cezanne/FspGuids.h | 4 ++-- 2 files changed, 11 insertions(+), 7 deletions(-) diff --git a/src/soc/amd/common/fsp/pci/pci_routing_info.c b/src/soc/amd/common/fsp/pci/pci_routing_info.c index 21423da210..2ea8dfcdbf 100644 --- a/src/soc/amd/common/fsp/pci/pci_routing_info.c +++ b/src/soc/amd/common/fsp/pci/pci_routing_info.c @@ -11,23 +11,27 @@ const struct pci_routing_info *get_pci_routing_table(size_t *entries) { static const struct pci_routing_info *routing_table; static size_t routing_table_entries; - size_t hob_size = 0; + const struct { + uint32_t num_of_entries; + struct pci_routing_info routing_table[]; + } __packed *routing_hob; if (routing_table) { *entries = routing_table_entries; return routing_table; } - routing_table = fsp_find_extension_hob_by_guid(AMD_FSP_PCIE_DEVFUNC_REMAP_HOB_GUID.b, + routing_hob = fsp_find_extension_hob_by_guid(AMD_FSP_PCIE_DEVFUNC_REMAP_HOB_GUID.b, &hob_size); - if (routing_table == NULL || hob_size == 0) { - printk(BIOS_ERR, "Couldn't find PCIe routing HOB.\n"); + if (routing_hob == NULL || hob_size == 0 || routing_hob->num_of_entries == 0) { + printk(BIOS_ERR, "ERROR: Couldn't find valid PCIe interrupt routing HOB.\n"); return NULL; } - routing_table_entries = hob_size / sizeof(struct pci_routing_info); + routing_table = routing_hob->routing_table; + routing_table_entries = routing_hob->num_of_entries; for (size_t i = 0; i < routing_table_entries; ++i) { printk(BIOS_DEBUG, "%02x.%x: group: %u, swizzle: %u, irq: %u\n", diff --git a/src/vendorcode/amd/fsp/cezanne/FspGuids.h b/src/vendorcode/amd/fsp/cezanne/FspGuids.h index c26daa5a42..0eca78e711 100644 --- a/src/vendorcode/amd/fsp/cezanne/FspGuids.h +++ b/src/vendorcode/amd/fsp/cezanne/FspGuids.h @@ -14,7 +14,7 @@ 0x87, 0xE1, 0x3F, 0xEB, 0x13, 0xC5, 0x66, 0x9A) #define AMD_FSP_PCIE_DEVFUNC_REMAP_HOB_GUID \ - GUID_INIT(0x00D54AA7, 0x0002, 0x47F5, \ - 0x00, 0x78, 0x08, 0x57, 0x00, 0x00, 0xA4, 0x11) + GUID_INIT(0X6D5CD69D, 0XFB24, 0X4461, \ + 0XAA, 0X32, 0X8E, 0XE1, 0XB3, 0X3, 0X31, 0X9C ) #endif /* __FSP_GUIDS__ */