Re: [PATCH v2 0/5] ACPI / EC: Add reference counting for requests and cleans up the grace periods support.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2/9/2015 3:23 AM, Zheng, Lv wrote:
Hi, Rafael

From: Rafael J. Wysocki [mailto:rjw@xxxxxxxxxxxxx]
Sent: Friday, February 06, 2015 9:27 AM

On Friday, February 06, 2015 08:57:37 AM Lv Zheng wrote:
This patchset contains 3 cleanups related to the EC driver:
1. Command flushing (command grace period)
    This patchset flushes EC commands before suspending/resuming, so that
    there won't be timeout for the incomplete commands after resuming.
2. Query flushing (query grace period)
    This patchset flushes EC event queries before suspending/resuming, so
    that there won't be broken events remained in the firmware queue.
3. Command storming prevention
    This patchset corrects command storming prevention logic because of
    the GPE raw handler mode.
The request reference count debugging messages can be used to detect broken
EC transactions. It should always drop to 1 when the driver is idle during
the runtime.

Note that after flushing before suspending, EC GPE is still enabled to keep
the old behavior.

Lv Zheng (5):
   ACPI/EC: Introduce STARTED/STOPPED flags to replace BLOCKED flag.
   ACPI/EC: Add command flushing support.
   ACPI/EC: Refine command storm prevention support.
   ACPI/EC: Add query flushing support.
   ACPI/EC: Add GPE reference counting debugging messages.

  drivers/acpi/ec.c       |  295 ++++++++++++++++++++++++++++++++++++++++-------
  drivers/acpi/internal.h |    1 +
  2 files changed, 254 insertions(+), 42 deletions(-)
So this is on top of the EC patches you sent previously, right?
Yes.

The sequence is:
ACPICA 20150204 release: http://www.spinics.net/lists/linux-acpi/msg55623.html
ACPI EC GPE race fixes: http://www.spinics.net/lists/linux-acpi/msg55633.html
And this series.

Unfortunately, patch [4/5] from this series broke system suspend on Acer Aspire S5 for me.

Apparently, the box hangs hard during the last stage of suspend (after offlining nonboot CPUs).

The issue is 100% reproducible and reverting the patch (along with [5/5] that depends on it) fixes the problem, so I'm going to push the reverts to Linus on Friday.

I can send you an acpidump output from the affected machine, but I won't be able to do any testing on it before Feb 23, so reverting is the only viable option for the time being.

Please let me know if you want the acpidump output.

Kind regards,
Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux