Bug 599679 - [REGRESSION] acpi-ec-add-delay-before-write patch disrupts ACPI events Clevo M720R
Summary: [REGRESSION] acpi-ec-add-delay-before-write patch disrupts ACPI events Clevo ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: 19
Hardware: All
OS: Linux
low
high
Target Milestone: ---
Assignee: Matthew Garrett
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-06-03 18:18 UTC by James
Modified: 2013-04-05 15:40 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-04-05 15:40:55 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Linux Kernel 14733 0 None None None Never

Description James 2010-06-03 18:18:51 UTC
The Fedora 13 kernels (2.6.33.3-85 through 2.6.33.5-112) don't seem to generate ACPI events in response to pressing the sleep button or closing the lid. This worked OK with the Fedora 12 series of kernels, in the versions with the multibyte EC access patches applied. (This is somewhat odd since these patches were presumably merged upstream. However, I've got another kernel here which has a patchset based off vanilla 2.6.33. This does work OK without any explicit multibyte EC patches.)

Comment 1 James 2010-06-03 20:00:37 UTC
OK, this is actually caused by acpi-ec-add-delay-before-write.patch (a kernel built without it works fine). This is on a Clevo M720R. I'll add a note to this effect on https://bugzilla.kernel.org/show_bug.cgi?id=14733 .

Comment 2 James 2010-07-07 08:39:47 UTC
Still present in kernel-2.6.33.6-142.rc1.fc13.x86_64. Is any further info needed to help make this patch work without disrupting operation on my notebook?

Comment 3 James 2010-07-09 16:00:45 UTC
Yay, appears to be working again with kernel-2.6.33.6-147.fc13.x86_64. Please don't break it in future! ;)

Comment 4 James 2010-07-09 16:06:20 UTC
Ack! Sorry, guys --- false closure on this. kernel-2.6.33.6-147.fc13.x86_64 does NOT fix this issue. What happened is that I warm-booted this kernel from one that works properly, and the sleep button worked.

Cold-booting any current Fedora kernel *after removing the battery briefly* results in a non-functioning lid and sleep button. I still need to drop the delay patch to get things working.

Please alter status as appropriate.

Comment 5 James 2010-07-09 18:53:27 UTC
The same with kernel-2.6.33.6-147.fc13.x86_64. On closer inspection, it seems the disruption to ACPI is more widespread than I initially reported. As well as no lid an sleep button events, changes to the power supply state (e.g. plugging in/removing AC adapter) are not detected, either.

Comment 6 James 2010-07-23 14:54:04 UTC
Still present with kernel-2.6.34.1-29.fc13.x86_64.

Comment 7 James 2010-09-16 05:37:34 UTC
Still present with kernel-2.6.34.7-56.fc13.x86_64.

Comment 8 James 2010-10-21 09:38:25 UTC
Updating release.

Still present in kernel-2.6.35.6-46.fc14.

Comment 9 James 2010-11-08 08:56:52 UTC
The disruptive patch is still there in kernel-2.6.36-1.fc15, please re-work it so the kernel is both bootable and keeps events on other notebooks.

Comment 10 Martin Belohorka 2011-03-13 17:28:57 UTC
Hi James!

I have just finished the same debugging that i guess that you concluded.
I am running on a Clevo M767TU

Comment 11 Martin Belohorka 2011-05-01 19:16:21 UTC
Still present in 2.6.35.12-90.fc14

Comment 12 James 2011-05-01 19:39:24 UTC
I guess the trouble with this patch is that it's the lesser of two evils... without it, some machines refuse to boot. I've added the tracker to the kernel.org Bugzilla entry; the devs are aware of our issue, which I think is why the patch has not gone upstream.

Comment 13 James 2011-08-25 23:36:00 UTC
Disruptive patch still present in kernel-2.6.40.3-0.fc15.x86_64

Comment 14 Dave Jones 2011-08-26 15:54:21 UTC
I just removed it from f16. For f15, it's a tough call because for people who currently have systems that boot that need it, we would regress for them.

If we can figure out a better approach in f16, we'll backport and then drop this in 15 too.

Comment 15 Bartosz Wierucki 2012-01-21 10:27:21 UTC
I upgraded from F15 to F16 some time ago and since that moment my laptop (Acer Aspire 5735Z) has been experiencing problems, unless I appended acpi=off to boot parameters.

After few minuts of use ACPI refuses to work. Neither AC/DC nor battery are recognized, backlight cannot be changed, and dmesg shows a flood of messages containing following information:

ACPI: EC: input buffer is not empty, aborting transaction
ACPI Exception: AE_TIME, Returned by Handler for [EmbeddedControl] 
ACPI Error: Method parse/execution failed 

Reboot doesn't work and also after powering it off my laptop doesn't want to boot unless I remove the battery for a moment before trying to power it on.

It took me some time to find out if there is a way for my laptop to run without having to have the ACPI system disabled. I was trying out different boot parameters, different kernels from different distributions and then I noticed this bugreport.

I applied changes from this patch to the new kernels 3.1.8 and now 3.1.9 and recompiled them. Since I moved to my recompiled kernel (the only change was this patch) I haven't experienced any of the abovementioned problems anymore. 

Is there a chance of readding changes introduced with this patch to Fedora kernels so that they both work for my Acer and don't generate problems with other equipment like Clevo?

I know there are other people affected by this bug in other systems so the most benefits could be achieved by fixing this issue upstream but I don't know whom it should be reported to.

Comment 16 Fedora End Of Life 2013-04-03 20:10:09 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 19 development cycle.
Changing version to '19'.

(As we did not run this process for some time, it could affect also pre-Fedora 19 development
cycle bugs. We are very sorry. It will help us with cleanup during Fedora 19 End Of Life. Thank you.)

More information and reason for this action is here:
https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora19


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