[coreboot-gerrit] Patch merged into coreboot/master: soc/intel/common: fix gsmi handler

gerrit at coreboot.org gerrit at coreboot.org
Mon Aug 8 18:39:07 CEST 2016


the following patch was just integrated into master:
commit 2d45c28675e1196ad03915ffba58b602363a19ed
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Fri Aug 5 21:26:53 2016 -0500

    soc/intel/common: fix gsmi handler
    
    The gsmi_exec() expects the parameter to be a pointer
    to the 32-bit register storage of the SMI save state.
    The previous code was passing a pointer with the value
    obtained from the saved-state -- not a pointer to the
    storage of the register value. This bug causes gsmi
    to not log events because it's interrogating the
    parameter buffer itself as if it were a pointer.
    
    BUG=chrome-os-partner:55932
    
    Change-Id: I37981424f1414edad1456b31cad1b99020d57db6
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: https://review.coreboot.org/16087
    Tested-by: build bot (Jenkins)
    Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-by: Furquan Shaikh <furquan at google.com>


See https://review.coreboot.org/16087 for details.

-gerrit



More information about the coreboot-gerrit mailing list