Avoid unnecessary warnings in xHCI

Bug #929656 reported by Leann Ogasawara on 2012-02-09
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Medium
Leann Ogasawara
Oneiric
Medium
Leann Ogasawara
Precise
Medium
Leann Ogasawara

Bug Description

== SRU Justification ==
When running Oneiric on a Chief River (Ivy Bridge/Panther Point) platform, the following messages are printed to dmesg which are harmless but could cause user concern. It's been requested we cherry-pick two upstream patches to eliminate these messages.

"[ 1.037314] xhci_hcd 0000:00:14.0: Failed to enable MSI-X"

"[ 1.037314] WARN: short transfer on control ep"

== Fix ==
The following upstream patches remove these warnings:

  commit 3b9783b277e66731891ab42eeaacebbdcdd6e629
  Author: Sarah Sharp <email address hidden>
  Date: Thu Dec 22 15:02:13 2011 -0800

    xhci: Remove warnings about MSI and MSI-X capabilities.

  commit 2a9227a5eeaeb3f91e3a72ceea4fa59016ca5d20
  Author: Sarah Sharp <email address hidden>
  Date: Tue Oct 25 13:55:30 2011 +0200

    xhci: Remove scary warnings about transfer issues.

== Impact ==
There should be minimal to no risk of regression as the patches are merely lowering the log level from an error message to a debug message. The debug messages will only appear if CONFIG_USB_XHCI_HCD_DEBUGGING is enabled, which it currently is not.

== Test Case ==
Boot with an unpatched kernel and confirm you see something similar to "xhci_hcd 0000:00:14.0: Failed to enable MSI-X" on boot and possibly see the "WARN: short transfer on control ep" when conducting normal operations on a USB device.

Changed in linux (Ubuntu Oneiric):
assignee: nobody → Leann Ogasawara (leannogasawara)
importance: Undecided → Medium
status: New → In Progress

I've cherry-picked the following two patches from v3.3-rc1 into Precise:

  commit 3b9783b277e66731891ab42eeaacebbdcdd6e629
  Author: Sarah Sharp <email address hidden>
  Date: Thu Dec 22 15:02:13 2011 -0800

    xhci: Remove warnings about MSI and MSI-X capabilities.

  commit 2a9227a5eeaeb3f91e3a72ceea4fa59016ca5d20
  Author: Sarah Sharp <email address hidden>
  Date: Tue Oct 25 13:55:30 2011 +0200

    xhci: Remove scary warnings about transfer issues.

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

For anyone able and willing to test, I've built an oneiric test kernel and placed it at the following location:

http://people.canonical.com/~ogasawara/lp929656/amd64/

description: updated

I have test confirmation from vanhoof, that the first error message goes away when running the above patched test kernel:

$ dmesg | grep xhci_hcd
<vanhoof> [ 2.329944] xhci_hcd 0000:00:14.0: PCI INT A -> GSI 21 (level, low) -> IRQ 21
<vanhoof> [ 2.329978] xhci_hcd 0000:00:14.0: setting latency timer to 64
<vanhoof> [ 2.329983] xhci_hcd 0000:00:14.0: xHCI Host Controller
<vanhoof> [ 2.330055] xhci_hcd 0000:00:14.0: new USB bus registered, assigned bus number 3
<vanhoof> [ 2.330175] xhci_hcd 0000:00:14.0: cache line size of 64 is not supported
<vanhoof> [ 2.330195] xhci_hcd 0000:00:14.0: irq 21, io mem 0xb22a0000
<vanhoof> [ 2.330254] xhci_hcd 0000:00:14.0: irq 42 for MSI/MSI-X
<vanhoof> [ 2.330485] xhci_hcd 0000:00:14.0: xHCI Host Controller
<vanhoof> [ 2.330528] xhci_hcd 0000:00:14.0: new USB bus registered, assigned bus number 4

Tim Gardner (timg-tpi) on 2012-02-09
Changed in linux (Ubuntu Oneiric):
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :

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

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

  [ Andy Whitcroft ]

  * d-i -- include the Hyper-V drivers in the virtio udeb
    - LP: #917135

  [ Felix Fietkau ]

  * (pre-stable): ath9k_hw: fix a RTS/CTS timeout regression
    - LP: #925602

  [ Keith Packard ]

  * SAUCE: drm/i915: Force explicit bpp selection for
    intel_dp_link_required
    - LP: #745112, #912387, #917330

  [ Leann Ogasawara ]

  * Fix typo in generic-pae description
    - LP: #928448
  * Rebase to v3.2.6

  [ Upstream Kernel Changes ]

  * procfs: parse mount options
    - CVE-2011-4917
  * procfs: add hidepid= and gid= mount options
    - CVE-2011-4917
  * proc: fix null pointer deref in proc_pid_permission()
    - CVE-2011-4917
  * xhci: Remove warnings about MSI and MSI-X capabilities.
    - LP: #929656
  * xhci: Remove scary warnings about transfer issues.
    - LP: #929656
  * x86, mce, therm_throt: Don't report power limit and package level
    thermal throttle events in mcelog
    - LP: #930288
  * rebase to v3.2.6
    - LP: #924320
    - LP: #918254
 -- Leann Ogasawara <email address hidden> Mon, 13 Feb 2012 13:00:08 -0800

Changed in linux (Ubuntu Precise):
status: Fix Committed → Fix Released
Herton R. Krzesinski (herton) wrote :

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

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!

tags: added: verification-needed-oneiric
tags: added: verification-done-oneiric
removed: verification-needed-oneiric
Launchpad Janitor (janitor) wrote :
Download full text (12.3 KiB)

This bug was fixed in the package linux - 3.0.0-16.29

---------------
linux (3.0.0-16.29) oneiric-proposed; urgency=low

  [Herton R. Krzesinski]

  * Release Tracking Bug
    - LP: #931661

  [ James M. Leddy ]

  * SAUCE: Bluetooth: Add support for BCM20702A0 [0a5c:21e6]
    - LP: #906832

  [ Manoj Iyer ]

  * SAUCE: Bluetooth: Add support for BCM20702A0 [0a5c:21e1]
    - LP: #906832

  [ Merlin Schumacher ]

  * SAUCE: acer-wmi: support for P key on TM8372
    - LP: #865807

  [ Tim Gardner ]

  * SAUCE: Add vendor specific ID (0a5c 21f3) for BCM20702A0
    - LP: #925552

  [ Upstream Kernel Changes ]

  * Revert "(pre-stable) HID: bump maximum global item tag report size to
    96 bytes"
    - LP: #922799
  * Revert "Revert "proc: enable writing to /proc/pid/mem""
    - LP: #922799
  * mtdoops: fix the oops_page_used array size
    - LP: #922799
  * mtd: mtdoops: skip reading initially bad blocks
    - LP: #922799
  * mtd: mtd_blkdevs: don't increase 'open' count on error path
    - LP: #922799
  * mtd: tests: stresstest: bail out if device has not enough eraseblocks
    - LP: #922799
  * drivers/rtc/interface.c: fix alarm rollover when day or month is
    out-of-range
    - LP: #922799
  * ext4: fix undefined behavior in ext4_fill_flex_info()
    - LP: #922799
  * ALSA: snd-usb-us122l: Delete calls to preempt_disable
    - LP: #922799
  * ALSA: ice1724 - Check for ac97 to avoid kernel oops
    - LP: #922799
  * ALSA: hda - Return the error from get_wcaps_type() for invalid NIDs
    - LP: #922799
  * drm/radeon/kms: workaround invalid AVI infoframe checksum issue
    - LP: #922799
  * drm/radeon/kms: disable writeback on pre-R300 asics
    - LP: #922799
  * radeon: Fix disabling PCI bus mastering on big endian hosts.
    - LP: #922799
  * NFS: Retry mounting NFSROOT
    - LP: #922799
  * NFSv4.1: fix backchannel slotid off-by-one bug
    - LP: #922799
  * nfs: fix regression in handling of context= option in NFSv4
    - LP: #922799
  * HID: bump maximum global item tag report size to 96 bytes
    - LP: #724831, #922799
  * UBI: fix missing scrub when there is a bit-flip
    - LP: #922799
  * UBI: fix use-after-free on error path
    - LP: #922799
  * PCI: Fix PCI_EXP_TYPE_RC_EC value
    - LP: #922799
  * PCI: msi: Disable msi interrupts when we initialize a pci device
    - LP: #922799
  * x86/PCI: Ignore CPU non-addressable _CRS reserved memory resources
    - LP: #922799
  * x86/PCI: build amd_bus.o only when CONFIG_AMD_NB=y
    - LP: #922799
  * SCSI: mpt2sas: Release spinlock for the raid device list before
    blocking it
    - LP: #922799
  * SCSI: mpt2sas : Fix for memory allocation error for large host credits
    - LP: #922799
  * xen/xenbus: Reject replies with payload > XENSTORE_PAYLOAD_MAX.
    - LP: #922799
  * ima: free duplicate measurement memory
    - LP: #922799
  * ima: fix invalid memory reference
    - LP: #922799
  * rtl8192se: Fix BUG caused by failure to check skb allocation
    - LP: #922799
  * mac80211: fix rx->key NULL pointer dereference in promiscuous mode
    - LP: #922799
  * memcg: add mem_cgroup_replace_page_cache() to fix LRU issue
    - LP: #922799
  * x86: Fix mmap random address range
    - LP: #92...

Changed in linux (Ubuntu Oneiric):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers