Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751732AbdFTCpW (ORCPT ); Mon, 19 Jun 2017 22:45:22 -0400 Received: from mga06.intel.com ([134.134.136.31]:32103 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750905AbdFTCpU (ORCPT ); Mon, 19 Jun 2017 22:45:20 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.39,363,1493708400"; d="scan'208";a="869734164" From: "Zheng, Lv" To: Bastien Nocera CC: Benjamin Tissoires , Peter Hutterer , Lennart Poettering , "Wysocki, Rafael J" , "Rafael J . Wysocki" , "Brown, Len" , Lv Zheng , "linux-acpi@vger.kernel.org" , "systemd-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , "linux-input@vger.kernel.org" Subject: RE: [systemd-devel] [WIP PATCH 0/4] Rework the unreliable LID switch exported by ACPI Thread-Topic: [systemd-devel] [WIP PATCH 0/4] Rework the unreliable LID switch exported by ACPI Thread-Index: AQHS2wd01O/IlIfVsk2vy8j9OHBM16IYiNMAgAmUZoCAAzAyYP//vNyAgACPuZD//4QFgIABmvZA///tuAAAEOsxgP//haIA//9u+qCAAKDZAP/7QTDAgApQYoD//zJqoA== Date: Tue, 20 Jun 2017 02:45:09 +0000 Message-ID: <1AE640813FDE7649BE1B193DEA596E886CED1C41@SHSMSX101.ccr.corp.intel.com> References: <20170601184632.2980-1-benjamin.tissoires@redhat.com> <20170607074848.GE27006@gardel-login> <20170613100617.GD29589@mail.corp.redhat.com> <1AE640813FDE7649BE1B193DEA596E886CED0386@SHSMSX101.ccr.corp.intel.com> <20170615064715.GA6195@jelly> <1AE640813FDE7649BE1B193DEA596E886CED047B@SHSMSX101.ccr.corp.intel.com> <20170615075755.GA10001@jelly> <1AE640813FDE7649BE1B193DEA596E886CED0A03@SHSMSX101.ccr.corp.intel.com> <20170616072322.GL5085@mail.corp.redhat.com> <1AE640813FDE7649BE1B193DEA596E886CED0AC1@SHSMSX101.ccr.corp.intel.com> <20170616080950.GM5085@mail.corp.redhat.com> <1AE640813FDE7649BE1B193DEA596E886CED0C5E@SHSMSX101.ccr.corp.intel.com> <33C872C7-EFB2-4CA0-8CAD-8B6E8916180C@hadess.net> <1AE640813FDE7649BE1B193DEA596E886CED1419@SHSMSX101.ccr.corp.intel.com> <1497910127.2559.1.camel@hadess.net> In-Reply-To: <1497910127.2559.1.camel@hadess.net> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZDI4NjFlOTAtMjE2ZC00YTJlLWFiMjYtYzA4OTI5OGY2YWEzIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE2LjUuOS4zIiwiVHJ1c3RlZExhYmVsSGFzaCI6IlBWYWFwVjZNUHVUXC9YeU9HQlozclF2SzgwQjBna2pCdjVUUis5U3N5YTl3PSJ9 x-ctpclassification: CTP_IC dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action 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 mail.home.local id v5K2jbOh028374 Content-Length: 3542 Lines: 95 Hi, > From: Bastien Nocera [mailto:hadess@hadess.net] > Subject: Re: [systemd-devel] [WIP PATCH 0/4] Rework the unreliable LID switch exported by ACPI > > On Mon, 2017-06-19 at 01:43 +0000, Zheng, Lv wrote: > > > > > > > > If you implement it in such a way that GNOME settings daemon > > > behaves weirdly, you'll get my revert > > > request in the mail. Do. Not. Ever. Lie. > > > > First, I don't know what should be reverted... > > I have 2 solutions here for review, and Benjamin has 1. > > And none of them has been upstreamed. > > We are just discussing. > > The discussion is getting tiring quite frankly. We've been over this > for nearly a year now, and with no end in sight. We have concerns to introduce too complicated logics to such a simple button driver especially the logics are related to platform firmware, input ABI and user space behaviors. I understand the situation. Anyway this shouldn't be a big deal. Let's prepare a smarter series to collect all fixes and solutions with runtime configurables and get that to the end users. So that we can figure out which is the simplest solution. But before that, let me ask several questions about gnome-setting-deamon. > > > However we need to get 1 of them upstreamed in next cycle. > > > > I think users won't startup gnome-setting-daemon right after resume. > > It should have already been started. > > > > There is only 1 platform may see delayed state update after resume. > > Let's see if there is a practical issue. > > 1. Before suspend, the "lid state" is "close", and > > 2. After resume, the state might remain "close" for a while > > Since libinput won't deliver close to userspace, > > and gnome-setting-daemon listens to key switches, there is no > > wrong behavior. > > It doesn't. It listens to UPower, which tells user-space whether there > is a lid switch, and whether it's opened or closed. Thanks for the information. However I don't see differences here. > > > 3. Then after several seconds, "open" arrives. > > gnome-setting-daemon re-arrange monitors and screen layouts in > > response to the new event. > > Just how is anyone supposed to know that there is an event coming? Will UPower deliver EV_SW key events to gnome-setting-daemon? > > > So there is no problem. IMO, there is no need to improve for post- > > resume case. > > > > Users will just startup gnome-setting-daemon once after boot. > > And it's likely that when it is started, the state is correct. > > You cannot rely on when gnome-settings-daemon will be started to make > *any* decision. Certainly not decisions on how the kernel should > behave. My bad wording, I just meant: When gnome-settings-daemon is started is not related to what we are discussing. Do you want to fix regressions? Or you want to fix new issues on recent platforms? If you want to fix regressions, I think Benjamin has submitted a revision to use old method mode, there shouldn't be regressions for gnome-settings-daemon. What else we want to do is to fix regressions related to systemd when we go back to default method mode. Since there is no issue with systemd 233 and after just applying a small change, systemd 229 can also be worked around, I mean dynamically add/remove input node is not strictly required for achieving our purposes. But if you want to fix new issues on new platforms, we can discuss further and determine which program should be changed and which program is the best candidate to stop all problems - the ACPI button driver or the user space. Cheers Lv