On Thu, Feb 22, 2007 at 04:12:04AM +0900, OGAWA Hirofumi wrote:
> YOSHIFUJI Hideaki / ?$B5HF#1QL@ <[EMAIL PROTECTED]> writes:
>
> > In article <[EMAIL PROTECTED]> (at Tue, 20 Feb 2007 20:53:45 -0800 (PST)),
> > Linus Torvalds <[EMAIL PROTECTED]> says:
> >
> >> But there's a ton of architecture updates (arm, mips, powerpc, x86, you
> >> name it), ACPI updates, and lots of driver work. And just a lot of
> >> cleanups.
> >
> > I cannot boot 2.6.21-rc1; it falls into OOM-Killer.
> >
> > Interesting error message I can see is:
> > request_module: runaway loop modprobe net-pf-1
> >
> > After bisecting, the commit
> > Driver core: let request_module() send a /sys/modules/kmod/-uevent
> > (id c353c3fb0700a3c17ea2b0237710a184232ccd7f) is to blame.
> >
> > Reverting it fixes the issue to me.
>
> /sbin/hotplug needs some module, but request_module() call /sbin/hotplug loop?
> Hm.. does the patch fix the problem?
How does it loop?
> BTW, mod_request_helper alias of /proc/sys/kernel/modprobe is really needed?
What do you mean?
> --
> OGAWA Hirofumi <[EMAIL PROTECTED]>
>
>
> Don't use uevent until udevd or something like other setup done.
>
> Signed-off-by: OGAWA Hirofumi <[EMAIL PROTECTED]>
> ---
>
> kernel/kmod.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff -puN kernel/kmod.c~kmod-uevent-fix kernel/kmod.c
> --- linux-2.6/kernel/kmod.c~kmod-uevent-fix 2007-02-22 03:42:37.000000000
> +0900
> +++ linux-2.6-hirofumi/kernel/kmod.c 2007-02-22 03:42:48.000000000 +0900
> @@ -90,11 +90,11 @@ int request_module(const char *fmt, ...)
> if (ret >= MODULE_NAME_LEN)
> return -ENAMETOOLONG;
>
> - strcpy(&modalias[strlen("MODALIAS=")], module_name);
> - kobject_uevent_env(&kmod_mk.kobj, KOBJ_CHANGE, uevent_envp);
> -
> - if (modprobe_path[0] == '\0')
> + if (modprobe_path[0] == '\0') {
> + strcpy(&modalias[strlen("MODALIAS=")], module_name);
> + kobject_uevent_env(&kmod_mk.kobj, KOBJ_CHANGE, uevent_envp);
> goto out;
> + }
No, we want to still emit these messgages, even if we have a real
"helper" application. I don't see how this would fix the problem.
thanks,
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
