Ubuntu
linux package

Avoid unnecessary warnings in xHCI

Bug #929656 reported by Leann Ogasawara
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Medium
Leann Ogasawara
Oneiric
Fix Released
Medium
Leann Ogasawara
Precise
Fix Released
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.

See original description
Leann Ogasawara (leannogasawara)
Changed in linux (Ubuntu Oneiric):
assignee: nobody → Leann Ogasawara (leannogasawara)
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

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
Leann Ogasawara (leannogasawara)
description: updated
description: updated
description: updated
Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

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
Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

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

Revision history for this message
Leann Ogasawara (leannogasawara) wrote :

Oneiric SRU submitted:

https://lists.ubuntu.com/archives/kernel-team/2012-February/018890.html

Tim Gardner (timg-tpi)
Changed in linux (Ubuntu Oneiric):
status: In Progress → Fix Committed
Revision history for this message
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
Revision history for this message
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
Leann Ogasawara (leannogasawara)
tags: added: verification-done-oneiric
removed: verification-needed-oneiric
Revision history for this message
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  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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