linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* RE: 2.6.16-rc5: known regressions
@ 2006-02-27  9:04 Yu, Luming
  2006-03-03  2:59 ` Sanjoy Mahajan
  0 siblings, 1 reply; 18+ messages in thread
From: Yu, Luming @ 2006-02-27  9:04 UTC (permalink / raw)
  To: linux-kernel, Linus Torvalds, Andrew Morton
  Cc: Tom Seeley, Dave Jones, Jiri Slaby, michael, mchehab,
	v4l-dvb-maintainer, video4linux-list, Brian Marete, Ryan Phillips,
	gregkh, linux-usb-devel, Sanjoy Mahajan, Brown, Len, linux-acpi,
	Mark Lord, Randy Dunlap, jgarzik, linux-ide, Duncan,
	Pavlik Vojtech, linux-input, Meelis Roos

>Subject    : S3 sleep hangs the second time - 600X
>References : http://bugzilla.kernel.org/show_bug.cgi?id=5989
>Submitter  : Sanjoy Mahajan <sanjoy@mrao.cam.ac.uk>
>Handled-By : Luming Yu <luming.yu@intel.com>
>Status     : is being debugged,
>             we might want to change the default back for 2.6.16:
>             http://lkml.org/lkml/2006/2/25/101
>

Accordint to bug report, the BIOS DSDT is modified.
I don't know how these changes affect the results
of suspend/resume. But, it is clear this is NOT right approach 
to fix problem. Hence, I need the testing report with 
un-modified DSDT on TP 600X, bios 1.11.

--Luming

^ permalink raw reply	[flat|nested] 18+ messages in thread
* RE: 2.6.16-rc5: known regressions
@ 2006-03-03  4:46 Yu, Luming
  0 siblings, 0 replies; 18+ messages in thread
From: Yu, Luming @ 2006-03-03  4:46 UTC (permalink / raw)
  To: Sanjoy Mahajan
  Cc: linux-kernel, Linus Torvalds, Andrew Morton, Tom Seeley,
	Dave Jones, Jiri Slaby, michael, mchehab, v4l-dvb-maintainer,
	video4linux-list, Brian Marete, Ryan Phillips, gregkh,
	linux-usb-devel, Brown, Len, linux-acpi, Mark Lord, Randy Dunlap,
	jgarzik, linux-ide, Duncan, Pavlik Vojtech, linux-input,
	Meelis Roos


>
>>> Subject    : S3 sleep hangs the second time - 600X
>>> References : http://bugzilla.kernel.org/show_bug.cgi?id=5989
>
>From: "Yu, Luming" <luming.yu@intel.com>
>> According to bug report, the BIOS DSDT is modified.  I don't know
>> how these changes affect the results of suspend/resume. But, it is
>> clear this is NOT right approach to fix problem. Hence, I need the
>> testing report with un-modified DSDT on TP 600X, bios 1.11.
>
>I'll try it, although I don't think I'll get any data on the problem.
>The unmodified DSDT (bios 1.11) lacks an S3 sleep object, so I had to
>modify the DSDT even to get S3 to sleep at all.  See
><http://bugzilla.kernel.org/show_bug.cgi?id=3534> for that discussion.
>In additional comment #4 there (2004-10-14), you said:
>
>  The root cause of [the missing S3 object] failure is that linux is
>  using element in
>
>  const char      *acpi_gbl_sleep_state_names[ACPI_S_STATE_COUNT] =
>  {
>	  "\_S0_",
>	  "\_S1_",
>	  "\_S2_",
>	  "\_S3_",
>	  "\_S4_",
>	  "\_S5_"
>  };
>
>  to call acpi_get_sleep_type_data, but your box define _S3 under the
>  device PNP0A03. So, the evaluating \_S3 will fail.
>
>  The workaround in DSDT is to change _S3 to \_S3_ .
>  We can fix it in acpi driver soon.

Hmm, this conclusion seems to be wrong. at that time, I said it too
early.  The real problem is this, if your box support S3, the _S3 method
should return from ELSE-statement which return package
{0x01,0x01,0x00,0x00}.

If you still use this
http://bugzilla.kernel.org/show_bug.cgi?id=3534#c10 to
override your DSDT, which bypass the testing and blindly assume BIOS or
platform
do support S3, then I suggest you to retest, and post dmesg with
UN-modified BIOS.

Thanks,
Luming


    Method (_S3, 0, NotSerialized)
    {
        If (BXPT)
        {
            Return (Package (0x04)
            {
                0x06,
                0x06,
                0x00,
                0x00
            })
        }
        Else
        {
            Return (Package (0x04)
            {
                0x01,
                0x01,
                0x00,
                0x00
            })
        }
    }

^ permalink raw reply	[flat|nested] 18+ messages in thread

end of thread, other threads:[~2006-03-08 11:13 UTC | newest]

Thread overview: 18+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <Pine.LNX.4.64.0602262122000.22647@g5.osdl.org>
2006-02-27  6:13 ` 2.6.16-rc5: known regressions Adrian Bunk
2006-02-27  6:26   ` Ryan Phillips
2006-02-27  6:39     ` Vojtech Pavlik
2006-02-27  9:14       ` 2.6.16-rc5: known regressions (ps2 mouse/keyboard issues) Duncan
2006-02-27  6:54   ` 2.6.16-rc5: known regressions Jeff Garzik
2006-02-27  7:08     ` Adrian Bunk
2006-02-28  9:40       ` Jens Axboe
2006-03-01  0:17         ` Randy.Dunlap
2006-03-04 13:18           ` Adrian Bunk
2006-02-27 13:36   ` Mark Lord
2006-02-27 14:09   ` Pavel Machek
2006-03-02 14:00   ` [v4l-dvb-maintainer] " Mauro Carvalho Chehab
2006-03-08 11:13     ` Brian Marete
2006-02-27  9:04 Yu, Luming
2006-03-03  2:59 ` Sanjoy Mahajan
2006-03-03 16:51   ` Matthew Garrett
2006-03-03 21:04     ` Sanjoy Mahajan
  -- strict thread matches above, loose matches on Subject: below --
2006-03-03  4:46 Yu, Luming

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).