Bug 181381 - Resume from suspend no longer works on Thinkpad X240
Summary: Resume from suspend no longer works on Thinkpad X240
Status: CLOSED CODE_FIX
Alias: None
Product: Power Management
Classification: Unclassified
Component: Hibernation/Suspend (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: Rafael J. Wysocki
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-24 20:28 UTC by Steinar H. Gunderson
Modified: 2016-11-17 00:49 UTC (History)
4 users (show)

See Also:
Kernel Version: 4.9.0-rc1
Subsystem:
Regression: Yes
Bisected commit-id:


Attachments

Description Steinar H. Gunderson 2016-10-24 20:28:18 UTC
Hi,

I have a Thinkpad X240, which suspends (to RAM) and resumes just fine with 4.7.4. However, in 4.9.0-rc1, it completely refuses to resume; when I try to press the power button to resume it from sleep, it just ignores the request completely and keeps pulsing the light.

bisect gives the following culprit:

commit 0c541332231e7957f23013e37f3720db33e7804c
Author: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Date:   Fri Jun 17 13:12:20 2016 +0200

    tpm: use tpm_pcr_read_dev() in tpm_do_selftest()

    Instead of a ad-hoc protocol message construction it is better to
    call tpm_pcr_read_dev().

    Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
    Reviewed-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>

I haven't tried 4.9.0-rc2, because that just hangs directly off of GRUB from me.
Comment 1 Rafael J. Wysocki 2016-10-24 20:35:24 UTC
On Monday, October 24, 2016 08:28:18 PM bugzilla-daemon@bugzilla.kernel.org wrote:
> https://bugzilla.kernel.org/show_bug.cgi?id=181381
> 
>             Bug ID: 181381
>            Summary: Resume from suspend no longer works on Thinkpad X240
>            Product: Power Management
>            Version: 2.5
>     Kernel Version: 4.9.0-rc1
>           Hardware: All
>                 OS: Linux
>               Tree: Mainline
>             Status: NEW
>           Severity: normal
>           Priority: P1
>          Component: Hibernation/Suspend
>           Assignee: rjw@rjwysocki.net
>           Reporter: steinar+kernel@gunderson.no
>                 CC: jarkko.sakkinen@linux.intel.com
>         Regression: No
> 
> Hi,
> 
> I have a Thinkpad X240, which suspends (to RAM) and resumes just fine with
> 4.7.4. However, in 4.9.0-rc1, it completely refuses to resume; when I try to
> press the power button to resume it from sleep, it just ignores the request
> completely and keeps pulsing the light.
> 
> bisect gives the following culprit:
> 
> commit 0c541332231e7957f23013e37f3720db33e7804c
> Author: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> Date:   Fri Jun 17 13:12:20 2016 +0200
> 
>     tpm: use tpm_pcr_read_dev() in tpm_do_selftest()
> 
>     Instead of a ad-hoc protocol message construction it is better to
>     call tpm_pcr_read_dev().
> 
>     Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
>     Reviewed-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
> 
> I haven't tried 4.9.0-rc2, because that just hangs directly off of GRUB from
> me.

Jarkko, Jason, any chance to have a look at this?

Thanks,
Rafael
Comment 2 Srinivas Pandruvada 2016-10-24 21:30:28 UTC
May be this (It has Fixes: 0c541332231e ("tpm: use tpm_pcr_read_dev() in tpm_do_selftest()"):

http://git.infradead.org/users/jjs/linux-tpmdd.git/commitdiff/65da72b7ddcdd8990e4783d09c7e86d90ccb4121?hp=1306d8e1c09fdc8ecb9ef235e2116352f810f9c5

There was some discussion here:
http://www.gossamer-threads.com/lists/linux/kernel/2541771
Comment 3 jarkko.sakkinen 2016-10-26 10:52:44 UTC
On Mon, Oct 24, 2016 at 10:42:23PM +0200, Rafael J. Wysocki wrote:
> On Monday, October 24, 2016 08:28:18 PM bugzilla-daemon@bugzilla.kernel.org
> wrote:
> > https://bugzilla.kernel.org/show_bug.cgi?id=181381
> > 
> >             Bug ID: 181381
> >            Summary: Resume from suspend no longer works on Thinkpad X240
> >            Product: Power Management
> >            Version: 2.5
> >     Kernel Version: 4.9.0-rc1
> >           Hardware: All
> >                 OS: Linux
> >               Tree: Mainline
> >             Status: NEW
> >           Severity: normal
> >           Priority: P1
> >          Component: Hibernation/Suspend
> >           Assignee: rjw@rjwysocki.net
> >           Reporter: steinar+kernel@gunderson.no
> >                 CC: jarkko.sakkinen@linux.intel.com
> >         Regression: No
> > 
> > Hi,
> > 
> > I have a Thinkpad X240, which suspends (to RAM) and resumes just fine with
> > 4.7.4. However, in 4.9.0-rc1, it completely refuses to resume; when I try
> to
> > press the power button to resume it from sleep, it just ignores the request
> > completely and keeps pulsing the light.
> > 
> > bisect gives the following culprit:
> > 
> > commit 0c541332231e7957f23013e37f3720db33e7804c
> > Author: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> > Date:   Fri Jun 17 13:12:20 2016 +0200
> > 
> >     tpm: use tpm_pcr_read_dev() in tpm_do_selftest()
> > 
> >     Instead of a ad-hoc protocol message construction it is better to
> >     call tpm_pcr_read_dev().
> > 
> >     Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> >     Reviewed-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
> > 
> > I haven't tried 4.9.0-rc2, because that just hangs directly off of GRUB
> from
> > me.
> 
> Jarkko, Jason, any chance to have a look at this?

I've fixed this a while ago but the pull request is still on hold:

http://www.spinics.net/lists/kernel/msg2356176.html 

> Thanks,
> Rafael

/Jarkko
Comment 4 Andreas Kloeckner 2016-11-08 04:26:48 UTC
Still present in Debian's 4.9rc3, observed on a ThinkPad X250.
Comment 5 Zhang Rui 2016-11-14 05:42:29 UTC
(In reply to Steinar H. Gunderson from comment #0)
> Hi,
> 
> I have a Thinkpad X240, which suspends (to RAM) and resumes just fine with
> 4.7.4. However, in 4.9.0-rc1, it completely refuses to resume; when I try to
> press the power button to resume it from sleep, it just ignores the request
> completely and keeps pulsing the light.
> 
> bisect gives the following culprit:
> 
> commit 0c541332231e7957f23013e37f3720db33e7804c
> Author: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> Date:   Fri Jun 17 13:12:20 2016 +0200
> 
>     tpm: use tpm_pcr_read_dev() in tpm_do_selftest()
> 
>     Instead of a ad-hoc protocol message construction it is better to
>     call tpm_pcr_read_dev().
> 
>     Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
>     Reviewed-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
> 
> I haven't tried 4.9.0-rc2, because that just hangs directly off of GRUB from
> me.

does the problem still exist in the latest upstream kernel?
Comment 6 Zhang Rui 2016-11-14 05:44:33 UTC
(In reply to Andreas Kloeckner from comment #4)
> Still present in Debian's 4.9rc3, observed on a ThinkPad X250.

hmmm, Andreas, this may or may not be the same problem.

does the problem still exist with this kernel
#git checkout 0c541332231e7957f23013e37f3720db33e7804c~

does the problem still exist with this kernel
#git checkout 0c541332231e7957f23013e37f3720db33e7804c
Comment 7 jarkko.sakkinen 2016-11-14 21:59:56 UTC
$ git describe --contains befd99656c5eb765fe9d96045c4cba099fd938db
v4.9-rc4~25^2
Comment 8 Andreas Kloeckner 2016-11-16 16:58:58 UTC
No longer present in Debian's 4.9rc5. Thanks for the help!
Comment 9 jarkko.sakkinen 2016-11-16 17:58:46 UTC
good to hear! thanks for reporting this.

Note You need to log in before you can comment on or make changes to this bug.