ledmon does not work in Ubuntu 14.04

Bug #1349920 reported by Paul Boven
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ledmon (Ubuntu)
Fix Released
Undecided
Jared Dominguez
Trusty
Fix Released
High
Unassigned

Bug Description

ledmon will not work in Ubuntu 14.04, due to a change in kernel 3.13.

I've tested this with ICH10R and C600/X79 chipset.

Symptoms: ledctl gives 'enclosure management not supported' on supported hardware.

This problem has just been fixed by upstream, see this patch:

http://sourceforge.net/p/ledmon/code/ci/6f7d38bbe5d7b13c84987cf3b3b550a11c39392e/

Please update the Ubuntu package to include this patch, so ledmon works again.

Installed package: ledmon 0.79-0.2
~$ lsb_release -rd
Description: Ubuntu 14.04 LTS
Release: 14.04

[Impact]

 * The type of libahci parameter ahci_em_messages was changed from int to
bool in kernel v3.13, which caused AHCI HBAs to not be detected
properly.

 * This basically means that ledmon no longer works in 14.04

[Test Case]

 * install ledmon and start up the openipmi service

 * The following ledctl commands should no longer give 'enclosure management not supported' on supported hardware with the fix in place:

sudo ledctl locate=/dev/nvme0n1
sudo ledctl locate=/dev/rssda
sudo ledctl locate_off=/dev/rssda
sudo ledctl locate_off=/dev/nvme0n1

[Regression Potential]

 * Low, as this bug affects 14.04 and higher due to the newer kernel version. Systems running the older kernel releases should be fine.

Revision history for this message
Jared Dominguez (jared-dominguez) wrote :

Thanks. I'm building with this patch now and will upload to Debian. I'll need to also have it SRU'ed.

Changed in ledmon (Ubuntu):
assignee: nobody → Daniel Jared Dominguez (jared-dominguez)
status: New → In Progress
Revision history for this message
Kent Baxley (kentb) wrote :

For Utopic, all that needs to be done is sync the latest package with Debian. For Trusty we need to pull in the upstream patch then test it out.

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

debdiff for trusty

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

Tested the upstream patch on Trusty using:

An NVMe device as well as a Micron-based pcieSSD device in a PowerEdge R820.

Before applying the fix, both of these commands threw the 'enclosure management not supported' message, although the LED's did indeed blink in response and stopped blinking when the respective locate_off commands were invoked:

sudo ledctl locate=/dev/nvme0n1
sudo ledctl locate=/dev/rssda
sudo ledctl locate_off=/dev/rssda
sudo ledctl locate_off=/dev/nvme0n1

With the patch applied the commands continued to blink / unblink the LED's as expected, but, I no longer saw the "not supported" message on the console.

Changed in dell-poweredge:
status: New → Confirmed
status: Confirmed → In Progress
importance: Undecided → High
assignee: nobody → Kent Baxley (kentb)
description: updated
Revision history for this message
Kent Baxley (kentb) wrote :

New debdiff for Trusty

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

There is a ledmon 0.79-2 uploaded to Debian with the fix. Waiting for it to show up.

Not sure if we need to sync that or just apply the attached Utopic debdiff.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ledmon - 0.79-2

---------------
ledmon (0.79-2) unstable; urgency=high

  * Fix for 3.13 kernel change (LP: #1349920)

 -- Daniel Jared Dominguez <email address hidden> Wed, 30 Jul 2014 14:19:11 -0500

Changed in ledmon (Ubuntu):
status: In Progress → Fix Released
Robie Basak (racb)
Changed in ledmon (Ubuntu Trusty):
status: New → Triaged
importance: Undecided → High
Revision history for this message
Robie Basak (racb) wrote :

Looks good. Uploaded to Trusty - thank you for preparing this. Now awaiting SRU team review.

A few really minor review comments. I changed the version from 0.79-0.2ubuntu1 to 0.79-0.2ubuntu0.1. Following the scheme at https://wiki.ubuntu.com/SecurityTeam/UpdatePreparation#Update_the_packaging just makes it really clear it's an SRU, and avoids any potential conflict with Utopic, though that couldn't happen in this case. I ran "update-maintainer" to fix the Maintainer field (https://wiki.ubuntu.com/DebianMaintainerField). And finally, even more minor, in the dep3 headers, "Origin: upstream, <url>" makes it clearer if upstream has committed the patch at the time of writing. But "Forwarded:" is also fine, so I haven't changed it.

Revision history for this message
Chris J Arges (arges) wrote : Please test proposed package

Hello Paul, or anyone else affected,

Accepted ledmon into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/ledmon/0.79-0.2ubuntu0.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!

Changed in ledmon (Ubuntu Trusty):
status: Triaged → Fix Committed
tags: added: verification-needed
Kent Baxley (kentb)
Changed in dell-poweredge:
status: In Progress → Fix Committed
Revision history for this message
Mark W Wenning (mwenning) wrote :

Verified on Dell Poweredge R820 .

tags: added: verification-done
removed: verification-needed
Revision history for this message
Kent Baxley (kentb) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ledmon - 0.79-0.2ubuntu0.1

---------------
ledmon (0.79-0.2ubuntu0.1) trusty; urgency=high

  * debian/patches/sysfs-bool-values.patch
    Add upstream fix for 3.13 kernel change (LP: #1349920)
 -- Kent Baxley <email address hidden> Wed, 30 Jul 2014 16:32:08 -0500

Changed in ledmon (Ubuntu Trusty):
status: Fix Committed → Fix Released
Revision history for this message
Scott Kitterman (kitterman) wrote : Update Released

The verification of the Stable Release Update for ledmon has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Changed in dell-poweredge:
status: Fix Committed → 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.