From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH 0/2] PM / sleep: Support for using suspend-to-idle by default Date: Wed, 7 Dec 2016 23:56:41 +0100 Message-ID: References: <9316960.49SRRJV8RJ@vostro.rjw.lan> <71203ae1bfe54eb39d79c54b51ba7ae8@ausx13mpc120.AMER.DELL.COM> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Mario Limonciello Cc: "Rafael J. Wysocki" , "Rafael J. Wysocki" , Linux PM , ACPI Devel Maling List , Len Brown , Linux Kernel Mailing List , Andy Lutomirski List-Id: linux-pm@vger.kernel.org On Wed, Dec 7, 2016 at 10:38 PM, wrote: >> -----Original Message----- >> From: rjwysocki@gmail.com [mailto:rjwysocki@gmail.com] On Behalf Of >> Rafael J. Wysocki >> Sent: Thursday, November 17, 2016 3:05 PM >> To: Limonciello, Mario >> Cc: Rafael J. Wysocki ; Linux PM > pm@vger.kernel.org>; ACPI Devel Maling List ; >> Len Brown ; Linux Kernel Mailing List > kernel@vger.kernel.org>; Andy Lutomirski >> Subject: Re: [PATCH 0/2] PM / sleep: Support for using suspend-to-idle by >> default >> >> On Thu, Nov 17, 2016 at 9:40 PM, wrote: >> > Hi Rafael, >> > >> >> -----Original Message----- >> >> From: Rafael J. Wysocki [mailto:rjw@rjwysocki.net] >> >> Sent: Wednesday, November 16, 2016 8:19 PM >> >> To: Linux PM list >> >> Cc: ACPI Devel Maling List ; Len Brown >> >> ; Linux Kernel Mailing List > >> kernel@vger.kernel.org>; Limonciello, Mario >> >> ; Andy Lutomirski >> >> Subject: [PATCH 0/2] PM / sleep: Support for using suspend-to-idle by >> >> default >> >> >> >> Hi, >> >> >> >> This series is a follow-up for a BoF discussion during the LPC. >> >> >> >> The discussion was about supporting Linux on modern laptops and one of >> >> the issues mentioned was the lack of support for suspend-to-idle in user >> >> space and if there's anything that can be done about that in the kernel. >> >> >> >> The following patches are my input. :-) >> >> >> >> The first one reworks the suspend interface to allow the "mem" string >> >> in /sys/power/state to represent multiple things that can be selected via >> >> an additional sysfs attribute. >> >> >> >> The second one makes ACPI select suspend-to-idle as the default >> suspend >> >> mode >> >> if so indicated in the FADT. >> >> >> >> Please let me know what you think. >> >> >> >> Thanks, >> >> Rafael >> > >> > Thanks for submitting. I like the approach, especially that it leaves room >> > to easily modify the behavior for debugging purposes and adding quirks. >> >> Well, that's the point. :-) >> >> > I tested your series on a machine that had this bit set in the FADT and >> > confirmed that it set up and used policy properly. >> > >> > Tested-By: Mario Limonciello >> >> Thanks! > > Rafael, > > I hadn't noticed these merged into any of your linux-pm.git branches. > > Do you have a plan for merging them? Or waiting for other feedback? They are in the linux-next branch (and therefore in linux-next), so on their way to 4.10-rc1. I usually only update topic branches at kernel.org when they are ready to go into the mainline and the ones I have locally are merged into the linux-next one beforehand. Thanks, Rafael