[coreboot] Patch merged into coreboot/master: cb313ee ACPI HEST table.

gerrit at coreboot.org gerrit at coreboot.org
Mon Apr 16 04:56:07 CEST 2012


the following patch was just integrated into master:
commit cb313ee348521ed0a2ee44c9481673d9ca969808
Author: zbao <fishbaozi at gmail.com>
Date:   Fri Apr 13 13:57:14 2012 +0800

    ACPI HEST table.
    
    HEST feature starts from ACPI 4.0.
    
    HEST is one of four kinds of tables of ACPI Platform Error
    Interfaces (APEI). In Windows world, APEI is called Windows Hardware
    Error Architecture (WHEA).
    
    APEI consists of four separate tables:
    1. Error Record Serialization Table (ERST)
    2. BOOT Error Record Table (BERT)
    3. Hardware Error Source Table (HEST)
    4. Error Injection Table (EINJ)
    All these 4 tables have the same header as FADT, MADT, etc. They are
    pointed by RSDP.
    
    For the HEST, it contains the error source. The types of them are
    defined as
    type description
    1. Machine Check Exception (MCE)
    2. Corrected Machine Check (CMC)
    3. NMI Error
    6. PCI Express Root Port AER
    7. PCI Express Device AER
    8. PCI Express Bridge AER
    9. Generic Hardware Error Source
    Error source types 3, 4, and 5 are reserved for legacy reasons and
    must not be used.
    
    Currently AMD board only provide part of "Machine Check
    Exception (MCE)" & Corrected Machine Check (CMC)". we need to provide
    the header of each error source. Other types of Error Sources is in
    TODO list.
    
    Only persimmon is tested. Linux can add HEST feature. The dmesg says,
    
    ACPI: HEST 0000000066fe5010 00198 (v03 CORE COREBOOT 00000000 CORE 00000000)
    ......
    HEST: Table parsing has been initialized.
    
    No more message is got.
    
    Windows can boot with this patch. Havent found a way to test it.
    
    Change-Id: I447e7f57b8e8f0433a145a43d0710910afabf00f
    Signed-off-by: Zheng Bao <zheng.bao at amd.com>
    Signed-off-by: zbao <fishbaozi at gmail.com>

Build-Tested: build bot (Jenkins) at Mon Apr 16 04:54:34 2012, giving +1
Reviewed-By: Peter Stuge <peter at stuge.se> at Mon Apr 16 04:47:44 2012, giving +2
See http://review.coreboot.org/888 for details.

-gerrit




More information about the coreboot mailing list