lkml.org 
[lkml]   [2017]   [Sep]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH v4 0/3] ACPI / EC: Fix EC event handling issues
Date
Hi, Rafael

I'm now working on v5 of this series.
Which
1. splits root causes in a more detailed way,
2. clarifies root causes in patch description with real bugs, and
3. is safer according to the known EC FW behaviors.
So you can discard v3/v4 from the patchwork site.
And I'll post v5 when everything is cleared to me.

Thanks and best regards
Lv

> From: Zheng, Lv
> Subject: [PATCH v4 0/3] ACPI / EC: Fix EC event handling issues
>
> EC events are special, required to be handled during suspend/resume. But
> there are special logics in Linux causing several issues related to the EC
> event handling:
> 1. During noirq stage, Linux cannot detect EC events which are target
> driven.
> 2. When EC event handling is enabled later than the other drivers, order
> problem could be observed.
> When fixing these problems, care should be taken to not to trigger
> regressions to the following problem which has alredy been fixed using
> different approach):
> 3. When EC event handling is enabled before the end of the noirq stage,
> EC event handling may stuck.
> This patchset fixes these issues.
>
> v4 of this patch series re-orders the fixes so that the fix of the problem
> 2 could be independent against the fix of the problem 1, this is done by
> refining the fix of the problem 2, making it immune to the problem 3.
>
> Lv Zheng (3):
> ACPI / EC: Fix possible driver order issue by moving EC event handling
> earlier
> ACPI / EC: Add event detection support for noirq stages
> ACPI / EC: Enable noirq stage event detection
>
> drivers/acpi/ec.c | 116 +++++++++++++++++++++++++++++++++++++++++++++---
> drivers/acpi/internal.h | 1 +
> 2 files changed, 111 insertions(+), 6 deletions(-)
>
> --
> 2.7.4

\
 
 \ /
  Last update: 2017-09-26 09:55    [W:0.284 / U:5.516 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site