[coreboot-gerrit] Change in coreboot[master]: src/intel/xeon_sp: add hardware error support (HEST)

[Arthur Heymans (Code Review)]

Attention is currently required from: Marc Jones, Anjaneya "Reddy" Chagam, Jonathan Zhang, Johnny Lin, Rocky Phagura, Tim Wawrzynczak, David Hendricks, Angel Pons, Morgan Jang, Patrick Rudolph, Tim Chu.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52090 )

Change subject: src/intel/xeon_sp: add hardware error support (HEST)
......................................................................

Patch Set 19:

(1 comment)

File src/soc/intel/xeon_sp/uncore.c:

https://review.coreboot.org/c/coreboot/+/52090/comment/384e9134_775be2b4 
PS18, Line 149: * +--------------------------+ cbmem_top
              :  * |     HEST ELOG            | (CONFIG_ERROR_LOG_BUFFER_SIZE)
              :  * +--------------------------+
> > Done...corrected the map so that HEST ELOG is above cbmem_top.
> > Yes, it cannot be part of CBMEM because in the E820 table, CBMEM is labeled as type16 memory (Unknown to the OS) and therefore ACPI drivers fail to acknowledge this memory and generate ACPI errors, unable to load HEST.
>
> type16? That's probably an error in the Linux trampoline then? It should be type 2. I'll try to figure out what is going on there.

Ok https://review.coreboot.org/c/coreboot/+/55069 fixes this.