Right, that is because "you" (slash the default) have specified in the
openmpi configuration file (/etc/default/openmpi) that IPMI_SI is needed
on your system. Thus, when it fails to load successfully, it is still
flagged as:

# no bits set = no errors
# bit 0 set = minimum modules aren't loaded
# bit 1 set = requested feature module isn't loaded
# bit 2 set = /dev/ipmi0 (or /dev/imb if using that instead) doesn't exist
# bit 3 set = /dev/watchdog doesn't exist
# bit 4 set = lockfile doesn't exist
# bit 5 set = modules are loaded even when asked to be unloaded

bit 0 being set (RETVAL | 1).

I guess we could change the logic in a different way such that bit 1
should be set in this case, rather than bit 0. But that would redefine
what the "minimum" modules are in OpenIPMI, which per the init-script
are ipmi_msghandler and ipmi_{si,smb} if those options are set in the
config file.

So I think what I provided is probably the best combination for the
configuration by default -- Robie did point out that my patch has a
logical issue in that the last added line is really a no-op. So I'll
spin up a new version and push it to my PPA. If you can test that one as
well, just to be sure it's functionally the same, then we'll work on
getting that into -proposed (once I learn how to do that :)

I'll provide another update once the new packages are present.

Thanks, Jeremy!

-- 
You received this bug notification because you are a member of Ubuntu
Server Team, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/1318317

Title:
  openipmi startup script removes kernel modules

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/openipmi/+bug/1318317/+subscriptions

-- 
Ubuntu-server-bugs mailing list
Ubuntu-server-bugs@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-server-bugs

Reply via email to