UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

ACPI / APEI: clear error status before acknowledging the error 

Currently we acknowledge errors before clearing the error status.
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.

Signed-off-by: Tyler Baicar <tbaicar@codeaurora.org>
Reviewed-by: Borislav Petkov <bp@suse.de>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
Tyler Baicar 2017-08-28 10:53:41 -06:00 committed by Rafael J. Wysocki
parent e931d0dab4
commit aaf2c2fb0f
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
drivers/acpi/apei/ghes.c
View File

@ -743,17 +743,19 @@ static int ghes_proc(struct ghes *ghes)
} }
ghes_do_proc(ghes, ghes->estatus); ghes_do_proc(ghes, ghes->estatus);
out:
ghes_clear_estatus(ghes);
if (rc == -ENOENT)
return rc;
/* /*
* GHESv2 type HEST entries introduce support for error acknowledgment, * GHESv2 type HEST entries introduce support for error acknowledgment,
* so only acknowledge the error if this support is present. * so only acknowledge the error if this support is present.
*/ */
if (is_hest_type_generic_v2(ghes)) { if (is_hest_type_generic_v2(ghes))
rc = ghes_ack_error(ghes->generic_v2); return ghes_ack_error(ghes->generic_v2);
if (rc)
return rc;
}
out:
ghes_clear_estatus(ghes);
return rc; return rc;
} }