Bug 15344 - hp_wmi: reading ac_adapter/*/status causes WMI to change backlight level -- caused by module autoload (1caab3c1a90be3aa4ec3599409d8fe044b077478)
Summary: hp_wmi: reading ac_adapter/*/status causes WMI to change backlight level -- c...
Status: CLOSED UNREPRODUCIBLE
Alias: None
Product: Drivers
Classification: Unclassified
Component: Platform (show other bugs)
Hardware: All Linux
: P1 normal
Assignee: Matthew Garrett
URL:
Keywords:
Depends on:
Blocks: 14230 56331
  Show dependency tree
 
Reported: 2010-02-17 18:24 UTC by bugzillakernelorg
Modified: 2013-04-09 06:23 UTC (History)
7 users (show)

See Also:
Kernel Version: 2.6.35-6-generic
Subsystem:
Regression: Yes
Bisected commit-id:


Attachments
lspci -vxxx output (11.01 KB, text/plain)
2010-02-21 17:12 UTC, bugzillakernelorg
Details
acpidump output (205.25 KB, text/plain)
2010-02-21 17:13 UTC, bugzillakernelorg
Details
grep . /sys/bus/acpi/drivers/*/*/path output (1.34 KB, text/plain)
2010-02-22 17:04 UTC, bugzillakernelorg
Details
Test patch (453 bytes, patch)
2010-03-10 21:24 UTC, Matthew Garrett
Details | Diff

Description bugzillakernelorg 2010-02-17 18:24:48 UTC
$ cat /proc/acpi/ac_adapter/ADP1/state 
state:                   on-line

lowers the backlight by one level every time its done

using ubuntu prepared kernels from upstream dated versions.
Linux krang 2.6.33-020633rc8-generic #020633rc8 SMP Sat Feb 13 10:09:50 UTC 2010 x86_64 GNU/Linux
Comment 1 ykzhao 2010-02-21 07:00:13 UTC
Will you please attach the output of acpidump, lspci -vxxx on your box?

thanks.
Comment 2 bugzillakernelorg 2010-02-21 17:12:29 UTC
Created attachment 25143 [details]
lspci -vxxx output

these logs aren't from a live running 2.6.33 as reported in the bug; they're from 2.6.31-20-generic (ubuntu)
Comment 3 bugzillakernelorg 2010-02-21 17:13:13 UTC
Created attachment 25144 [details]
acpidump output
Comment 4 Zhang Rui 2010-02-22 06:12:21 UTC
please attach the output of "grep . /sys/bus/acpi/drivers/*/*/path".
Comment 5 bugzillakernelorg 2010-02-22 17:04:01 UTC
Created attachment 25165 [details]
grep . /sys/bus/acpi/drivers/*/*/path output

same as before, from 2.6.31-20-generic
Comment 6 Len Brown 2010-02-23 02:50:18 UTC
this bug is marked as a regression.
please indicate the latest kernel version that worked properly
dmesg from that kernel may be helpful as well.
Comment 7 bugzillakernelorg 2010-02-23 03:12:51 UTC
i started doing just that when i saw the metablock mail; 2.6.32.8 doesn't do it, 2.6.33-rc1 does
Comment 8 Zhang Rui 2010-02-23 07:17:27 UTC
this regression is probably introduced by commit 1caab3c1a90be3aa4ec3599409d8fe044b077478
could you please verify if the problem still exists after this patch is reverted?
Comment 9 Zhang Rui 2010-02-24 03:08:30 UTC
or you can just blacklist the wmi.ko driver and see if the problem still exists.

BTW: in order to blacklist the wmi.ko, you need to
1.open /etc/modprobe.d/blacklist.conf file
2.add this line in the file: blacklist wmi.ko
3.reboot.
Comment 10 bugzillakernelorg 2010-02-24 06:36:49 UTC
yea; that was it, it didn't revert cleanly but acpi_wmi_init returning -ENODEV
did the trick

oof i only saw the last comment when i tried to post this reply; that sure would have been easier than what i did :> (but at least i know the process and have .33 oackages i can actually use now)

thanks for your time
Comment 11 Zhang Rui 2010-02-24 07:05:23 UTC
It seems that it's this piece of AML code in _PSR method that changes the backlight.

                            If (Local1)
                            {
                                Store (0x03, ^^^^^WMID.EVID)
                                Store (Zero, ^^^^^WMID.EVDT)
                                Notify (^^^^^WMID, 0x80)
                                Store (One, ACST)
                            }
                            Else
                            {
                                Store (Zero, ACST)
                            }

Carlos,
do you know what it means?
As the notification 0x80 is not handled by ACPI wmi driver, do you think 
                                Store (0x03, ^^^^^WMID.EVID)
                                Store (Zero, ^^^^^WMID.EVDT)
changes the backlight?

do you think this is a AML code problem and a kernel bug?
Comment 12 Len Brown 2010-03-02 02:53:36 UTC
This appears to be specific to when the wmi driver is loaded.
Indeed, I expect that it still happens when the ACPI "video"
driver is not present, is that true?
Comment 13 bugzillakernelorg 2010-03-04 02:15:47 UTC
i couldn't remove the acpi video driver to test; as it was already in use, and the only module with wmi in the name on this computer is hp_wmi, and it indeed stops doing it when its removed (as if i had did what i did in comment #10)
Comment 14 bugzillakernelorg 2010-03-05 21:56:51 UTC
as part of trying to re-enliven my card reader without a reboot; i modprobe'd hp_wmi on 2.6.31-20-generic (ubuntu) and it exhibited the same backlight behaviour as reported on .33
Comment 15 Zhang Rui 2010-03-10 05:50:35 UTC
Carlos and Matthew,
any comments on this?
Comment 16 bugzillakernelorg 2010-03-10 19:08:18 UTC
my touchpad disable key and brightness keys haven't worked since comment #14 :\
Comment 17 Carlos Corbacho 2010-03-10 21:04:13 UTC
Can I get some clarification here, because I'm having trouble parsing the previous comments - does this bug happen when hp-wmi is unloaded but with wmi still loaded? Because blacklisting wmi also stops hp-wmi loading, so doesn't tell me anything useful.

I'd be more suspicious that it's a bug with hp-wmi doing the wrong thing with the event being triggered here - EVID and EVDT seem to be being used only for _WED (which is used to provide additional information for events).
Comment 18 Matthew Garrett 2010-03-10 21:08:37 UTC
Huh. Event 3 used to mean "Brightness down". Let me look into this some more.
Comment 19 Matthew Garrett 2010-03-10 21:09:20 UTC
Ah, Carlos is entirely correct - the use of the same table for two different purposes has caught me out. I'll work on a patch.
Comment 20 Matthew Garrett 2010-03-10 21:24:48 UTC
Created attachment 25461 [details]
Test patch

This ought to fix it...
Comment 21 Zhang Rui 2010-03-11 08:08:36 UTC
do you mean the hp-wmi driver should not handle the brightness up/down events?

I know little about wmi/hp-wmi, is it possible that a platform doesn't follow ACPI video extension but routes the hotkey events via the wmi stuff?

Note that in this BIOS, the notification is sent from the _PSR method, which doesn't make sense to me.
We should send such events when AC is plugged/unplugged, but never when reading the AC status, right?
Comment 22 Matthew Garrett 2010-03-11 13:47:27 UTC
The brightness up/down events don't appear to be sent by any hardware I can find. I believe that it was specific to legacy machines that didn't expose the backlight via ACPI, but in that case we can't do anything with the events anyway. In this case, the hardware is actually sending event 0x4 with extended information of 0x3, which is separate but difficult to separate due to the way I implemented hp-wmi originally. I've no idea why the ahrdware is sending an event in that case, but we should be able to just ignore it.
Comment 23 Zhang Rui 2010-03-12 02:33:16 UTC
okay, I see.

bugzillakernelorg@lez.ath.cx

please re-open this bug if the patch in comment #20 doesn't fix the problem for you.
Comment 24 bugzillakernelorg 2010-03-13 00:48:12 UTC
i've tried the patch; but #16 is still happening, and it happens even with the "stable" kernel (2.6.31-20-generic) i normally use, after i modprobe'd hp_wmi on that same kernel, i literally loaded it once and it has not worked since

the other soft buttons that used to work correctly now are malfunctioning as well (like Fn+F4 sending Meta+L???), if anyone has any pointers on how to restore the original operation i would appreciate it, as it survives reboots and i have no idea where it could have kept record of this change
Comment 25 bugzillakernelorg 2010-03-13 01:00:45 UTC
agh sorry for the noise but i forgot to put this in my last post; this is what dmesg says for my touchpad disable button since it broke

[  111.745759] atkbd serio0: Unknown key pressed (translated set 2, code 0xd8 on isa0060/serio0).
[  111.745765] atkbd serio0: Use 'setkeycodes e058 <keycode>' to make it known.
[  111.752206] atkbd serio0: Unknown key released (translated set 2, code 0xd8 on isa0060/serio0).
[  111.752211] atkbd serio0: Use 'setkeycodes e058 <keycode>' to make it known.

the other keys just seem dead or send weird stuff (as mentioned for Fn+F4)
Comment 26 bugzillakernelorg 2010-03-20 05:54:34 UTC
i really miss my touchpad disable button; if anyone has any idea of what i can do to restore it, or where to look to do so, i'd appreciate it
Comment 27 Zhang Rui 2010-03-22 08:01:11 UTC
ping Matthew...
Comment 28 Matthew Garrett 2010-03-23 15:51:55 UTC
Try the "reset to defaults" option in your BIOS.
Comment 29 bugzillakernelorg 2010-03-25 01:42:49 UTC
thanks for the suggestion, i tried that to no avail; i'm on freenode and can be contacted at this email address if anyone wants to help troubleshoot; i can't rule out my distro but i have no idea where to even start looking
Comment 30 bugzillakernelorg 2010-05-13 12:42:25 UTC
hope this isn't bad form; copied comment i made on another bug (https://bugzilla.kernel.org/show_bug.cgi?id=15895)

...

> further, my brightness keys have been broken for a while, sending wrong
> keysyms; and have been since:
> https://bugzilla.kernel.org/show_bug.cgi?id=15344, and as alluded to in the
> first post
> 
> i'd _really_ like to fix my keysyms, before the hp_wmi thing it was sending
> the
> right keysyms for all the extra keys, it hasn't since; is this sort of thing
> permanent? i also downgraded my bios checked functionality (was still broke)
> and then upgraded it again in the spirit of comment #28 on #15344 after that
> suggestion failed, and that also failed.
> 
> i'm glad to report and troubleshoot bugs but if its going to break my laptop
> i'll be sad :[

which was just yesterday; which means my laptop is still dinged
Comment 31 bugzillakernelorg 2010-06-01 23:27:27 UTC
(In reply to comment #28)
> Try the "reset to defaults" option in your BIOS.

I got around to seeing what happens in windows wrt the now broken keys; they still work as expected.
Comment 32 bugzillakernelorg 2010-07-04 21:20:28 UTC
this is a weird one; its doing it again with 2.6.35-6-generic (from kernel-ppa/ppa ppa for ubuntu)

my touchpad is still broke too :[
Comment 33 bugzillakernelorg 2010-09-21 03:06:18 UTC
upgraded to ubuntu 10.10 beta which includes this kernel; problem still exists
Comment 34 Thomas Renninger 2010-09-23 08:39:07 UTC
> my touchpad is still broke too :[
I have a patch series pending which should fix that. Hopefully I can send the stuff soon, possibly the next days.
Comment 35 Florian Mickler 2010-10-07 20:07:05 UTC
What is the ETA for these? 2.6.37-rc1 or 2.6.38?
Comment 36 Florian Mickler 2011-02-05 17:41:14 UTC
Is this still an issue in 2.6.37?

Thomas, did your patch series land? Do you have a pointer to it?
Comment 37 Florian Mickler 2011-03-06 00:27:08 UTC
Closing this now as unreproducible.

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