Skip to contentSkip to conversation

4.1-rc7, thinkpad X60: platform mode hibernation does very wrong thing

DISCUSSION Pavel Machek 3 participants 9 messages Jun 17, 2015 – Jun 30, 2015 1 message not in the archive ACPI

lore
Share & tools
lore
#0 lore raw

Hi!

On thinkpad X60, if I hibernate in platform mode, it does not power down, but I get black screen, moon icon blinking, and CPU fan running (CPU probably too).

I can hold the power button down to force powerdown... if I notice what is going on before the battery is empty.

Any ideas what could be wrong there?

platform mode hibernation works ok on thinkpad T40p, and shutdown mode hibernation works ok on thinkpad X60 with the same kernel.

Best regards,
Pavel

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw

Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Quoted 11 lines · linus

there might be something wrong with driver, can you provide serial log with no_console_suspend? or using rtctrace

---Original---
From: "Pavel Machek"<[email protected]>
Date: 2015/06/17 20:15:28
To: "rjw"<[email protected]>;"lenb"<[email protected]>;"kernel list"<[email protected]>;"linux-acpi"<[email protected]>;"linux-pm"<[email protected]>;
Subject: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Hi!

Quoted 8 lines

On thinkpad X60, if I hibernate in platform mode, it does not power down, but I get black screen, moon icon blinking, and CPU fan running (CPU probably too).

I can hold the power button down to force powerdown... if I notice what is going on before the battery is empty.

Any ideas what could be wrong there?

Not right away.

Have you tried any previous 4.1-rc kernels?

Rafael

#3 lore raw

Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Hi!

Quoted 3 lines · linus
Quoted 2 lines

there might be something wrong with driver, can you provide serial log with no_console_suspend? or using rtctrace

Who are you?

The hibernation image is stored successfully, that means drivers work ok. Serial console... might be fun to set up, but might help, right.

Quoted 6 lines
Quoted 6 lines

Hi!

On thinkpad X60, if I hibernate in platform mode, it does not power down, but I get black screen, moon icon blinking, and CPU fan running (CPU probably too).

Quoted 8 lines
Quoted 4 lines

I can hold the power button down to force powerdown... if I notice what is going on before the battery is empty.

Any ideas what could be wrong there?

Not right away.

Have you tried any previous 4.1-rc kernels?

Yes, and it was broken there, too. It is possible that 4.0 is broken, too, I did not use hibernation for some time.

3.14 kernel works ok. I'm trying v3.19 now.

Best regards,
Pavel

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
#4 lore raw

Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Quoted 9 lines · Pavel Machek

Hi!

Quoted 3 lines · linus
Quoted 2 lines

there might be something wrong with driver, can you provide serial log with no_console_suspend? or using rtctrace

Who are you?

The hibernation image is stored successfully, that means drivers work ok.

Well, not entirely necessarily. We execute the ->poweroff callback for all of them in the last step, maybe one of these hangs.

You can verify that by adding a reboot trigger after the dpm_suspend_end() invocation in hibernation_platform_enter(). If it triggers 100% of the time, drivers are OK.

Quoted 13 lines

Serial console... might be fun to set up, but might help, right.

Quoted 11 lines
Quoted 10 lines

Hi!

On thinkpad X60, if I hibernate in platform mode, it does not power down, but I get black screen, moon icon blinking, and CPU fan running (CPU probably too).

I can hold the power button down to force powerdown... if I notice what is going on before the battery is empty.

Any ideas what could be wrong there?

Quoted 8 lines
Quoted 3 lines

Not right away.

Have you tried any previous 4.1-rc kernels?

Yes, and it was broken there, too. It is possible that 4.0 is broken, too, I did not use hibernation for some time.

3.14 kernel works ok. I'm trying v3.19 now.

I see.

OK, it would be good to narrow down the time frame when it broke.

Thanks,
Rafael

#5 lore raw

Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Quoted 7 lines · Rafael J. Wysocki
Quoted 6 lines · Pavel Machek

Hi!

Quoted 3 lines · linus
Quoted 2 lines

there might be something wrong with driver, can you provide serial log with no_console_suspend? or using rtctrace

Quoted 6 lines
Quoted 3 lines

Who are you?

The hibernation image is stored successfully, that means drivers work ok.

Well, not entirely necessarily. We execute the ->poweroff callback for all of them in the last step, maybe one of these hangs.

shutdown mode works ok, that's why I think driver problem is not too likely.

Quoted 8 lines
Quoted 4 lines

Yes, and it was broken there, too. It is possible that 4.0 is broken, too, I did not use hibernation for some time.

3.14 kernel works ok. I'm trying v3.19 now.

I see.

OK, it would be good to narrow down the time frame when it broke.

3.19 is broken. Some kind of strange 3.17 works ok.

(On an additional note, x60 fails to boot when "hot" halting at

"ACPI: All ACPI Tables successfully acquired"

That was happening before, but it is really annoying for debugging. As you do two reboots, machine gets hot enough, and you have to wait to cool it down. Any ideas what to do there?)

I tried no_console_suspend=1, but it still suspended the console.
Pavel

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
#6 lore raw

Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Quoted 11 lines · Pavel Machek

3.19 is broken. Some kind of strange 3.17 works ok.

(On an additional note, x60 fails to boot when "hot" halting at

"ACPI: All ACPI Tables successfully acquired"

That was happening before, but it is really annoying for debugging. As you do two reboots, machine gets hot enough, and you have to wait to cool it down. Any ideas what to do there?)

I tried no_console_suspend=1, but it still suspended the console.

And this all reminded me of https://lkml.org/lkml/2015/3/18/133 . So I think that's another system not covered by commit ab3be73fa7b4 ("drm/i915: gen4: work around hang during hibernation").

Hope this helps,

Paul Bolle

#7 lore raw

da2bc1b9db3351addd293e5b82757efe1f77ed1d breaks hibernation was Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Hi!

Quoted 3 lines

And this all reminded me of https://lkml.org/lkml/2015/3/18/133 . So I think that's another system not covered by commit ab3be73fa7b4 ("drm/i915: gen4: work around hang during hibernation").

Yes, another system broken by

commit da2bc1b9db3351addd293e5b82757efe1f77ed1d

Author: Imre Deak <[email protected]>
Date:   Thu Oct 23 19:23:26 2014 +0300

drm/i915: add poweroff_late handler

The suspend_late handler saves some registers and powers off the device,
so it doesn't have a big overhead. Calling it at S4
poweroff_late time
makes the power off handling identical to the S3 suspend and
S4 freeze
handling, so do this for consistency.

And no, adding another system to the "blacklist" is a not good idea, because we know there are other broken systems out there. Can we revert da2bc1b9db3351addd293e5b82757efe1f77ed1d, please? Consistency is nice, but not having regressions is even nicer.
Pavel

PS: Now I have real Linus and fake Linus on the Cc list. Or maybe two real Linuses, its hard to tell.

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
#8 lore raw

Re: da2bc1b9db3351addd293e5b82757efe1f77ed1d breaks hibernation was Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Hi!

Quoted 9 lines
Quoted 3 lines

And this all reminded me of https://lkml.org/lkml/2015/3/18/133 . So I think that's another system not covered by commit ab3be73fa7b4 ("drm/i915: gen4: work around hang during hibernation").

Yes, another system broken by

commit da2bc1b9db3351addd293e5b82757efe1f77ed1d

Author: Imre Deak <[email protected]>
Date:   Thu Oct 23 19:23:26 2014 +0300

Imre, your patch already broken 3 machines, and now we discovered it breakes another. Can you please revert it before it causes more damage?

Thanks,
Pavel

Quoted 20 lines

drm/i915: add poweroff_late handler

The suspend_late handler saves some registers and powers off the device,
so it doesn't have a big overhead. Calling it at S4
poweroff_late time
makes the power off handling identical to the S3 suspend and
S4 freeze
handling, so do this for consistency.

And no, adding another system to the "blacklist" is a not good idea, because we know there are other broken systems out there. Can we revert da2bc1b9db3351addd293e5b82757efe1f77ed1d, please? Consistency is nice, but not having regressions is even nicer.
Pavel

PS: Now I have real Linus and fake Linus on the Cc list. Or maybe two real Linuses, its hard to tell.

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
#9 lore raw

Re: da2bc1b9db3351addd293e5b82757efe1f77ed1d breaks hibernation was Re: 4.1-rc7, thinkpad X60: platform mode hibernation does very wrongthing

Quoted 8 lines · Pavel Machek

Hi!

Quoted 6 lines
Quoted 3 lines

And this all reminded me of https://lkml.org/lkml/2015/3/18/133 . So I think that's another system not covered by commit ab3be73fa7b4 ("drm/i915: gen4: work around hang during hibernation").

Yes, another system broken by

Quoted 7 lines
Quoted 3 lines

commit da2bc1b9db3351addd293e5b82757efe1f77ed1d

Author: Imre Deak <[email protected]>
Date:   Thu Oct 23 19:23:26 2014 +0300

Imre, your patch already broken 3 machines, and now we discovered it breakes another. Can you please revert it before it causes more damage?

It looks like Imre has just sent a fix for that. Can you please test that one?

Rafael