Ubuntu
linux package

New revision Chelsio NICs don't work with cxgb3 driver from 2.6.24

Bug #201893 reported by Roland Dreier on 2008-03-13

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

Bug Description

Binary package hint: linux-source-2.6.24

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=8.04
DISTRIB_CODENAME=hardy
DISTRIB_DESCRIPTION="Ubuntu hardy (development branch)"

I installed a new Chelsio T302 ethernet NIC in a system running hardy's 2.6.24-12. The network interface did not work and I got the following in my kernel log:

  [ 2051.124913] eth2: Chelsio T302 10/100/1000BASE-T RNIC (rev 4) PCI Express x4 MSI-X
  [ 2052.175071] cxgb3 0000:02:00.0: TP out of Rx pages (0x1000000)
  [ 2052.175084] cxgb3 0000:02:00.0: encountered fatal error, operation suspended
  [ 2052.175095] cxgb3 0000:02:00.0: FW status: 0x0, 0x0, 0x0, 0x0

A Chelsio driver developer told me:

The driver in 2.6.24 does not unfortunately support the latest chipset
(rev 4). Full support will go in 2.6.25. A two liner fix needs to be applied to get it supported.

Here is the issue: T3C adds parity error detection. Without full initialization, the HW will send false positives at load time. They should be ignored, but the 2.6.24 driver misinterprets one of them, leading to this fatal error.

I am attaching the patch that he sent, which fixes the problem on my system and is upstream for 2.6.25 as part of commit a2604be5488095657aeb1a09c3f08d9f760132ec ("cxgb3 - HW set up updates"). It would be nice if this could go into the Hardy kernel so that this HW works out of the box.

Revision history for this message

Roland Dreier (roland.dreier) wrote on 2008-03-13:

patch to fix parity set up in cxgb3 on new rev Chelsio T3 NICs Edit (823 bytes, text/plain)

Revision history for this message

Leann Ogasawara (leannogasawara) wrote on 2008-03-14:

Thanks Roland. I'll have the kernel team take a look.

Changed in linux:
assignee:	nobody → ubuntu-kernel-team
importance:	Undecided → Medium
status:	New → Triaged

Colin Ian King (colin-king) on 2008-03-18

Changed in linux:
assignee:	ubuntu-kernel-team → colin-king
status:	Triaged → In Progress

Colin Ian King (colin-king) on 2008-03-18

Changed in linux:
status:	In Progress → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2008-03-30:

This bug was fixed in the package linux - 2.6.24-13.23

---------------
linux (2.6.24-13.23) hardy; urgency=low

[Alessio Igor Bogani]

* rt: Updated configuration files

[Ben Collins]

  * openvz: New custom flavour for OpenVZ
  * config: Disable IDE AMD driver in favor of PATA version
    - LP: #181561
  * config: Disable IDE VIA driver in favor of PATA version
    - LP: #181561
  * drivers/video: Restore gutsy backlight dimming behavior
    - LP: #205261
  * build/config: Enable CONFIG_CIFS_WEAK_PW_HASH
    - LP: #202445

[Colin Ian King]

* SAUCE: Add support for version 4 of Chelsio NICs in cxgb3 driver
- LP: #201893

[Kees Cook]

  * AppArmor: re-add missing "type" field in syslog reports.
    - LP: #202888
  * kvm: reset TSS on x86_64 to avoid ioperm bitmap corruption
    - LP: #144900

[Stefan Bader]

  * USB: EHCI: add separate IAA watchdog timer
    - LP: #198619
  * SAUCE: Always use SCO protocol (disable eSCO support)
    - LP: #39414
  * PM: Introduce PM_EVENT_HIBERNATE callback state
    - LP: #201086

[Tim Gardner]

  * Disable DRM suspend/resume on pre-915 Intel chips
    - LP: #207496
  * frame buffer regression - screen blank except for blinking cursor after fbcon
    vtswitch
    - LP: #201591

-- Tim Gardner <email address hidden> Wed, 19 Mar 2008 10:05:05 -0400