Li Jinfeng has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69500 )
Change subject: src/soc/intel/xeon_sp/ras/hest.c: fix HEST table length ......................................................................
src/soc/intel/xeon_sp/ras/hest.c: fix HEST table length
`current' is the very begining of table, so `next' - `current' is the total legnth of the table. header->length is not zero but initialized with the length of header.
Change-Id: I4780785bac782aa966def3f042f365a1ead9d9d8 Signed-off-by: Li Jinfeng lijinfeng01@inspur.com --- M src/soc/intel/xeon_sp/ras/hest.c 1 file changed, 15 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/69500/1
diff --git a/src/soc/intel/xeon_sp/ras/hest.c b/src/soc/intel/xeon_sp/ras/hest.c index 9eb54d9..2da720a 100644 --- a/src/soc/intel/xeon_sp/ras/hest.c +++ b/src/soc/intel/xeon_sp/ras/hest.c @@ -65,7 +65,7 @@ next = hest + 1; next += acpi_hest_add_ghes(next); hest->error_source_count += 1; - header->length += next - current; + header->length = next - current; return header->length; }