Hi, On Thu, Jun 1, 2017 at 1:50 PM, Tom Lanyon <tom@xxxxxxxxxxxxx> wrote: > [resend as text/plain] > > On Thu, 2017-06-01 at 01:23 +0200, Rafael J. Wysocki wrote: >> Hi All, >> >> This is a follow-up for a patch series posted some time ago: >> >> http://marc.info/?l=linux-kernel&m=149324246701378&w=2 > > I've applied Rafael's s2idle-dell-test branch to 4.12.0-rc3 and tested > on a Dell 9365 and, whilst it's significantly improved, it's not yet > working correctly. > > Previously I could suspend (s2idle and deep), but it took an awkward > ~8 second press of the power button to get it to resume, and I could > never get resume to work when triggering suspend/resume via close/open > of the lid. > > With this patchset applied, I can suspend (s2idle) and a momentary > press of the power button resumes successfully. I can also use the > lid switch to both suspend and resume successfully. > > However, the EC events appear to trigger the machine to wake very > frequently whilst it's supposed to be suspended. This is visible via > the kernel messages at the end of this mail (leaving it in a suspended > state for a few hours resulted in many thousands of these messages), > and the high power draw witnessed. > > Let me know if there's anything I can do to help debug further. Quoting from my cover letter: "After this series there still is a concern regarding the possible increase of power draw that may result from the processing of non-wakeup EC events while suspended which is why the change only affects Dell XPS13 9360 and 9365 for now." So that is what happens, unfortunately, and we can't do much about it at the moment. The only way to avoid that would be to reconfigure the EC during suspend to stop generating non-wakeup events, but today we have no reliable way to do that. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
