Mario, thanks for CCing this to linux-pm. On Tue, Jan 17, 2017 at 5:57 PM, <mario.limoncie...@dell.com> wrote: > Hi Paul, > > Thanks for raising this topic and including me. > Suspend to Idle support in Linux as an alternative S3 on x86 is a new > topic. In all, I expected that some problems would arise as a result > of this patch, and I hope they will spur interesting discussions and > solutions. > > Something that might not be immediately obvious is that by supporting > suspend to idle, new driver bugs are going to be identified for drivers > that don't properly put hardware into low enough power modes for the CPU > to go into the proper state. Traditionally with S3 the BIOS would be > responsible > for powering devices down before S3 and back up when exiting S3. > This onus is now on the OS. > >> -----Original Message----- >> From: Paul Menzel [mailto:pmen...@molgen.mpg.de] >> Sent: Tuesday, January 17, 2017 8:34 AM >> To: Rafael J. Wysocki <rafael.j.wyso...@intel.com>; Limonciello, Mario >> <mario_limoncie...@dell.com> >> Cc: Thorsten Leemhuis <li...@leemhuis.info>; Greg Kroah-Hartman >> <gre...@linuxfoundation.org>; Tomas Winkler <tomas.wink...@intel.com>; >> Jan Niehusmann <j...@gondor.com>; Usyskin, Alexander >> <alexander.usys...@intel.com>; linux-kernel@vger.kernel.org; Chen, Yu C >> <yu.c.c...@intel.com>; Sarvela, Tomi P <tomi.p.sarv...@intel.com>; Daniel >> Blueman <dan...@quora.org>; Len Brown <len.br...@intel.com>; linux- >> p...@vger.kernel.org >> Subject: Regression on Dell XPS13 (was: [char-misc for 4.10-rc4 V2] mei: bus: >> enable OS version only for SPT and newer) >> >> Dear Rafael, dear Mario, >> >> >> On 01/17/17 09:14, Thorsten Leemhuis wrote: >> > Greg Kroah-Hartman wrote on 16.01.2017 12:05: >> >> On Sun, Jan 15, 2017 at 11:58:30AM +0100, Greg Kroah-Hartman wrote: >> >>> On Sun, Jan 15, 2017 at 07:19:03AM +0000, Winkler, Tomas wrote: >> >>>>> Subject: Re: [char-misc for 4.10-rc4 V2] mei: bus: enable OS >> >>>>> version only for SPT and newer On Sat, Jan 14, 2017 at 08:27:31PM >> >>>>> +0100, Paul Menzel wrote: >> >>>>>> On 2017-01-13 14:00, Greg Kroah-Hartman wrote: >> >>>>>>> On Wed, Jan 11, 2017 at 03:26:06PM +0100, Paul Menzel wrote: >> >>>>>>>> On 01/11/17 15:12, Winkler, Tomas wrote: >> >>>>>>>>>>> On 01/11/17 10:24, Winkler, Tomas wrote: >> >>>>>>>>>>>>> On Wed, Jan 11, 2017 at 01:27:21AM +0200, Tomas Winkler >> wrote: >> >>>>>>>>>>>>>> On older platforms the command should be just ignored >> by >> >>>>>>>>>>>>>> the firmware but some older platforms misbehave so it's >> >>>>>>>>>>>>>> safer to send the command only if required. >> >>>>>>>>>>>>> Thanks! This fixes suspend-to-ram for me (on a Thinkpad >> x201s). >> >>>>>>>>>>>> What about Dell XPS13? >> >> There is a regression with Linux 4.10-rc{1,2,3} on the Intel Kaby Lake device >> Dell XPS13 (9360). Hitting the power button, the light of the power button >> never goes off. Pressing it again, nothing happen. Only pressing it like ten >> seconds the screen comes back for five seconds, and then it seems to try to >> suspend again.
Paul, Can you please check if reverting commit 08b98d329165 (PM / sleep / ACPI: Use the ACPI_FADT_LOW_POWER_S0 flag) helps? If not, please send me the output of "cat /sys/power/mem_sleep" from the affected system. Thanks, Rafael
