* Jinxed VAIO wreckage - current state of affairs
@ 2007-06-09 19:54 Thomas Gleixner
2007-06-09 20:59 ` Rafael J. Wysocki
0 siblings, 1 reply; 13+ messages in thread
From: Thomas Gleixner @ 2007-06-09 19:54 UTC (permalink / raw)
To: LKML; +Cc: Andrew Morton, Ingo Molnar, Len Brown, Rafael J. Wysocki
Andrew's jinxed VAIO breaks with the high resolution timer updates in a
very strange way. Andrew identified the following patch as the culprit:
http://www.tglx.de/projects/hrtimers/2.6.22-rc4/broken-out/clockevents-fix-resume-logic.patch
This makes no sense at all. The patch just moves the timer restart to a
different (later) place in the code and does exactly the same thing as
the current code does.
On resume the VAIO is stuck in the following place:
<Andrews debug session>
We finish swsusp_save() and a few other functions then we go
hibernate
->platform_finish
->acpi_hibernation_finish
->acpi_leave_sleep_state
->acpi_evaluate_object
and there it dies, in this call:
status = acpi_evaluate_object(NULL, METHOD_NAME__WAK, &arg_list, NULL);
I wonder how your patch caused that?
<debugs further>
OK, it gets to the last statement in acpi_evaluate_object():
return_ACPI_STATUS(status);
but doesn't hit the printk on return to the caller,
acpi_leave_sleep_state().
</Andrews debug session>
Some data points:
This happens only, when the local apic timer is used. With PIT the
resume works fine.
I back ported the full high res stuff to 2.6.20. On 2.6.20 the VAIO
survives that patch.
Can the suspend/resume and ACPI wizards please give some hint how to
track this 100% reproducible wreckage down.
Thanks,
tglx
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-09 19:54 Jinxed VAIO wreckage - current state of affairs Thomas Gleixner
@ 2007-06-09 20:59 ` Rafael J. Wysocki
2007-06-09 21:05 ` Thomas Gleixner
2007-06-10 3:08 ` Andrew Morton
0 siblings, 2 replies; 13+ messages in thread
From: Rafael J. Wysocki @ 2007-06-09 20:59 UTC (permalink / raw)
To: Thomas Gleixner; +Cc: LKML, Andrew Morton, Ingo Molnar, Len Brown
On Saturday, 9 June 2007 21:54, Thomas Gleixner wrote:
> Andrew's jinxed VAIO breaks with the high resolution timer updates in a
> very strange way. Andrew identified the following patch as the culprit:
>
> http://www.tglx.de/projects/hrtimers/2.6.22-rc4/broken-out/clockevents-fix-resume-logic.patch
>
> This makes no sense at all. The patch just moves the timer restart to a
> different (later) place in the code and does exactly the same thing as
> the current code does.
>
> On resume the VAIO is stuck in the following place:
>
> <Andrews debug session>
>
> We finish swsusp_save() and a few other functions then we go
>
> hibernate
> ->platform_finish
> ->acpi_hibernation_finish
> ->acpi_leave_sleep_state
> ->acpi_evaluate_object
>
> and there it dies, in this call:
>
> status = acpi_evaluate_object(NULL, METHOD_NAME__WAK, &arg_list, NULL);
>
> I wonder how your patch caused that?
>
> <debugs further>
>
> OK, it gets to the last statement in acpi_evaluate_object():
>
> return_ACPI_STATUS(status);
>
> but doesn't hit the printk on return to the caller,
> acpi_leave_sleep_state().
>
> </Andrews debug session>
>
> Some data points:
>
> This happens only, when the local apic timer is used. With PIT the
> resume works fine.
>
> I back ported the full high res stuff to 2.6.20. On 2.6.20 the VAIO
> survives that patch.
>
> Can the suspend/resume and ACPI wizards please give some hint how to
> track this 100% reproducible wreckage down.
Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
There are a couple of patches in there that might help in theory, this series
in particular:
swsusp-remove-incorrect-code-from-userc.patch
swsusp-remove-code-duplication-between-diskc-and-userc.patch
swsusp-introduce-restore-platform-operations.patch
swsusp-fix-hibernation-code-ordering.patch
swsusp-remove-code-duplication-between-diskc-and-userc-fix.patch
Greetings,
Rafael
--
"Premature optimization is the root of all evil." - Donald Knuth
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-09 20:59 ` Rafael J. Wysocki
@ 2007-06-09 21:05 ` Thomas Gleixner
2007-06-10 3:08 ` Andrew Morton
1 sibling, 0 replies; 13+ messages in thread
From: Thomas Gleixner @ 2007-06-09 21:05 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: LKML, Andrew Morton, Ingo Molnar, Len Brown
On Sat, 2007-06-09 at 22:59 +0200, Rafael J. Wysocki wrote:
> > Can the suspend/resume and ACPI wizards please give some hint how to
> > track this 100% reproducible wreckage down.
>
> Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
>
> There are a couple of patches in there that might help in theory, this series
> in particular:
>
> swsusp-remove-incorrect-code-from-userc.patch
> swsusp-remove-code-duplication-between-diskc-and-userc.patch
> swsusp-introduce-restore-platform-operations.patch
> swsusp-fix-hibernation-code-ordering.patch
> swsusp-remove-code-duplication-between-diskc-and-userc-fix.patch
I'm working on a hrt version against -mm, so we can figure this out.
tglx
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-09 20:59 ` Rafael J. Wysocki
2007-06-09 21:05 ` Thomas Gleixner
@ 2007-06-10 3:08 ` Andrew Morton
2007-06-11 1:37 ` Thomas Davis
2007-06-15 12:05 ` Thomas Gleixner
1 sibling, 2 replies; 13+ messages in thread
From: Andrew Morton @ 2007-06-10 3:08 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: Thomas Gleixner, LKML, Ingo Molnar, Len Brown
On Sat, 9 Jun 2007 22:59:49 +0200 "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
> Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
People would have heard if it was busted ;)
Have seen occasional hangs in e100 resume-from-RAM, and occasional
all-black-and-dead symptoms after resume-from-RAM, but it seems to work at
least 90% of the time.
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-10 3:08 ` Andrew Morton
@ 2007-06-11 1:37 ` Thomas Davis
2007-06-15 12:05 ` Thomas Gleixner
1 sibling, 0 replies; 13+ messages in thread
From: Thomas Davis @ 2007-06-11 1:37 UTC (permalink / raw)
To: Andrew Morton
Cc: Rafael J. Wysocki, Thomas Gleixner, LKML, Ingo Molnar, Len Brown
Andrew Morton wrote:
> On Sat, 9 Jun 2007 22:59:49 +0200 "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
>
>> Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
>
> People would have heard if it was busted ;)
>
> Have seen occasional hangs in e100 resume-from-RAM, and occasional
> all-black-and-dead symptoms after resume-from-RAM, but it seems to work at
> least 90% of the time.
You doing than I am on my S580p.
if AHCI is loaded, damn thing will not turn off. Goofy part is, another
Sony S80p at work does NOT have this probelm - same bios, same drive
firmware.
Suspend to disk mostly works - sometimes when you return, the screen is
kinda wonky.
Suspend to ram - going in appears to work, coming out it's dead.
thomas
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-10 3:08 ` Andrew Morton
2007-06-11 1:37 ` Thomas Davis
@ 2007-06-15 12:05 ` Thomas Gleixner
2007-06-15 15:13 ` Johannes Weiner
2007-06-15 15:23 ` Randy Dunlap
1 sibling, 2 replies; 13+ messages in thread
From: Thomas Gleixner @ 2007-06-15 12:05 UTC (permalink / raw)
To: Andrew Morton; +Cc: Rafael J. Wysocki, LKML, Ingo Molnar, Len Brown
On Sat, 2007-06-09 at 20:08 -0700, Andrew Morton wrote:
> On Sat, 9 Jun 2007 22:59:49 +0200 "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
>
> > Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
>
> People would have heard if it was busted ;)
Just found a brown paperbag bug in the resume patch logic. Sigh, I was
staring at that code for month without noticing.
Andrew,
can you please test
http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt9.patch
Thanks,
tglx
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-15 12:05 ` Thomas Gleixner
@ 2007-06-15 15:13 ` Johannes Weiner
2007-06-15 15:23 ` Randy Dunlap
1 sibling, 0 replies; 13+ messages in thread
From: Johannes Weiner @ 2007-06-15 15:13 UTC (permalink / raw)
To: Thomas Gleixner; +Cc: Linux Kernel Mailing List
Hi Thomas,
On Fri, Jun 15, 2007 at 02:05:57PM +0200, Thomas Gleixner wrote:
> Just found a brown paperbag bug in the resume patch logic. Sigh, I was
> staring at that code for month without noticing.
>
> Andrew,
>
> can you please test
>
> http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt9.patch
404
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-15 12:05 ` Thomas Gleixner
2007-06-15 15:13 ` Johannes Weiner
@ 2007-06-15 15:23 ` Randy Dunlap
2007-06-15 15:31 ` Thomas Gleixner
1 sibling, 1 reply; 13+ messages in thread
From: Randy Dunlap @ 2007-06-15 15:23 UTC (permalink / raw)
To: Thomas Gleixner
Cc: Andrew Morton, Rafael J. Wysocki, LKML, Ingo Molnar, Len Brown
On Fri, 15 Jun 2007 14:05:57 +0200 Thomas Gleixner wrote:
> On Sat, 2007-06-09 at 20:08 -0700, Andrew Morton wrote:
> > On Sat, 9 Jun 2007 22:59:49 +0200 "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
> >
> > > Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
> >
> > People would have heard if it was busted ;)
>
> Just found a brown paperbag bug in the resume patch logic. Sigh, I was
> staring at that code for month without noticing.
>
> Andrew,
>
> can you please test
>
> http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt9.patch
bad URL, not found.
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-15 15:23 ` Randy Dunlap
@ 2007-06-15 15:31 ` Thomas Gleixner
2007-06-16 6:31 ` Andrew Morton
0 siblings, 1 reply; 13+ messages in thread
From: Thomas Gleixner @ 2007-06-15 15:31 UTC (permalink / raw)
To: Randy Dunlap
Cc: Andrew Morton, Rafael J. Wysocki, LKML, Ingo Molnar, Len Brown
On Fri, 2007-06-15 at 08:23 -0700, Randy Dunlap wrote:
> On Fri, 15 Jun 2007 14:05:57 +0200 Thomas Gleixner wrote:
>
> > On Sat, 2007-06-09 at 20:08 -0700, Andrew Morton wrote:
> > > On Sat, 9 Jun 2007 22:59:49 +0200 "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
> > >
> > > > Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
> > >
> > > People would have heard if it was busted ;)
> >
> > Just found a brown paperbag bug in the resume patch logic. Sigh, I was
> > staring at that code for month without noticing.
> >
> > Andrew,
> >
> > can you please test
> >
> > http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt9.patch
>
> bad URL, not found.
http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt10.patch
tglx
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-15 15:31 ` Thomas Gleixner
@ 2007-06-16 6:31 ` Andrew Morton
2007-06-16 6:47 ` Thomas Gleixner
0 siblings, 1 reply; 13+ messages in thread
From: Andrew Morton @ 2007-06-16 6:31 UTC (permalink / raw)
To: Thomas Gleixner
Cc: Randy Dunlap, Rafael J. Wysocki, LKML, Ingo Molnar, Len Brown
On Fri, 15 Jun 2007 17:31:06 +0200 Thomas Gleixner <tglx@linutronix.de> wrote:
> On Fri, 2007-06-15 at 08:23 -0700, Randy Dunlap wrote:
> > On Fri, 15 Jun 2007 14:05:57 +0200 Thomas Gleixner wrote:
> >
> > > On Sat, 2007-06-09 at 20:08 -0700, Andrew Morton wrote:
> > > > On Sat, 9 Jun 2007 22:59:49 +0200 "Rafael J. Wysocki" <rjw@sisk.pl> wrote:
> > > >
> > > > > Hmm, how's 2.6.22-rc4-mm2 doing on the Vaio?
> > > >
> > > > People would have heard if it was busted ;)
> > >
> > > Just found a brown paperbag bug in the resume patch logic. Sigh, I was
> > > staring at that code for month without noticing.
> > >
> > > Andrew,
> > >
> > > can you please test
> > >
> > > http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt9.patch
> >
> > bad URL, not found.
>
> http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt10.patch
>
That suspends and resumes OK.
What was the bug?
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-16 6:31 ` Andrew Morton
@ 2007-06-16 6:47 ` Thomas Gleixner
2007-06-16 6:53 ` Thomas Gleixner
0 siblings, 1 reply; 13+ messages in thread
From: Thomas Gleixner @ 2007-06-16 6:47 UTC (permalink / raw)
To: Andrew Morton
Cc: Randy Dunlap, Rafael J. Wysocki, LKML, Ingo Molnar, Len Brown
On Fri, 2007-06-15 at 23:31 -0700, Andrew Morton wrote:
> >
> > http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt10.patch
> >
>
> That suspends and resumes OK.
>
> What was the bug?
A stupid state check, which prevented the PIT to be setup again. So the
box got stuck waiting for a timer to expire.
Still I can not explain, why this resulted in this strange "disappear in
the return instruction" behavior.
tglx
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: Jinxed VAIO wreckage - current state of affairs
2007-06-16 6:47 ` Thomas Gleixner
@ 2007-06-16 6:53 ` Thomas Gleixner
2007-06-16 7:19 ` Andrew Morton
0 siblings, 1 reply; 13+ messages in thread
From: Thomas Gleixner @ 2007-06-16 6:53 UTC (permalink / raw)
To: Andrew Morton
Cc: Randy Dunlap, Rafael J. Wysocki, LKML, Ingo Molnar, Len Brown
On Sat, 2007-06-16 at 08:47 +0200, Thomas Gleixner wrote:
> On Fri, 2007-06-15 at 23:31 -0700, Andrew Morton wrote:
> > >
> > > http://www.tglx.de/projects/hrtimers/2.6.22-rc4/patch-2.6.22-rc4-hrt10.patch
> > >
> >
> > That suspends and resumes OK.
> >
> > What was the bug?
>
> A stupid state check, which prevented the PIT to be setup again. So the
> box got stuck waiting for a timer to expire.
>
> Still I can not explain, why this resulted in this strange "disappear in
> the return instruction" behavior.
I put up a fixed patch series against rc4-mm to:
http://www.tglx.de/projects/hrtimers/2.6.22-rc4-mm2/patch-2.6.22-rc4-mm2-hrt3.patches.tar.bz2
tglx
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2007-06-16 7:20 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-06-09 19:54 Jinxed VAIO wreckage - current state of affairs Thomas Gleixner
2007-06-09 20:59 ` Rafael J. Wysocki
2007-06-09 21:05 ` Thomas Gleixner
2007-06-10 3:08 ` Andrew Morton
2007-06-11 1:37 ` Thomas Davis
2007-06-15 12:05 ` Thomas Gleixner
2007-06-15 15:13 ` Johannes Weiner
2007-06-15 15:23 ` Randy Dunlap
2007-06-15 15:31 ` Thomas Gleixner
2007-06-16 6:31 ` Andrew Morton
2007-06-16 6:47 ` Thomas Gleixner
2007-06-16 6:53 ` Thomas Gleixner
2007-06-16 7:19 ` Andrew Morton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).