Messages in this thread Patch in this message | | | From | Rusty Russell <> | Subject | [PATCH] Shut up about the damn modules already... | Date | Thu, 12 Feb 2004 14:13:32 +1100 |
| |
Please apply before 2.6.3.
In almost all distributions, the kernel asks for modules which don't exist, such as "net-pf-10" or whatever. Changing "modprobe -q" to "succeed" in this case is hacky and breaks some setups, and also we want to know if it failed for the fallback code for old aliases in fs/char_dev.c, for example.
Just remove the debugging message which fill people's logs: the correct way of debugging module problems is something like this:
echo '#! /bin/sh' > /tmp/modprobe echo 'echo "$@" >> /tmp/modprobe.log' >> /tmp/modprobe echo 'exec /sbin/modprobe "$@"' >> /tmp/modprobe chmod a+x /tmp/modprobe echo /tmp/modprobe > /proc/sys/kernel/modprobe
Thanks! Rusty.
diff -urpN --exclude TAGS -X /home/rusty/devel/kernel/kernel-patches/current-dontdiff --minimal linux-2.6.3-rc2-bk1/kernel/kmod.c tmp/kernel/kmod.c --- linux-2.6.3-rc2-bk1/kernel/kmod.c 2004-01-10 13:59:39.000000000 +1100 +++ tmp/kernel/kmod.c 2004-02-12 14:07:33.000000000 +1100 @@ -105,16 +105,6 @@ int request_module(const char *fmt, ...) } ret = call_usermodehelper(modprobe_path, argv, envp, 1); - if (ret != 0) { - static unsigned long last; - unsigned long now = jiffies; - if (now - last > HZ) { - last = now; - printk(KERN_DEBUG - "request_module: failed %s -- %s. error = %d\n", - modprobe_path, module_name, ret); - } - } atomic_dec(&kmod_concurrent); return ret; } -- Anyone who quotes me in their sig is an idiot. -- Rusty Russell. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |