RFC: InfiniBand hotplug agent

[Roland Dreier <rdreier@cisco.com>]

I merged new feature for the InfiniBand stack in kernel 2.6.17.  There
will be /sys/class/infiniband/<dev>/node_desc files.  This can be used
by userspace to change the IB node description, which is retrieved by
various network management stuff.  Putting the hostname in the node
description makes things a lot nicer, since it provides a link from
the IB network view back to the real world.

Right now on my systems, I'm using the script below, which takes the
output of hostname and adds it to the node description.  I put this in
/lib/udev/infiniband.agent along with a udev rule like

    SUBSYSTEM="infiniband", RUN+="infiniband.agent"

to run it.

So I have two questions:

 - Is this script reasonable?  I'm by no means a hotplug expert, and
   I'd like to get critiques and suggestions for how to write a proper
   agent.

 - What's the right way to get this into distributions?  File wishlist
   bugs against udev with the rule and the script included?

Thanks,
  Roland

Here's the script:

#!/bin/sh -e
#
# infiniband agent
#

cd /lib/udev/
. ./hotplug.functions

if [ "$ACTION" = "add" ]; then
    DESC_PATH="/sys/$DEVPATH/node_desc"
    if [ -e "$DESC_PATH" ]; then
        DESC="`hostname` (`cat $DESC_PATH`)"
        echo -n $DESC > $DESC_PATH
    fi
fi

exit 0


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid\x110944&bid$1720&dat\x121642
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel