Attention is currently required from: Marc Jones, Furquan Shaikh, Duncan Laurie, Angel Pons. Hello build bot (Jenkins), Furquan Shaikh, Patrick Georgi, Jonathan Zhang, Duncan Laurie, Paul Menzel, Rocky Phagura, Subrata Banik, Angel Pons, Patrick Rudolph, Lance Zhao, Marc Jones, Martin Roth, Marshall Dawson,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/49286
to look at the new patch set (#5).
Change subject: src/acpi: Add APEI EINJ support ......................................................................
src/acpi: Add APEI EINJ support
This adds full EINJ support with trigger action tables. The actual error injection functionality is HW specific. Therefore, HW specific code should call acpi_create_einj with an address where action table resides. The default params of the action table are filled out by the common code. Control is then returned back to the caller to modify or override default parameters. If no changes are needed, caller can simply add the acpi table. At runtime, FW is responsible for filling out the action table with the proper entries. The action table memory is shared between FW and OS. This memory should be marked as reserved in E820 table.
Tested on Deltalake mainboard. Boot to OS, load the EINJ driver ( modprobe EINJ) and verify EINJ memory entries are in /proc/iomem. Further tested by injecting errors via the APEI file nodes. More information on error injection is here. https://www.kernel.org/doc/Documentation/acpi/apei/einj.txt
Change-Id: I29c6a861c564ec104f2c097f3e49b3e6d38b040e Signed-off-by: Rocky Phagura rphagura@fb.com --- M src/acpi/Kconfig M src/acpi/acpi.c M src/include/acpi/acpi.h 3 files changed, 290 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/49286/5