Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967060AbdIZIyF (ORCPT ); Tue, 26 Sep 2017 04:54:05 -0400 Received: from mga07.intel.com ([134.134.136.100]:24654 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966863AbdIZIyB (ORCPT ); Tue, 26 Sep 2017 04:54:01 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,440,1500966000"; d="scan'208";a="1175862750" From: Lv Zheng To: "Rafael J . Wysocki" , "Rafael J . Wysocki" , Len Brown Cc: Lv Zheng , Lv Zheng , linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org Subject: [PATCH 0/2] ACPI / EC: EC regression fixes related to EC event stuck Date: Tue, 26 Sep 2017 16:53:57 +0800 Message-Id: X-Mailer: git-send-email 2.7.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2077 Lines: 47 There are 2 regressions reported on bugzilla. Link: https://bugzilla.kernel.org/show_bug.cgi?id=196847 [#1] Link: https://bugzilla.kernel.org/show_bug.cgi?id=196833 [#2] It looks they are related to the modification against the timing of acpi_ec_enable_event() invocation, but they are all due to different root causes. For link 1, its root cause is: we rely on acpi_ec_suspend/resume() to control the timing of EC event handling enablement, but ECDT is not an ACPI device, thus the PM hooks are not invoked for it. For link 2, reporter seems to favor a fix that can fix both of the problems: 1. EC event can stuck if there is no triggering source. 2. Post-resume enabling of EC event handling may be too late. But actually the report is only against problem 1 which is known to have a very low and accepted reproduce ratio. For problem 2, it should has a sympton related to the driver resume order rather than EC event stuck. And its fix should be acpi.ec_freeze_events=Y related. Note acpi.ec_freeze_events=Y is a feature we need to keep for now as: 1. It's reasonable for EC driver to act as an agent for EC event handling while EC transaction is handled by ACPI core handler, thus 1.1. unbind/suspend means suspending EC event handling; 1.2. bind/resume means resuming EC event handling. 2. According to the known EC FW facts, it takes time for EC FW to initialize after boot/resume. This patchset fixes the root causes of the 2 regressions and keeps the possibility of tuning acpi.ec_freeze_events=Y. Note that PATCH 02 depends on PATCH 01 to be safe for driver unbind. Lv Zheng (2): ACPI / EC: Fix a regression related to the triggering source of the EC event handling ACPI / EC: Fix a regression related to the PM ops support of ECDT device drivers/acpi/ec.c | 81 +++++++++++++++++++++++++++++---------------- drivers/acpi/internal.h | 1 + drivers/acpi/scan.c | 21 ++++++++++++ include/acpi/acpi_bus.h | 1 + include/acpi/acpi_drivers.h | 1 + 5 files changed, 76 insertions(+), 29 deletions(-) -- 2.7.4