Fn key control of backlight broken after upgrade to 10.10

Bug #658198 reported by Michael Doube on 2010-10-11
38
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Linux
Fix Released
Medium
linux (Fedora)
Won't Fix
Medium
linux (Ubuntu)
Undecided
Tim Gardner
Maverick
Undecided
Tim Gardner
Natty
Undecided
Tim Gardner

Bug Description

After an upgrade to Ubuntu 10.10 the previously natively-supported Fn key control of backlight brightness is gone. I can control backlight with xbacklight, as before.

Models of notebook affected include Sony Vaio SZ4 and SZ6.

Git bisect locates the bug to commit bcbfc24ead6935714a001ab6f1f725763075f98c
in Maverick's kernel branch
http://kernel.ubuntu.com/git?p=ubuntu/ubuntu-maverick.git;a=summary

This bug has been fixed in Linus' mainline, commit de391d12500ede13faa7ecadbe900a8de1f76cfa

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: acpi-support 0.137
ProcVersionSignature: Ubuntu 2.6.35-22.33-generic 2.6.35.4
Uname: Linux 2.6.35-22-generic x86_64
Architecture: amd64
Date: Mon Oct 11 11:21:39 2010
InstallationMedia: Ubuntu 10.04 "Lucid Lynx" - Alpha amd64 (20100412)
ProcEnviron:
 LANG=en_GB.utf8
 SHELL=/bin/bash
SourcePackage: acpi-support
---
AlsaVersion: Advanced Linux Sound Architecture Driver Version 1.0.23.
AplayDevices:
 **** List of PLAYBACK Hardware Devices ****
 card 0: Intel [HDA Intel], device 0: STAC92xx Analog [STAC92xx Analog]
   Subdevices: 1/1
   Subdevice #0: subdevice #0
Architecture: amd64
ArecordDevices:
 **** List of CAPTURE Hardware Devices ****
 card 0: Intel [HDA Intel], device 0: STAC92xx Analog [STAC92xx Analog]
   Subdevices: 1/1
   Subdevice #0: subdevice #0
AudioDevicesInUse:
 USER PID ACCESS COMMAND
 /dev/snd/controlC0: mdoube 1739 F.... pulseaudio
CRDA: Error: [Errno 2] No such file or directory
Card0.Amixer.info:
 Card hw:0 'Intel'/'HDA Intel at 0xfc500000 irq 48'
   Mixer name : 'SigmaTel STAC9872AK'
   Components : 'HDA:83847662,104d1e00,00100201 HDA:14f12c06,104d1700,00100000'
   Controls : 11
   Simple ctrls : 7
DistroRelease: Ubuntu 10.10
HibernationDevice: RESUME=UUID=a017c7e4-4a29-4ba1-bfea-52cd609781d8
InstallationMedia: Ubuntu 10.04 "Lucid Lynx" - Alpha amd64 (20100412)
MachineType: Sony Corporation VGN-SZ650N
Package: linux (not installed)
PccardctlIdent:
 Socket 0:
   no product info available
PccardctlStatus:
 Socket 0:
   no card
ProcCmdLine: BOOT_IMAGE=/boot/vmlinuz-2.6.35-22-generic root=UUID=31cf7381-5708-4104-8d9e-a2eaaec7cbe7 ro quiet splash
ProcEnviron:
 PATH=(custom, no user)
 LANG=en_GB.utf8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.35-22.33-generic 2.6.35.4
Regression: Yes
RelatedPackageVersions: linux-firmware 1.38
Reproducible: Yes
RfKill:
 0: phy0: Wireless LAN
  Soft blocked: no
  Hard blocked: no
Tags: maverick kernel-graphics regression-release needs-upstream-testing
Uname: Linux 2.6.35-22-generic x86_64
UserGroups:

dmi.bios.date: 04/04/2008
dmi.bios.vendor: Phoenix Technologies LTD
dmi.bios.version: R0122S5
dmi.board.asset.tag: N/A
dmi.board.name: VAIO
dmi.board.vendor: Sony Corporation
dmi.board.version: N/A
dmi.chassis.type: 10
dmi.chassis.vendor: Sony Corporation
dmi.chassis.version: N/A
dmi.modalias: dmi:bvnPhoenixTechnologiesLTD:bvrR0122S5:bd04/04/2008:svnSonyCorporation:pnVGN-SZ650N:pvrJ002UR2M:rvnSonyCorporation:rnVAIO:rvrN/A:cvnSonyCorporation:ct10:cvrN/A:
dmi.product.name: VGN-SZ650N
dmi.product.version: J002UR2M
dmi.sys.vendor: Sony Corporation

---
Architecture: amd64
DistroRelease: Ubuntu 10.10
InstallationMedia: Ubuntu 10.04 "Lucid Lynx" - Alpha amd64 (20100412)
Package: linux (not installed)
ProcEnviron:
 PATH=(custom, no user)
 LANG=en_GB.utf8
 SHELL=/bin/bash
Tags: maverick
Uname: Linux 2.6.36-rc7-linusgit-1.0+ x86_64
UnreportableReason: The running kernel is not an Ubuntu kernel
UserGroups:

Michael Doube (michael-doube) wrote :
Gary M (garym) on 2010-10-11
tags: added: regression-release
Michael Doube (michael-doube) wrote :

Bug not present if I boot with last 10.04 kernel, which was left as an option in my Grub menu. Reassigning to Linux as a kernel issue.

affects: acpi-support (Ubuntu) → linux (Ubuntu)
tags: added: apport-collected
description: updated

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

apport information

Michael Doube (michael-doube) wrote :

I just compiled a kernel from Linus' git repo, using the config from Ubuntu's 2.6.32.22 as a base and using defaults for everything else, and backlight is working fine

mdoube@doris:~$ uname -a
Linux doris 2.6.36-rc7-linusgit-1.0+ #4 SMP Mon Oct 11 18:03:39 CEST 2010 x86_64 GNU/Linux

So I guess this might be an Ubuntu kernel config issue

description: updated
Michael Doube (michael-doube) wrote :
Michael Doube (michael-doube) wrote :

git bisect found the commit containing the bug. I see drivers/input/evdev.c and drivers/input/input.c contain changes to key mapping.

bcbfc24ead6935714a001ab6f1f725763075f98c is the first bad commit
commit bcbfc24ead6935714a001ab6f1f725763075f98c
Author: Jarod Wilson <email address hidden>
Date: Wed Aug 18 11:16:41 2010 -0600

    UBUNTU: SAUCE: Update ir-core to linuxtv/other which should be merged for 2.6.36.

    BugLink: http://bugs.launchpad.net/bugs/609234

    Patch generated from the linuxtv staging/other branch, with a few
    additional pending fixes merged in, and just about everything not
    essential to the ir-core update chopped out.

    (Patch generated 2010.07.16)

    Signed-off-by: Jarod Wilson <email address hidden>
    Signed-off-by: Mario Limonciello <email address hidden>>
    Acked-by: Tim Gardner <email address hidden>
    Signed-off-by: Leann Ogasawara <email address hidden>

:040000 040000 999ef710a5992e79503aed592784270864047ba0 95287bb491152189b5a2f9583e07460b87972b05 M Documentation
:040000 040000 e0ed2a34e90ed54390f4156d516195ce017f9d92 25b7382d65b6bd61bc8d560831dbe67a5926807e M drivers
:040000 040000 42cc3ebe4ba10c518e27989eeb7620a8688cf3d4 7b2b0c1b7488cd103931b90a4d84e396b31b00cb M include

description: updated
description: updated
Michael Doube (michael-doube) wrote :

Jarod from Redhat tells me that patches have been merged upstream, so I will endeavour to reverse bisect to find the fix...

Michael Doube (michael-doube) wrote :

...And what a nightmare that is. No chance. Hopefully this will go away once Ubuntu merges upstream again, because it is gone there, however, Fedora 14 RC1 is affected too.

Michael Doube (michael-doube) wrote :

Possibly related fedora bug

Changed in linux (Fedora):
importance: Undecided → Unknown
status: New → Unknown
Michael Doube (michael-doube) wrote :

And ubuntu bug #568611 might be related too. Upstream kernels work fine - I have 2.6.36 mainline bug-free.

Michael Doube (michael-doube) wrote :

Mainline kernel 2.6.37-rc1 is affected. Will attempt a bisect to find the offending commit

Michael Doube (michael-doube) wrote :

After some heavy bisecting, I found the offending commit:

8613e4c2872a87cc309a42de2c7091744dc54d0e is the first bad commit
commit 8613e4c2872a87cc309a42de2c7091744dc54d0e
Author: Mauro Carvalho Chehab <email address hidden>
Date: Thu Sep 9 21:54:22 2010 -0700

    Input: add support for large scancodes

    Several devices use a high number of bits for scancodes. One important
    group is the Remote Controllers. Some new protocols like RC-6 define a
    scancode space of 64 bits.

    The current EVIO[CS]GKEYCODE ioctls allow replace the scancode/keycode
    translation tables, but it is limited to up to 32 bits for scancode.

    Also, if userspace wants to clean the existing table, replacing it by
    a new one, it needs to run a loop calling the ioctls over the entire
    sparse scancode space.

    To solve those problems, this patch extends the ioctls to allow drivers
    handle scancodes up to 32 bytes long (the length could be extended in
    the future should such need arise) and allow userspace to query and set
    scancode to keycode mappings not only by scancode but also by index.

    Compatibility code were also added to handle the old format of
    EVIO[CS]GKEYCODE ioctls.

    Folded fixes by:
    - Dan Carpenter: locking fixes for the original implementation
    - Jarod Wilson: fix crash when setting keycode and wiring up get/set
                    handlers in original implementation.
    - Dmitry Torokhov: rework to consolidate old and new scancode handling,
                       provide options to act either by index or scancode.

    Signed-off-by: Mauro Carvalho Chehab <email address hidden>
    Signed-off-by: Dan Carpenter <email address hidden>
    Signed-off-by: Jarod Wilson <email address hidden>
    Signed-off-by: Dmitry Torokhov <email address hidden>

:040000 040000 88897dc6073df285885f2a0e22f07833a55753f3 0ce85c424a2c7d36036959bb70702c544afa9b27 M drivers
:040000 040000 6914657f7124ebf4ad170bbbbab4943bb2b91105 4b7036e9697c0e10d43c1d47111eef8877aa77be M include

Michael Doube (michael-doube) wrote :

#23022 was marked a duplicate

Michael Doube (michael-doube) wrote :

Patch has been submitted upstream, see linux bugzilla

description: updated
Brad Figg (brad-figg) on 2010-12-04
tags: added: acpi-apic
Michael Doube (michael-doube) wrote :

This bug is still present after updating last night to uname -a :
Linux doris 2.6.35-24-generic #42-Ubuntu SMP Thu Dec 2 02:41:37 UTC 2010 x86_64 GNU/Linux

The bug still persists in the latest update. Linux 2.6.35-25-generic #43-Ubuntu SMP Thu Jan 6 22:25:21 UTC 2011 x86_64 GNU/Linux. Is there an targeted kernel version in which the fix will be integrated? I don't want to spam this list every time a new kernel update is out.

Changed in linux:
status: Unknown → Fix Released
Peter Rhone (prhone-gmail) wrote :

Same problem with Sony Vaio TZ11XN (PCG-4L2M), no brightness control via Function keys.

Tim Gardner (timg-tpi) on 2011-01-28
Changed in linux (Ubuntu):
assignee: nobody → Tim Gardner (timg-tpi)
status: New → In Progress
Changed in linux (Ubuntu Natty):
status: In Progress → Fix Released
Changed in linux (Ubuntu Maverick):
assignee: nobody → Tim Gardner (timg-tpi)
status: New → In Progress
Andy Whitcroft (apw) on 2011-01-31
Changed in linux (Ubuntu Maverick):
status: In Progress → Fix Committed
Changed in linux:
importance: Unknown → Medium
Tim Gardner (timg-tpi) wrote :

Michael Doube - please try the Maverick kernel in the pre-proposed PPA at https://launchpad.net/~kernel-ppa/+archive/pre-proposed

Peter Rhone (prhone-gmail) wrote :

Tim: Nice, that works perfectly.

Michael Doube (michael-doube) wrote :

Tim - pre-proposed kernel fixes this bug:

mdoube@doris:~$ uname -a
Linux doris 2.6.35-26-generic #46pre201102030901-Ubuntu SMP Thu Feb 3 09:22:35 UTC 2011 x86_64 GNU/Linux

Accepted linux into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Brad Figg (brad-figg) on 2011-02-15
tags: added: verification-needed-maverick
Brad Figg (brad-figg) wrote :

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed' to 'verification-done'.

If verification is not done by one week from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

Tim Gardner (timg-tpi) on 2011-02-16
tags: added: verification-done-maverick
removed: verification-needed-maverick
Launchpad Janitor (janitor) wrote :
Download full text (12.1 KiB)

This bug was fixed in the package linux - 2.6.35-27.48

---------------
linux (2.6.35-27.48) maverick-proposed; urgency=low

  [ Steve Conklin ]

  * Release Tracking Bug
    - LP: #723335

  [ Upstream Kernel Changes ]

  * thinkpad-acpi: avoid keymap pitfall
    - LP: #722747

linux (2.6.35-27.47) maverick-proposed; urgency=low

  [ Brad Figg ]

  * Release Tracking Bug
    - LP: #716532

  [ Upstream Kernel Changes ]

  * Revert "USB: gadget: Allow function access to device ID data during
    bind()"
    - LP: #714732
  * net: fix rds_iovec page count overflow, CVE-2010-3865
    - LP: #709153
    - CVE-2010-3865
  * Input: fix typo in keycode validation supporting large scancodes
    - LP: #658198
  * net: ax25: fix information leak to userland, CVE-2010-3875
    - LP: #710714
    - CVE-2010-3875
  * net: ax25: fix information leak to userland harder, CVE-2010-3875
    - LP: #710714
    - CVE-2010-3875
  * net: packet: fix information leak to userland, CVE-2010-3876
    - LP: #710714
    - CVE-2010-3876
  * net: tipc: fix information leak to userland, CVE-2010-3877
    - LP: #711291
    - CVE-2010-3877
  * posix-cpu-timers: workaround to suppress the problems with mt exec,
    CVE-2010-4248
    - LP: #712609
    - CVE-2010-4248
  * sys_semctl: fix kernel stack leakage, CVE-2010-4083
    - LP: #712749
    - CVE-2010-4083
  * thinkpad-acpi: lock down size of hotkey keymap
    - LP: #712174
  * thinkpad-acpi: add support for model-specific keymaps
    - LP: #712174
  * thinkpad-acpi: Add KEY_CAMERA (Fn-F6) for Lenovo keyboards
    - LP: #712174
  * x86, hotplug: Use mwait to offline a processor, fix the legacy case
    - LP: #714732
  * fuse: verify ioctl retries
    - LP: #714732
  * fuse: fix ioctl when server is 32bit
    - LP: #714732
  * ALSA: hda: Use position_fix=1 for Acer Aspire 5538 to enable capture on
    internal mic
    - LP: #685161, #714732
  * ALSA: hda: Use model=lg quirk for LG P1 Express to enable playback and
    capture
    - LP: #595482, #714732
  * drm/radeon/kms: don't apply 7xx HDP flush workaround on AGP
    - LP: #714732
  * drm/kms: remove spaces from connector names (v2)
    - LP: #714732
  * drm/radeon/kms: fix vram base calculation on rs780/rs880
    - LP: #714732
  * nohz: Fix printk_needs_cpu() return value on offline cpus
    - LP: #714732
  * nohz: Fix get_next_timer_interrupt() vs cpu hotplug
    - LP: #714732
  * nfsd: Fix possible BUG_ON firing in set_change_info
    - LP: #714732
  * NFS: Fix fcntl F_GETLK not reporting some conflicts
    - LP: #714732
  * sunrpc: prevent use-after-free on clearing XPT_BUSY
    - LP: #714732
  * hwmon: (adm1026) Allow 1 as a valid divider value
    - LP: #714732
  * hwmon: (adm1026) Fix setting fan_div
    - LP: #714732
  * EDAC: Fix workqueue-related crashes
    - LP: #714732
  * amd64_edac: Fix interleaving check
    - LP: #714732
  * ASoC: Fix swap of left and right channels for WM8993/4 speaker boost
    gain
    - LP: #714732
  * ASoC: Fix off by one error in WM8994 EQ register bank size
    - LP: #714732
  * ASoC: WM8580: Fix R8 initial value
    - LP: #714732
  * ASoC: fix deemphasis control in wm8904/55/60 codecs
    - LP: #714732
  * bootmem: Add alloc_bootmem_...

Changed in linux (Ubuntu Maverick):
status: Fix Committed → Fix Released
Changed in linux (Fedora):
importance: Unknown → Medium
status: Unknown → Won't Fix
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.