lkml.org 
[lkml]   [2008]   [Dec]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: Runaway loop with the current git.
    > >        max_modprobes = min(max_threads/2, MAX_KMOD_CONCURRENT);
    > > atomic_inc(&kmod_concurrent);
    > > if (atomic_read(&kmod_concurrent) > max_modprobes) {
    > > /* We may be blaming an innocent here, but unlikely */
    > > if (kmod_loop_msg++ < 5)
    > > printk(KERN_ERR
    > > "request_module: runaway loop modprobe
    > > %s\n", module_name);
    > > atomic_dec(&kmod_concurrent);
    > > return -ENOMEM;
    > > }
    > >
    > > Happy now. Print it out, share it with friends, find someone who can read
    > > C if you are stuck.
    >
    > It does not work, that's all.

    It works for me.

    > Reproduce the bug and look at it for yourself.

    Well since you've got a reproducer and this code works for me (I've tested
    it just fine), why don't you go and reproduce the problem then post a fix
    to that code I quoted instead of all this reordering rubbish. If you fix
    this code not only won't you risk all the mess from re-ordering
    initialisations around the kernel but you'll fix non console related
    looping which you imply is also broken as you claim that code doesn't
    work for you.

    If I deliberately break my module utils I see a sequence of modprobes
    which then hits kmod_concurrent limit then causes a -ENOMEM back to
    userspace which then fails the file open. The bug report also shows the
    printk is displayed so the runaway loop *was* detected and the code paths
    taken which stopped the loop.

    I get open -> modprobe -> open -> modprobe -> open -> modprobe ... ->
    open fail, then open fail, open fail, open fail, open fail back to the
    first modprobe exiting.

    Your proposal to keep the current recent modprobe parameter strings would
    shorten the amount of recursion but it wouldn't change the result that I
    can see. If I open /dev/console early and wrongly from a modprobe then I
    ultimately get a failing open just as I should do.

    Alan


    \
     
     \ /
      Last update: 2008-12-07 21:03    [W:2.618 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site