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.
    On Sun, Dec 07, 2008 at 05:28:55PM +0000, Alan Cox (alan@lxorguk.ukuu.org.uk) wrote:
    > > > /dev/console is a logical mapping to a device which may well be
    > > > different, loaded after PCI is initialised and dependant on PCI.
    > >
    > > So wrong. If no driver is associated, like early, in that case, we
    > > must return -ENODEV, instead of calling modprobe in a loop. It's a
    > > built-in device, and it's easy to fix.
    >
    > You've clearly no idea how initrd even works have you ? If it just
    > returned -ENODEV you wouldn't be able to open the console and you
    > wouldn't trigger the loading of the module to get the console running. So
    > you've now completely buggered the boot process.
    >
    > The correct sequence is
    >
    > Open device
    > Kernel issues hotplug message
    > Hotplug script loads drivers to policy
    >
    >
    > The problem case you have due to initrd bugs is
    >
    > Open device
    > Kernel issues hotplug message
    > Hotplug script opens same device (BUG)

    Everyone understands that, what you do not want to get, is that this
    case can be handled by the kernel so that there would be no recursion.
    And instead of thinking how to fix it, you just try to shut it up.

    There may be another case, when the same happens inside the kernel, i.e.
    module being loaded requires console and the same happens again. Similar
    problem exists for network console, when there is no underlying device
    yet, but it is handled.

    Fortunately console is the most common example (maybe even the only
    one), so this case can be fixed easily. Moreover, because of subtle
    tty ordering, when everything is in kernel, it still may be triggered,
    as was shown previously.

    And while having dumb console device sounds awful for you, it is a
    bulletprof solution even for non-expected userspace behaviout. And so
    far the only objection was that it may break something, which you
    believe may happen not even being tested.

    Alan, let's make some progress on this fingerpointing. If Herbert's
    patch fixes the crypto loading problem, it will find its way upstream
    for the current tree, and in the merge window Kay's patch may be applied
    and widely tested. Thoughts?

    --
    Evgeniy Polyakov


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