Attention is currently required from: Gaggery Tsai, Lance Zhao, Paul Menzel, Subrata Banik, Tim Wawrzynczak.
Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/77253?usp=email )
Change subject: src/acpi: Add BDAT support in ACPI ......................................................................
Patch Set 3:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/77253/comment/5998dd25_d7e1b103 : PS2, Line 10:
Please elaborate what that table is.
The BDAT table looks like SMBIOS taken several levels further.
Here's a link to the spec: https://uefi.org/sites/default/files/resources/BDAT%20Specification%20v4.0%2...
I looked at that spec, saw the information being passed, and my next thought was "okay, Why?". Here's what I found from an OCP summit presentation titled "Open Firmware DRAM Training Data Interface".
``` As we can see there is no standard way to expose the critical DRAM Training data to OS and higher software layer Why we need it to expose it higher layer? ■ It helps in Data center operations where the holistic view provides more valuable data and better decisions can be made. ● (like) “If you want to run something reliable with predictable performance, avoid that system in that corner” ● (like) “Change the DIMM #8 in that corner system, it might fail - I saw more frequent errors over there” ```