> I looked into the corresponding script (/etc/init.d/modutils),
> the culprit is probably
>
>         echo -n "$module - "
>         MODTOLOAD=`find / -name $module.o |sort |sed -n 1p`
>         if [ "$MODTOLOAD" = ""  ] ;then
>            module="` echo $module | cut -c-8`"
>            MODTOLOAD=`find / -name $module.o |sort |sed -n 1p `
>         fi
>         if [ ! "$MODTOLOAD" = "" ] ;then
>         insmod $MODTOLOAD $args
>         fi
>
> It starts its search at /, which is probably fine to detect all
> sorts of modules in the directory tree.

This obviously is the part that produces the original error message.

Not looking everywhere for modules might break some setups things.
Unfortunately, busybox find provides neither -xdev nor -prune to avoid
looking in /proc. I suggest just throwing away find's stderr

> # echo ip_conntrack_irc.o | cut -c-8

> This IMHO does not enhance the chance to find the correct module and
should > be done away with.

I agree. I propose

    echo -n "$module - "
    MODTOLOAD=`find / -name $module.o 2>/dev/null | head -n 1`
    if [ ! "$MODTOLOAD" = "" ] ;then
      insmod $MODTOLOAD $args
    else
      echo "not found"
    fi

Discussion:
- throw away error messages of find
- why the sort? doesn't make sense to me at all.
- I prefer head -n 1 to sed -n 1p
- forget the search with 8.3
- error reporting



-------------------------------------------------------
This SF.net email is sponsored by: IBM Linux Tutorials.
Become an expert in LINUX or just sharpen your skills.  Sign up for IBM's
Free Linux Tutorials.  Learn everything from the bash shell to sys admin.
Click now! http://ads.osdn.com/?ad_id=1278&alloc_id=3371&op=click

_______________________________________________
leaf-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/leaf-devel

Reply via email to