RE: [RFC PATCH v3 00/14] ACPI/EC: Add event storm prevention and cleanup command storm prevention.

Zheng, Lv
2014-07-23T01:31:45Z

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 2492 bytes --]

Hi,

> From: Rafael J. Wysocki [mailto:rjw@rjwysocki.net]
> Sent: Wednesday, July 23, 2014 6:15 AM
>
> On Tuesday, July 22, 2014 01:25:00 AM Zheng, Lv wrote:
> > Hi, Rafael
> >
> > > From: Rafael J. Wysocki [mailto:rjw@rjwysocki.net]
> > > Sent: Tuesday, July 22, 2014 9:12 AM
> > >
> > > On Monday, July 21, 2014 02:04:51 PM Lv Zheng wrote:
> > > > Note that this patchset is very stable now, it is sent as RFC because it
> > > > depends on an ACPICA GPE enhancement series which might be merged from
> > > > ACPICA upstream.
> > >
> > > Do I remember correctly that this is the plan?
> > >
> > > So I'm expecting to receive the Linux versions of the relevant ACPICA changes
> > > from you and then I'll apply this material on top of them.
> > >
> > > We don't need to wait for the next ACPICA release with this I think, but
> > > I'd like the GPE changes to be applied to upstream ACPICA at least before
> > > I get them.
> >
> > Yes, I'm trying.
> > I'll re-send this series after an ACPICA release cycle that contains the dependent GPE series.
> >
> > Let me highlight the real value of this EC series:
> > This is a good IO driver material to demonstrate:
> > 1. runtime idle: this is not implemented yet because of ACPICA issues that are not root caused, let me show this possibility this
> below.
> > 2. storming safe: can also deal with all kinds of silicon without worrying about IRQ storming.
> >
> > On top of this, after
> > 1. making sure that acpi_evaluate_object(_Qxx) won't be a blocking point, and extending the referenced period to the end of the
> evaluation,
> > 2. adding 1 more patch to the ACPICA series, using a flag to bypass the automatic GPE disabling/enabling,
> > 3. adding 1 more patch to make EC event poller to disable GPE when sleeping,
> > Linux EC driver can run without GPE enabled when idle.
> > Which means GPE is enabled only when:
> > 1. there is an EC command issued from the EC space handler or
> > 2. the event poller thread is timed out or woken up by the EVT_SCI.
> >
> > So I hope this IO driver enhancement can be a good material to show such possibility.
>
> Yes, it is a good patchset, but I'd like to merge it in an ordered way.
> That is, ACPICA upstream first, patches for Linux from that, the EC series on
> top of this. OK?

Sure.

Thanks and best regards
-Lv

>
> Rafael

ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éÝ¶\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dÊ‡Ú™ë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥