GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/xeon_sp: Fix HEST table length 

"current" points to the start of HEST table, so "next - current" already
includes the size of its header, no need for increment here. This issue
was found on SPR-SP platform. The length of HEST table is now correct
with this patch.

Change-Id: I6ff1e8e24612b7356772d582ff9a7e53863419db
Signed-off-by: Jeff Li <lijinfeng01@inspur.com>
Signed-off-by: Ziang Wang <ziang.wang@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/75738
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
This commit is contained in:
Jeff Li 2023-06-09 15:49:12 +08:00 committed by Lean Sheng Tan
parent 7fb661fa8a
commit 575eb73951
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/soc/intel/xeon_sp/ras/hest.c
View File

@ -65,7 +65,7 @@ static unsigned long acpi_fill_hest(acpi_hest_t *hest)
next = hest + 1; next = hest + 1;
next += acpi_hest_add_ghes(next); next += acpi_hest_add_ghes(next);
hest->error_source_count += 1; hest->error_source_count += 1;
header->length += next - current; header->length = next - current;
return header->length; return header->length;
} }