On Wed, Jan 14, 2015 at 07:24:29PM +0100, Denys Vlasenko wrote:
> On Mon, Jan 12, 2015 at 3:37 PM, Natanael Copa <[email protected]> wrote:
> > Seems like that fixed the depmod issue but modprobe appears to be
> > totally broke.
> 
> I'm sorry, but this is hardly a proper bug report. Can you do better?
> 
> I just tried building defconfig bbox 1.23.0 + patch, and modprobing
> a module. It worked.
> It's not like "modprobe is so broken it's trivially to reproduce it".

defconfig is modprobe-small, not full modprobe/insmod/...; Alpine Linux
uses full (ie, turn off "simplified modutils").

Testing current git, defconfig + full modutils (after "make baseline")
# ./busybox_old depmod
# ./busybox_old modprobe usblp; grep usblp /lib/modules/3.14.27/modules.dep
modprobe: module usblp not found in modules.dep
kernel/drivers/usb/class/usblp.ko: kernel/drivers/usb/core/usbcore.ko 
kernel/drivers/usb/usb-common.ko
# ./busybox_old modprobe kernel/drivers/usb/class/usblp
# lsmod |grep usblp
usblp                  17751  0 
usbcore               131110  5 usblp,uvcvideo,ehci_pci,ehci_hcd,uhci_hcd



Commentary:
Everything *worked* with 1.23 + the check for / disabled, and the
basename()-like stuff left in place.
modprobe deals with three types of names:
* aliases (may contain '/')
* module paths, relative to /lib/modules/`uname -r` ("kernel/.../module.ko")
* module names ("module"; equivalent to `basename $MODPATH .ko`)

By definition, a module name (type 3) cannot contain '/'.
However, the check for '/' checked the command line, which can be an alias.

Thanks,
Isaac Dunham
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to