On 8/29/2017 2:16 AM, Borislav Petkov wrote:I haven't heard anything on this patch. It would be great to have it in 4.14 :)
On Mon, Aug 28, 2017 at 10:53:41AM -0600, Tyler Baicar wrote:Hello Rafael,
Currently we acknowledge errors before clearing the error status.Reviewed-by: Borislav Petkov <bp@xxxxxxx>
This could cause a new error to be populated by firmware in-between
the error acknowledgment and the error status clearing which would
cause the second error's status to be cleared without being handled.
So, clear the error status before acknowledging the errors.
Also, make sure to acknowledge the error if the error status read
fails.
V3: Seperate check for -ENOENT return value
V2: Only send error ack if there was an error populated
ÂÂÂÂ Remove curly braces that are no longer needed
Signed-off-by: Tyler Baicar <tbaicar@xxxxxxxxxxxxxx>
---
 drivers/acpi/apei/ghes.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)
Were you able to apply this patch?