On Thu, Jul 07, 2016 at 12:37:10PM +0800, Lv Zheng wrote:
> (Updated to add v2 indicator)
>
> There is an order issue in ec_remove_handlers() that the functions invoked
> in it are not invoked in the reversed order of their appearance in
> ec_install_handlers(). This existing issue has been triggered by the
> following commit:
> Commit: dcf15cbded656a12335bc4151f3f75f10080a375
> Subject: ACPI / EC: Fix a boot EC regresion by restoring boot EC
> The commit invokes ec_remove_handlers() during runtime, thus uncovers this
> issue. This patch fixes this regression.
>
> Fixes: dcf15cbded65 ("ACPI / EC: Fix a boot EC regresion by restoring boot EC")
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=102421
> Reported-by: Wolfram Sang <wsa@xxxxxxxxxxxxx>
> Reported-by: Nicholas <nkudriavtsev@xxxxxxxxx>
> Cc: Wolfram Sang <wsa@xxxxxxxxxxxxx>
> Cc: Nicholas <nkudriavtsev@xxxxxxxxx>
> Signed-off-by: Lv Zheng <lv.zheng@xxxxxxxxx>
Tested-by: Wolfram Sang <wsa@xxxxxxxxxxxxx>
The status infos are back to normal and the EC error reports are gone
from the log. Looks good! Thanks for the quick fix.
Wolfram
Attachment:
signature.asc
Description: PGP signature
