Bug 11930
| Summary: | 2.6.27 regression - Battery not detected when EC switches to poll mode | ||
|---|---|---|---|
| Product: | ACPI | Reporter: | churchmice (firefoxelectric) |
| Component: | EC | Assignee: | ykzhao (yakui.zhao) |
| Status: | CLOSED PATCH_ALREADY_AVAILABLE | ||
| Severity: | normal | CC: | acpi-bugzilla, firefoxelectric |
| Priority: | P1 | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Kernel Version: | 2.6.27-r2 | Subsystem: | |
| Regression: | Yes | Bisected commit-id: | |
| Attachments: |
acpi dump
dmesg of 2.6.27-r2 dmesg of tuxonice-2.6.25-r7 ec.c from tuxonice-2.6.25-r7 ec.c from gentoo-sources-2.6.27-r2 output of lspci -vv dmesg of 2.6.27-r2 with ec.c borrowed from tuxonice-2.6.25.-r7 dmesg of patched ec.c dmesg from the ec.c patch provided by Alexey Starikovskiy |
||
|
Description
churchmice
2008-11-01 02:27:52 UTC
Created attachment 18565 [details]
acpi dump
Created attachment 18566 [details]
dmesg of 2.6.27-r2
Created attachment 18567 [details]
dmesg of tuxonice-2.6.25-r7
Created attachment 18568 [details]
ec.c from tuxonice-2.6.25-r7
Created attachment 18569 [details]
ec.c from gentoo-sources-2.6.27-r2
Created attachment 18570 [details]
output of lspci -vv
Created attachment 18571 [details]
dmesg of 2.6.27-r2 with ec.c borrowed from tuxonice-2.6.25.-r7
Hi, Churchmice
On your laptops there exists the following warning message:
>ACPI: EC: GPE storm detected, disabling EC GPE
Will you please try the latest kernel(2.6.28-rc2) and see whether the problem still exists?
Thanks.
(In reply to comment #8) > Hi, Churchmice > On your laptops there exists the following warning message: > >ACPI: EC: GPE storm detected, disabling EC GPE > Will you please try the latest kernel(2.6.28-rc2) and see whether the > problem still exists? > Thanks. > vanilla 2.6.28-rc2 fixed this problem. Thank you for your work. Thanks for the test. Will you please try the patch set in http://bugzilla.kernel.org/show_bug.cgi?id=11930#C1,2,3,4 and see whether the problem still exists? As there is another problem about EC driver on the 2.6.28-rc2 kernel, it will be great if you can try the patch set. Thanks. (In reply to comment #10) > Thanks for the test. > Will you please try the patch set in > http://bugzilla.kernel.org/show_bug.cgi?id=11930#C1,2,3,4 and see whether the > problem still exists? > As there is another problem about EC driver on the 2.6.28-rc2 kernel, it > will be great if you can try the patch set. > Thanks. > There is some problem with the link you provided above. I can not find the patch, will you please double check the link? I am patch the 2.6.27-4 kernel with the ec.c from 2.6.28-rc2, and that works great. PS: From the name i can tell you are chinese, so do I. It is my fault. And I am sorry that I give the incorrect address in comment #10. The patch set is in http://bugzilla.kernel.org/show_bug.cgi?id=11896#C1,2,3,4 Thanks. (In reply to comment #12) > It is my fault. And I am sorry that I give the incorrect address in comment > #10. > The patch set is in > http://bugzilla.kernel.org/show_bug.cgi?id=11896#C1,2,3,4 > Thanks. > > > I applied the four patches you created one by one, recompiled the kernel ( still using 2.6.27-r2, only ec.c is patched). After reboot, it keeps popping noisy information " DB->get: method not permitted before handle's open method The battery is detected correctly, only with so much annoying messages. Below is my dmesg. Created attachment 18636 [details]
dmesg of patched ec.c
(In reply to comment #12) > It is my fault. And I am sorry that I give the incorrect address in comment > #10. > The patch set is in > http://bugzilla.kernel.org/show_bug.cgi?id=11896#C1,2,3,4 > Thanks. > > > sorry, the "DB->get:method not permitted before handle's open method" is another bug which is not regarded for this bug report. I tried tuxonice-sources-2.6.25-r7,and the same warning message reappears. I remember i have updated the system, so it must be brought by one of the updated package, i will figure which it is. As far as this bug is concerned, the patch you provided works fine with my machine. Great job. (In reply to comment #12) > It is my fault. And I am sorry that I give the incorrect address in comment > #10. > The patch set is in > http://bugzilla.kernel.org/show_bug.cgi?id=11896#C1,2,3,4 > Thanks. > > > sorry, the "DB->get:method not permitted before handle's open method" is another bug which is not regarded for this bug report. I tried tuxonice-sources-2.6.25-r7,and the same warning message reappears. I remember i have updated the system, so it must be brought by one of the updated package, i will figure which it is. As far as this bug is concerned, the patch you provided works fine with my machine. Great job. Confirmed. The "DB->get:method not permitted before handle's open method" is due to sys-libs/ncurses-5.7. After downloaded to sys-libs/ncurses-5.6-r2, the annonying messages disappeared. In a word, the patched worked, at least on my machine. I am now trying from Alexey Starikovskiy, hope it works also. (In reply to comment #12) > It is my fault. And I am sorry that I give the incorrect address in comment > #10. > The patch set is in > http://bugzilla.kernel.org/show_bug.cgi?id=11896#C1,2,3,4 > Thanks. > > > The patch from Alexey Starikovskiy works too, but with some extra delay ( approximately 5s),and some annoyning messages. See the output of dmesg for detailed illustration. What shall I do next ? Created attachment 18638 [details]
dmesg from the ec.c patch provided by Alexey Starikovskiy
per comment #9 > vanilla 2.6.28-rc2 fixed this problem. > Thank you for your work. this bug report is closed. re: tests for patches in bug 11986, please comment in bug 11986 > ACPI: EC: non-query interrupt received, switching to interrupt mode > __ratelimit: 40 callbacks suppressed this is not good and should be pointed out in that bug report. |