Ubuntu
linux package

ASPM doesn't work completely on Asus Zenbook (regression from 3.1)

Bug #962038 reported by Timo Jyrinki on 2012-03-22

This bug affects 5 people

Affects		Status	Importance	Assigned to	Milestone
	linux (Ubuntu)	Fix Released	Medium	Colin Ian King
	Precise	Fix Released	Medium	Colin Ian King

Bug Description

ASPM doesn't work completely on Asus Zenbook UX31 (regression from kernel 3.1 with pcie_aspm=force)

The issue reported at [1] holds true. With 12.04 kernel, the following is seen at the moment:
$ sudo lspci -vv | grep ASPM
LnkCap: Port #0, Speed unknown, Width x0, ASPM unknown, Latency L0 <64ns, L1 <1us
LnkCtl: ASPM Disabled; Disabled- Retrain- CommClk-
LnkCap: Port #1, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <1us, L1 <16us
LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk-
LnkCap: Port #2, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <512ns, L1 <16us
LnkCtl: ASPM Disabled; RCB 64 bytes Disabled- Retrain- CommClk+
LnkCap: Port #4, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <512ns, L1 <16us
LnkCtl: ASPM Disabled; RCB 64 bytes Disabled- Retrain- CommClk+
LnkCap: Port #0, Speed 2.5GT/s, Width x1, ASPM L0s L1, Latency L0 <4us, L1 <64us
LnkCtl: ASPM Disabled; RCB 64 bytes Disabled- Retrain- CommClk+
LnkCap: Port #0, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 unlimited, L1 unlimited
LnkCtl: ASPM Disabled; RCB 64 bytes Disabled- Retrain- CommClk+

With two enable-aspm [2] calls, as follows:

ROOT_COMPLEX="00:1c.1"
ENDPOINT="02:00.0"
---
ROOT_COMPLEX="00:1c.3"
ENDPOINT="03:00.0"

the following is gotten instead and everything continues to function fine:

$ sudo lspci -vv | grep ASPM
LnkCap: Port #0, Speed unknown, Width x0, ASPM unknown, Latency L0 <64ns, L1 <1us
LnkCtl: ASPM Disabled; Disabled- Retrain- CommClk-
LnkCap: Port #1, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <1us, L1 <16us
LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk-
LnkCap: Port #2, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <512ns, L1 <16us
LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk+
LnkCap: Port #4, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 <512ns, L1 <16us
LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk+
LnkCap: Port #0, Speed 2.5GT/s, Width x1, ASPM L0s L1, Latency L0 <4us, L1 <64us
LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk+
LnkCap: Port #0, Speed 5GT/s, Width x1, ASPM L0s L1, Latency L0 unlimited, L1 unlimited
LnkCtl: ASPM L0s L1 Enabled; RCB 64 bytes Disabled- Retrain- CommClk+

[1] http://ubuntuforums.org/showthread.php?t=1865577&page=130#post11771228
[2] http://bombadil.infradead.org/~mcgrof/scripts/enable-aspm
---
AlsaVersion: Advanced Linux Sound Architecture Driver Version 1.0.24.
ApportVersion: 1.94.1-0ubuntu2
Architecture: amd64
ArecordDevices:
**** List of CAPTURE Hardware Devices ****
card 0: PCH [HDA Intel PCH], device 0: ALC269VB Analog [ALC269VB Analog]
   Subdevices: 1/1
   Subdevice #0: subdevice #0
AudioDevicesInUse:
USER PID ACCESS COMMAND
/dev/snd/controlC0: timo 1780 F.... pulseaudio
CRDA: Error: [Errno 2] Tiedostoa tai hakemistoa ei ole
Card0.Amixer.info:
Card hw:0 'PCH'/'HDA Intel PCH at 0xdfe00000 irq 51'
   Mixer name : 'Intel CougarPoint HDMI'
   Components : 'HDA:10ec0269,10431427,00100100 HDA:80862805,80860101,00100000'
   Controls : 18
   Simple ctrls : 8
DistroRelease: Ubuntu 12.04
EcryptfsInUse: Yes
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Alpha amd64 (20120316)
MachineType: ASUSTeK Computer Inc. UX31E
Package: linux (not installed)
ProcEnviron:
TERM=xterm
PATH=(custom, user)
LANG=fi_FI.UTF-8
SHELL=/bin/bash
ProcFB: 0 inteldrmfb
ProcKernelCmdLine: BOOT_IMAGE=/boot/vmlinuz-3.2.0-19-generic root=UUID=3a913d68-5aad-4ac9-9436-4798da5b8fc2 ro crashkernel=384M-2G:64M,2G-:128M quiet splash drm.vblankoffdelay=1 i915.i915_enable_fbc=1 i915.lvds_downclock=1 i915.semaphores=1 vt.handoff=7
ProcVersionSignature: Ubuntu 3.2.0-19.31-generic 3.2.12
RelatedPackageVersions:
linux-restricted-modules-3.2.0-19-generic N/A
linux-backports-modules-3.2.0-19-generic N/A
linux-firmware 1.72
StagingDrivers: rts5139 mei
Tags: precise staging
Uname: Linux 3.2.0-19-generic x86_64
UpgradeStatus: No upgrade log present (probably fresh install)
UserGroups: adm cdrom dip lpadmin plugdev sambashare sudo
dmi.bios.date: 12/26/2011
dmi.bios.vendor: American Megatrends Inc.
dmi.bios.version: UX31E.210
dmi.board.asset.tag: ATN12345678901234567
dmi.board.name: UX31E
dmi.board.vendor: ASUSTeK Computer Inc.
dmi.board.version: 1.0
dmi.chassis.asset.tag: No Asset Tag
dmi.chassis.type: 10
dmi.chassis.vendor: ASUSTeK Computer Inc.
dmi.chassis.version: 1.0
dmi.modalias: dmi:bvnAmericanMegatrendsInc.:bvrUX31E.210:bd12/26/2011:svnASUSTeKComputerInc.:pnUX31E:pvr1.0:rvnASUSTeKComputerInc.:rnUX31E:rvr1.0:cvnASUSTeKComputerInc.:ct10:cvr1.0:
dmi.product.name: UX31E
dmi.product.version: 1.0
dmi.sys.vendor: ASUSTeK Computer Inc.

See original description

Tags:

Related branches

lp:ubuntu/precise/linux-armadaxp

lp:ubuntu/precise-proposed/linux-ti-omap4

lp:ubuntu/precise-proposed/linux-armadaxp

Colin Ian King (colin-king) on 2012-03-22

Changed in linux (Ubuntu):
assignee:	nobody → Colin King (colin-king)
importance:	Undecided → Medium
status:	New → In Progress

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: AcpiTables.txt

AcpiTables.txt Edit (377.5 KiB, text/plain)

apport information

tags:	added: apport-collected precise staging
description:	updated

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: AlsaDevices.txt

AlsaDevices.txt Edit (508 bytes, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: AplayDevices.txt

AplayDevices.txt Edit (265 bytes, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: BootDmesg.txt

BootDmesg.txt Edit (57.3 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: Card0.Amixer.values.txt

Card0.Amixer.values.txt Edit (1.6 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: Card0.Codecs.codec.0.txt

Card0.Codecs.codec.0.txt Edit (8.5 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: Card0.Codecs.codec.3.txt

Card0.Codecs.codec.3.txt Edit (3.0 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: CurrentDmesg.txt

CurrentDmesg.txt Edit (2.7 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: IwConfig.txt

IwConfig.txt Edit (473 bytes, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: Lspci.txt

#10

Lspci.txt Edit (8.7 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: Lsusb.txt

#11

Lsusb.txt Edit (417 bytes, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: PciMultimedia.txt

#12

PciMultimedia.txt Edit (635 bytes, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: ProcCpuinfo.txt

#13

ProcCpuinfo.txt Edit (3.6 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: ProcInterrupts.txt

#14

ProcInterrupts.txt Edit (2.4 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: ProcModules.txt

#15

ProcModules.txt Edit (2.7 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: PulseList.txt

#16

PulseList.txt Edit (16.5 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: RfKill.txt

#17

RfKill.txt Edit (175 bytes, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: UdevDb.txt

#18

UdevDb.txt Edit (110.1 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: UdevLog.txt

#19

UdevLog.txt Edit (284.8 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22: WifiSyslog.txt

#20

WifiSyslog.txt Edit (332.7 KiB, text/plain)

apport information

Revision history for this message

Timo Jyrinki (timo-jyrinki) wrote on 2012-03-22:

#21

Colin sent me a patched kernel 3.2.0-20-generic #32+aspm1 - this fixes the regression in so that pcie_aspm=force works again on Zenbook!

In summary:

- 3.2 kernel currently in Ubuntu 12.04: no ASPM with or without pcie_aspm=force
- The 3.2+aspm1 kernel from Colin King: ASPM fully in action again when pcie_aspm=force is used

So the actual regression was that even pcie_aspm=force didn't help anymore, unlike with 3.1 kernel, and the patched kernel fixes this.

Revision history for this message

Colin Ian King (colin-king) wrote on 2012-03-22:

#22

For reference, kernel .debs at: http://zinc.canonical.com/~cking/lp-962038/

Revision history for this message

Colin Ian King (colin-king) wrote on 2012-03-26:

#23

0001-PCI-Allow-pcie_aspm-force-to-work-even-when-FADT-ind.patch Edit (1.2 KiB, text/plain)

For reference, the patch used to address this issue.

Ubuntu Foundations Team Bug Bot (crichton) on 2012-03-26

tags:

added: patch

Tim Gardner (timg-tpi) on 2012-03-30

Changed in linux (Ubuntu Precise):
status:	In Progress → Fix Committed

Revision history for this message

Ubuntu Foundations Team Bug Bot (crichton) wrote on 2012-03-30:

#24

The attachment "0001-PCI-Allow-pcie_aspm-force-to-work-even-when-FADT-ind.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

Revision history for this message

Launchpad Janitor (janitor) wrote on 2012-04-03:

#25

Download full text (5.2 KiB)

This bug was fixed in the package linux - 3.2.0-22.35

---------------
linux (3.2.0-22.35) precise; urgency=low

[ Andy Whitcroft ]

  * Revert "SAUCE: hotkey quirks for various Zeptro Znote and Fujitsu Amilo
    laptops"
  * SAUCE: (no-up) elide some ioctl warnings which are known benign
    - LP: #972355

[ Brad Figg ]

  * SAUCE (no-up) Provide a param for allowing the BIOS to handle changing
    the brightness on AC/battery status changes.
    - LP: #949311
  * SAUCE (drop after 3.4) Quirk for enabling backlight hotkeys on Samsung
    N150P
    - LP: #875893

[ Colin Ian King ]

  * SAUCE: PCI: Allow pcie_aspm=force to work even when FADT indicates it
    is unsupported
    - LP: #962038

[ Daniel Vetter ]

* SAUCE: (drop after 3.5) drm/i915: reinstate GM45 TV detection fix
- LP: #638939

[ Kees Cook ]

* SAUCE: SECCOMP: audit: always report seccomp violations
* SAUCE: SECCOMP: adjust prctl constant

[ Leann Ogasawara ]

  * [Config] Enable CONFIG_USBIP_CORE=m
    - LP: #900384
  * Rebase to v3.2.14
  * [Config] Updateconfigs after rebase to v3.2.14

[ Stefan Bader ]

* d-i: Fix module name for dm-raid45
- LP: #969248

[ Tim Gardner ]

* SAUCE: remove __initdata from vesafb_fix
- LP: #969309

[ Upstream Kernel Changes ]

This bug was fixed in the package linux - 3.2.0-22.35

---------------
linux (3.2.0-22.35) precise; urgency=low

[ Andy Whitcroft ]

* Revert "SAUCE: hotkey quirks for various Zeptro Znote and Fujitsu Amilo
    laptops"
  * SAUCE: (no-up) elide some ioctl warnings which are known benign
    - LP: #972355

[ Brad Figg ]

[ Colin Ian King ]

* SAUCE: PCI: Allow pcie_aspm=force to work even when FADT indicates it
    is unsupported
    - LP: #962038

[ Daniel Vetter ]

* SAUCE: (drop after 3.5) drm/i915: reinstate GM45 TV detection fix
    - LP: #638939

[ Kees Cook ]

* SAUCE: SECCOMP: audit: always report seccomp violations
  * SAUCE: SECCOMP: adjust prctl constant

[ Leann Ogasawara ]

* [Config] Enable CONFIG_USBIP_CORE=m
    - LP: #900384
  * Rebase to v3.2.14
  * [Config] Updateconfigs after rebase to v3.2.14

[ Stefan Bader ]

* d-i: Fix module name for dm-raid45
    - LP: #969248

[ Tim Gardner ]

* SAUCE: remove __initdata from vesafb_fix
    - LP: #969309

[ Upstream Kernel Changes ]

* Revert "sched: tg->se->load should be initialised to tg->shares"
  * toshiba_acpi: make one-bit bitfields unsigned
    - LP: #810015
  * ACPI: EC: Add ec_get_handle()
    - LP: #810015
  * toshiba_acpi: Support alternate hotkey interfaces
    - LP: #810015
  * toshiba_acpi: Support additional hotkey scancodes
    - LP: #810015
  * toshiba_acpi: Refuse to load on machines with buggy INFO
    implementations
    - LP: #810015
  * ata_piix: Add Toshiba Satellite Pro A120 to the quirks list due to
    broken suspend functionality.
    - LP: #886850
  * sweep the floors and convert some .get_drvinfo routines to strlcpy
    - LP: #921793
  * be2net: init (vf)_if_handle/vf_pmac_id to handle failure scenarios
    - LP: #921793
  * be2net: stop checking the UE registers after an EEH error
    - LP: #921793
  * be2net: don't log more than one error on detecting EEH/UE errors
    - LP: #921793
  * be2net: stop issuing FW cmds if any cmd times out
    - LP: #921793
  * be2net: Fix TX queue create for Lancer
    - LP: #921793
  * be2net: add register dump feature for Lancer
    - LP: #921793
  * be2net: Add EEPROM dump feature for Lancer
    - LP: #921793
  * be2net: Fix VLAN promiscous mode for Lancer
    - LP: #921793
  * be2net: Use V1 query link status command for lancer
    - LP: #921793
  * be2net: Move to new SR-IOV implementation in Lancer
    - LP: #921793
  * be2net: Fix error recovery paths
    - LP: #921793
  * be2net: Add error handling for Lancer
    - LP: #921793
  * be2net: Use new hash key
    - LP: #921793
  * be2net: Fix non utilization of RX queues
    - LP: #921793
  * be2net: netpoll support
    - LP: #921793
  * be2net: update some counters to display via ethtool
    - LP: #921793
  * be2net: workaround to fix a bug in BE
    - LP: #921793
  * be2net: fix ethtool ringparam reporting
    - LP: #921793
  * be2net: refactor/cleanup vf configuration code
    - LP: #921793
  * be2net: Add support for Skyhawk cards
    - LP: #921793
  * be2net: Fix INTx processing for Lancer
    - LP: #921793
  * be2net: fix range check for set_qos for a VF
    - LP: #921793
  * be2net: query link status in be_open()
    - LP: #921793
  * netdev: make net_device_ops const
    - LP: #921793
  * be2net: create RSS rings even in multi-channel configs
    - LP: #921793
  * ethtool: Null-terminate filename passed to ethtool_ops::flash_device
    - LP: #921793
  * sched: Fix nohz load accounting
    - LP: #838811
  * ACPICA: Fix regression in FADT revision checks
    - LP: #883441
  * asus-wmi: move WAPF variable into quirks_entry
    - LP: #969576
  * eeepc-wmi: refine quirks handling
    - LP: #969576
  * drm/i915: Sanitize BIOS debugging bits from PIPECONF
    - LP: #968307
  * drivers: hv: Get rid of some unnecessary code
  * drivers: hv: kvp: Add/cleanup connector defines
  * drivers: hv: kvp: Move the contents of hv_kvp.h to hyperv.h
  * net/hyperv: Convert camel cased variables in rndis_filter.c to lower
    cases
  * net/hyperv: Correct the assignment in netvsc_recv_callback()
  * net/hyperv: Remove the unnecessary memset in rndis_filter_send()
  * tools: hv: Use hyperv.h to get the KVP definitions
  * drivers: hv: Cleanup the kvp related state in hyperv.h
  * drivers: hv: kvp: Cleanup the kernel/user protocol
  * drivers: hv: Increase the number of VCPUs supported in the guest
  * net/hyperv: Fix data corruption in rndis_filter_receive()
  * net/hyperv: Add support for vlan trunking from guests
  * Drivers: hv: Add new message types to enhance KVP
  * net/hyperv: fix erroneous NETDEV_TX_BUSY use
  * NLS: improve UTF8 -> UTF16 string conversion routine
  * Drivers: hv: Support the newly introduced KVP messages in the driver
  * Tools: hv: Fully support the new KVP verbs in the user level daemon
  * Tools: hv: Support enumeration from all the pools
  * Drivers: scsi: storvsc: Properly handle errors from the host
  * net: mv643xx_eth: fix build error
    - LP: #921793
  * rebase to v3.2.14
 -- Leann Ogasawara <leann.ogasawara@canonical.com>   Tue, 03 Apr 2012 08:59:16 -0700