[lkml]   [2000]   [Dec]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] mutliple root devs (take II)
    > Hmmm, I don't like your array thing (also in v.I of the patch),
    > limiting us to <n> possible root devices, where n==8. A better
    > approach might be to iterate over the root= arguments when mounting. I
    > know why you used the array -- easier to code.
    I was unsure if it was okay to be using kmalloc during early stages of
    init/main.c so I decided to follow the example allready set and just use a
    static array - can anyone advise on being able to do this dynamically?

    > One potential problem with the patch is that you have changed behavior
    > some people are relying on. If you use 'syslinux' to boot, for
    > example, the SYSLINUX.CFG file can define a default command line
    > including root=. Then you can augment that line at runtime by typing
    > in your own command line. Your patch makes it impossible, in this
    > situation, to override the default root device from the syslinux
    > command line. A kludge to make it work again would be to process the
    > root devices in reverse. That would be ugly and unintuitive, though.
    Yeah you would need to patch lilo as well to handle the new syntax amongst
    other things. I use grub and have no troubles along these lines.
    Refrencing the idea in the follow on message about just using the last
    root= only and not allowing multiple root= would work around this. I
    guess I can't think of any really good reason why having multiple root= is
    a necissary feature.


    Fear the future? Change the past.
    This message has resulted in an increase in the entropy of the universe

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 12:51    [W:0.019 / U:15.664 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site