Re: [BUG] s2idle: Surface Pro 3 suspend/resume issues

[Zhang Rui <rui.zhang@intel.com>]

On Mon, 2018-01-08 at 07:55 +0000, Valentin Manea wrote:
> Again with correct linux-pm
> 
> Hi,
> 
>    Somewhere between 4.12 and 4.13 the s2idle doesn't work anymore on
> the
> Surface Pro 3.

will you please be more specific about the symptom?
what command do you use to do s2idle, and how do you wake it up?


BTW, does the problem still exists in the latest upstream kernel?

thanks,
rui
>    After some bisecting I found 3 patches that seem to be at fault
> ACPI / sleep: EC-based wakeup from suspend-to-idle on recent systems
> ->
> 0f1a83833a0ba93d6986d9e30c8fe35d7d45c17c
> 
> ACPI / PM: Ignore spurious SCI wakeups from suspend-to-idle ->
> 33e4f80ee69b5168badf37edbfed796eb48434b9
> 
> ACPI / PM: Clean up device wakeup enable/disable code ->
> 235d81a630ca2d39818da96f0c14bc960ffbaeb5
> 
>    It doesn't seem to be just one patch, I've tried combinations of
> the
> 3, but only with all three reverted s2idle suspend and resume starts
> working again.
>    I'm not versed in x86 and ACPI so my testing is very coarse
> because
> there is some re-factoring included there. Do you have any idea which
> parts could be casing the problem? Or at least how to debug further?
> 
> Thanks,
> Valentin