Bug 8958 - regression: fan doesn't work anymore
Summary: regression: fan doesn't work anymore
Status: CLOSED CODE_FIX
Alias: None
Product: ACPI
Classification: Unclassified
Component: EC (show other bugs)
Hardware: All Linux
: P1 blocking
Assignee: Alexey Starikovskiy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-29 13:47 UTC by Daniel Ritz
Modified: 2008-10-17 21:30 UTC (History)
1 user (show)

See Also:
Kernel Version: 2.6.23-rc4
Subsystem:
Regression: Yes
Bisected commit-id:


Attachments
patch fixing the problem (1.31 KB, text/x-diff)
2007-08-29 13:48 UTC, Daniel Ritz
Details
dmesg of 2.6.23-rc4 with patch (12.37 KB, text/plain)
2007-08-29 13:51 UTC, Daniel Ritz
Details
acpidump output (97.24 KB, text/plain)
2007-08-29 13:52 UTC, Daniel Ritz
Details
Revive acpi_fake_ecdt kernel option (1.33 KB, patch)
2007-08-30 04:59 UTC, Alexey Starikovskiy
Details | Diff
Check if boot_ec was really found (835 bytes, patch)
2007-08-30 05:15 UTC, Alexey Starikovskiy
Details | Diff

Description Daniel Ritz 2007-08-29 13:47:35 UTC
Most recent kernel where this bug did not occur: 2.6.23-rc2 (maybe rc3)
Distribution: ubuntu
Hardware Environment: Toshiba Tecra 8000 Laptop (P3)
Software Environment:
Problem Description:
the fan doesn't work any more...isected it to that one:

cd8c93a4e04dce8f00d1ef3a476aac8bd65ae40b is first bad commit
commit cd8c93a4e04dce8f00d1ef3a476aac8bd65ae40b
Author: Alexey Starikovskiy <astarikovskiy@suse.de>
Date:   Fri Aug 3 17:52:48 2007 -0400

    ACPI: EC: If ECDT is not found, look up EC in DSDT.

    Some ASUS laptops access EC space from device _INI methods, but do not
    provide ECDT for early EC setup. In order to make them function properly,
    there is a need to find EC is DSDT before any _INI is called.

    Similar functionality was turned on by acpi_fake_ecdt=1 command line
    before. Now it is on all the time.

    http://bugzilla.kernel.org/show_bug.cgi?id=8598

    Signed-off-by: Alexey Starikovskiy <astarikovskiy@suse.de>
    Signed-off-by: Len Brown <len.brown@intel.com>
Steps to reproduce: boot (or try to)
Comment 1 Daniel Ritz 2007-08-29 13:48:59 UTC
Created attachment 12622 [details]
patch fixing the problem

from Alexey Starikovskiy <astarikovskiy@suse.de>
Comment 2 Daniel Ritz 2007-08-29 13:51:43 UTC
Created attachment 12623 [details]
dmesg of 2.6.23-rc4 with patch

with the fixup patch because it doesn't boot up to a writeable file system.
(and i don't find my serial cable)
Comment 3 Daniel Ritz 2007-08-29 13:52:48 UTC
Created attachment 12624 [details]
acpidump output
Comment 4 Anonymous Emailer 2007-08-29 16:33:27 UTC
Reply-To: akpm@linux-foundation.org

On Wed, 29 Aug 2007 13:47:36 -0700 (PDT) bugme-daemon@bugzilla.kernel.org wrote:

> http://bugzilla.kernel.org/show_bug.cgi?id=8958
> 
>            Summary: regression: fan doesn't work anymore
>            Product: ACPI
>            Version: 2.5
>      KernelVersion: 2.6.23-rc4
>           Platform: All
>         OS/Version: Linux
>               Tree: Mainline
>             Status: NEW
>           Severity: blocking
>           Priority: P1
>          Component: Other
>         AssignedTo: acpi_other@kernel-bugs.osdl.org
>         ReportedBy: daniel.ritz@gmx.ch
> 
> 
> Most recent kernel where this bug did not occur: 2.6.23-rc2 (maybe rc3)
> Distribution: ubuntu
> Hardware Environment: Toshiba Tecra 8000 Laptop (P3)
> Software Environment:
> Problem Description:
> the fan doesn't work any more...isected it to that one:
> 
> cd8c93a4e04dce8f00d1ef3a476aac8bd65ae40b is first bad commit
> commit cd8c93a4e04dce8f00d1ef3a476aac8bd65ae40b
> Author: Alexey Starikovskiy <astarikovskiy@suse.de>
> Date:   Fri Aug 3 17:52:48 2007 -0400
> 
>     ACPI: EC: If ECDT is not found, look up EC in DSDT.
> 
>     Some ASUS laptops access EC space from device _INI methods, but do not
>     provide ECDT for early EC setup. In order to make them function properly,
>     there is a need to find EC is DSDT before any _INI is called.
> 
>     Similar functionality was turned on by acpi_fake_ecdt=1 command line
>     before. Now it is on all the time.
> 
>     http://bugzilla.kernel.org/show_bug.cgi?id=8598
> 
>     Signed-off-by: Alexey Starikovskiy <astarikovskiy@suse.de>
>     Signed-off-by: Len Brown <len.brown@intel.com>
> Steps to reproduce: boot (or try to)
> 

I'm not sure that I understand this report ;)

Michal, pleas track it as a post-2.6.22 regression.

Daniel, if you have a patch which you think is needed for 2.6.23 can
you please send it?
Comment 5 Michal Piotrowski 2007-08-29 16:37:32 UTC
On 30/08/2007, Andrew Morton <akpm@linux-foundation.org> wrote:
> On Wed, 29 Aug 2007 13:47:36 -0700 (PDT) bugme-daemon@bugzilla.kernel.org
> wrote:
>
> > http://bugzilla.kernel.org/show_bug.cgi?id=8958
> >
> >            Summary: regression: fan doesn't work anymore
> >            Product: ACPI
> >            Version: 2.5
> >      KernelVersion: 2.6.23-rc4
> >           Platform: All
> >         OS/Version: Linux
> >               Tree: Mainline
> >             Status: NEW
> >           Severity: blocking
> >           Priority: P1
> >          Component: Other
> >         AssignedTo: acpi_other@kernel-bugs.osdl.org
> >         ReportedBy: daniel.ritz@gmx.ch
> >
> >
> > Most recent kernel where this bug did not occur: 2.6.23-rc2 (maybe rc3)
> > Distribution: ubuntu
> > Hardware Environment: Toshiba Tecra 8000 Laptop (P3)
> > Software Environment:
> > Problem Description:
> > the fan doesn't work any more...isected it to that one:
> >
> > cd8c93a4e04dce8f00d1ef3a476aac8bd65ae40b is first bad commit
> > commit cd8c93a4e04dce8f00d1ef3a476aac8bd65ae40b
> > Author: Alexey Starikovskiy <astarikovskiy@suse.de>
> > Date:   Fri Aug 3 17:52:48 2007 -0400
> >
> >     ACPI: EC: If ECDT is not found, look up EC in DSDT.
> >
> >     Some ASUS laptops access EC space from device _INI methods, but do not
> >     provide ECDT for early EC setup. In order to make them function
> properly,
> >     there is a need to find EC is DSDT before any _INI is called.
> >
> >     Similar functionality was turned on by acpi_fake_ecdt=1 command line
> >     before. Now it is on all the time.
> >
> >     http://bugzilla.kernel.org/show_bug.cgi?id=8598
> >
> >     Signed-off-by: Alexey Starikovskiy <astarikovskiy@suse.de>
> >     Signed-off-by: Len Brown <len.brown@intel.com>
> > Steps to reproduce: boot (or try to)
> >
>
> I'm not sure that I understand this report ;)
>
> Michal, pleas track it as a post-2.6.22 regression.

I'm tracking this already

Subject         : the fan doesn't work any more
References      : http://lkml.org/lkml/2007/8/28/359
Last known good : ?
Submitter       : Daniel Ritz <daniel.ritz-ml@swissonline.ch>
Caused-By       : Alexey Starikovskiy <astarikovskiy@suse.de>
                  commit cd8c93a4e04dce8f00d1ef3a476aac8bd65ae40b
Handled-By      : Alexey Starikovskiy <astarikovskiy@suse.de>
Patch           : http://lkml.org/lkml/2007/8/29/15
Status          : patch was suggested

>
> Daniel, if you have a patch which you think is needed for 2.6.23 can
> you please send it?
>

Status          : patch was suggested

Regards,
Michal
Comment 6 Alexey Starikovskiy 2007-08-30 04:59:03 UTC
Created attachment 12626 [details]
Revive acpi_fake_ecdt kernel option

Daniel,
Could you please check if the following patch helps as well?
Also, I don't understand why plain -rc4 breaks for you, as your system does not have EC at all, and acpi_get_devices() should just return NOT_FOUND error code.
Could you please make a screen shot of hanged system if you cant get a dmesg from it?
Comment 7 Alexey Starikovskiy 2007-08-30 05:15:42 UTC
Created attachment 12627 [details]
Check if boot_ec was really found

I was wrong about acpi_get_devices() -- it happily return AE_OK if no device was found.
Please check that this minimal patch makes your system boot as before.
Comment 8 Daniel Ritz 2007-08-30 10:06:43 UTC
yes, that one (attachment 12627 [details]) works. and it makes perfect sense.
thanks. please push upstream.
Comment 9 Len Brown 2007-08-31 13:30:11 UTC
2d8348b429b4ae5cc47449c787881221fe43af4b
ACPI: EC: Check if boot_ec was really found in DSDT

shipped in 2.6.23-rc4-git3
closed.

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