Add upstream fix for Dell PCIeSSD devices

Bug #1174386 reported by Kent Baxley
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ledmon (Debian)
Fix Released
Unknown
ledmon (Ubuntu)
Fix Released
Low
Unassigned
Raring
Fix Released
Low
Unassigned

Bug Description

[Impact]

 * The ledmon version included in Raring contained a bug that broke PCIe SSD devices
   on Dell Server Hardware. The Dell PCIE SSDs do not have a 'hostN' field in their sysfs paths as they are block devices directly connected to their host PCI device.

 * The bug caused these devices to stop being recognized properly in the OS.

[Test Case]

 * Install ledmon 0.74-1 onto a raring server with the appropriate PCIE SSD hardware.

 * run 'sudo ledmon' to start the led monitor.

 * Look in /var/log/syslog..you'll see messages such as the following repeated continuously several times as the OS tries to figure out what the SSD device is:

ledmon[2449]: Device (null) : No host_id!
ledmon[2449]: DETACHED DEV '(null)' in failed state

 * With the upstream patch in place and after restarting ledmon, the "no host_id" and "DETACHED_DEV" messages will go away and the PCIe SSD devices are detected correctly.

[Regression Potential]

 * Low. This fix is very targeted to the PCIE SSD hardware on Dell systems, and should not break any other devices.

 * Tested by me on the specific hardware to ensure that the regression has gone away.

[Other Info]

Here is the upstream commit that eventually went into upstream ledmon 0.75:

http://ledmon.git.sourceforge.net/git/gitweb.cgi?p=ledmon/ledmon;a=commitdiff;h=d6b7c55869fa7a5224844407ef2d5aab785c4db8

Related branches

Revision history for this message
Kent Baxley (kentb) wrote :

Without the fix, here's how to reproduce:

1) install ledmon on a Dell PowerEdge system with PCIe SSD hardware.
2) Startup ledmon (sudo ledmon)
3) Look in /var/log/syslog..you'll see messages such as the following repeated several times:

ledmon[2449]: Device (null) : No host_id!
ledmon[2449]: DETACHED DEV '(null)' in failed state

4) With the upstream patch in place and after restarting ledmon, the "no host_id" and "DETACHED_DEV" messages will go away and the PCIe SSD devices are detected correctly.

Changed in ledmon (Ubuntu):
status: New → Confirmed
Kent Baxley (kentb)
description: updated
Changed in ledmon (Ubuntu):
importance: Undecided → Low
Kent Baxley (kentb)
description: updated
Changed in ledmon (Ubuntu Raring):
importance: Undecided → Low
status: New → In Progress
Changed in ledmon (Ubuntu):
status: Confirmed → Fix Committed
Changed in ledmon (Debian):
status: Unknown → New
Revision history for this message
Brian Murray (brian-murray) wrote :

There was an incorrectly formatted bug reference so this was not auto-closed.

ledmon (0.74-1ubuntu1) saucy; urgency=low

  * Add patch to fix Dell PCIe SSD Devices (LP#: 1174386)

Changed in ledmon (Ubuntu):
status: Fix Committed → Fix Released
Changed in ledmon (Ubuntu Raring):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Kent, or anyone else affected,

Accepted ledmon into raring-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/ledmon/0.74-1ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Revision history for this message
Kent Baxley (kentb) wrote :

Tested ledmon from raring-proposed and the problem is fixed. Thanks.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Brian Murray (brian-murray) wrote :

I've released the updated package version.

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

Other bug subscribers

Remote bug watches

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