Messages in this thread | | | From | "Rafael J. Wysocki" <> | Date | Wed, 18 Jan 2017 03:18:41 +0100 | Subject | Re: Regression on Dell XPS13 (was: [char-misc for 4.10-rc4 V2] mei: bus: enable OS version only for SPT and newer) |
| |
Mario, thanks for CCing this to linux-pm.
On Tue, Jan 17, 2017 at 5:57 PM, <Mario.Limonciello@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:pmenzel@molgen.mpg.de] >> Sent: Tuesday, January 17, 2017 8:34 AM >> To: Rafael J. Wysocki <rafael.j.wysocki@intel.com>; Limonciello, Mario >> <Mario_Limonciello@Dell.com> >> Cc: Thorsten Leemhuis <linux@leemhuis.info>; Greg Kroah-Hartman >> <gregkh@linuxfoundation.org>; Tomas Winkler <tomas.winkler@intel.com>; >> Jan Niehusmann <jan@gondor.com>; Usyskin, Alexander >> <alexander.usyskin@intel.com>; linux-kernel@vger.kernel.org; Chen, Yu C >> <yu.c.chen@intel.com>; Sarvela, Tomi P <tomi.p.sarvela@intel.com>; Daniel >> Blueman <daniel@quora.org>; Len Brown <len.brown@intel.com>; linux- >> pm@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
| |