Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759173AbbBICXq (ORCPT ); Sun, 8 Feb 2015 21:23:46 -0500 Received: from mga02.intel.com ([134.134.136.20]:49647 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755625AbbBICXo (ORCPT ); Sun, 8 Feb 2015 21:23:44 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,541,1418112000"; d="scan'208";a="682866601" From: "Zheng, Lv" To: "Rafael J. Wysocki" CC: "Wysocki, Rafael J" , "Brown, Len" , Lv Zheng , "linux-kernel@vger.kernel.org" , "linux-acpi@vger.kernel.org" Subject: RE: [PATCH v2 0/5] ACPI / EC: Add reference counting for requests and cleans up the grace periods support. Thread-Topic: [PATCH v2 0/5] ACPI / EC: Add reference counting for requests and cleans up the grace periods support. Thread-Index: AQHQQajx86koHFWciU69KWZ80WW0U5znmfWA Date: Mon, 9 Feb 2015 02:23:37 +0000 Message-ID: <1AE640813FDE7649BE1B193DEA596E88026D4B94@SHSMSX101.ccr.corp.intel.com> References: <20141119121615.GA2514@node.dhcp.inet.fi> <1711204.9KEDEeQMDo@vostro.rjw.lan> In-Reply-To: <1711204.9KEDEeQMDo@vostro.rjw.lan> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id t192NrMZ008251 Content-Length: 2086 Lines: 53 Hi, Rafael > From: Rafael J. Wysocki [mailto:rjw@rjwysocki.net] > 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. Thanks and best regards -Lv > > > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center. ????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?