ipmi_devintf needs a MODALIAS of platform:ipmi_si

[James Troup]

I'm trying to use the HP Proliant health monitoring software which is
based on IPMI in modern HP hardware. Ubuntu detects the iLO/IPMI
hardware in the server and correctly loads the ipmi_si module, but
unfortunately, the daemon won't start without also manually modprobing
ipmi_devintf.

| $ sudo /etc/init.d/hpasmd start
| * Starting HP Proliant Health Monitor...
| hpasmxld: get_options failed: Illegal seek
| ...fail!
| $ sudo modprobe ipmi_devintf
| $ sudo /etc/init.d/hpasmd start
| * Starting HP Proliant Health Monitor...
| ...done.
| $

Scott says the ipmi_devintf module should have a MODALIAS of
platform:ipmi_si as a hint to userspace to try and load the
ipmi_devintf module when it loads ipmi_si.

[Scott James Remnant (Canonical) (canonical-scott)]

The ipmi_devintf module contains the userspace interface for IPMI
devices, yet will not be loaded automatically with a system
interface handler driver.

Add a MODULE_ALIAS for the "platform:ipmi_si" MODALIAS exported by
the ipmi_si driver, so that userspace knows of the recommendation.

Signed-off-by: Scott James Remnant <email address hidden>
---
 drivers/char/ipmi/ipmi_devintf.c | 1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/char/ipmi/ipmi_devintf.c b/drivers/char/ipmi/ipmi_devintf.c
index 64e1c16..ecaf369 100644
--- a/drivers/char/ipmi/ipmi_devintf.c
+++ b/drivers/char/ipmi/ipmi_devintf.c
@@ -957,3 +957,4 @@ module_exit(cleanup_ipmi);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Corey Minyard <email address hidden>");
 MODULE_DESCRIPTION("Linux device interface for the IPMI message handler.");
+MODULE_ALIAS("platform:ipmi_si");
--
1.5.6.3

[James Troup (elmo)]

I rebuilt the hardy kernel with Scott's patch applied and can confirm it fixes my problem. The Proliant health monitoring software now starts correctly on boot.

[Leann Ogasawara (leannogasawara)]

Looks like Tim has applied this to Intrepid:

ogasawara@yoji:~/ubuntu-intrepid$ git log 98b20ae2a50f0367b2369a577bb93b67dc2a3178
commit 98b20ae2a50f0367b2369a577bb93b67dc2a3178
Author: Scott James Remnant <email address hidden>
Date: Tue Oct 21 15:18:36 2008 +0100

    add MODULE_ALIAS to load ipmi_devintf with ipmi_si

[Leann Ogasawara (leannogasawara)]

Just moving this from Fix Committed to Fix Released.