Re: [PATCH] mutliple root devs (take II)

From: Peter Samuelson (peter@cadcamlab.org)
Date: Fri Dec 01 2000 - 18:49:06 EST

  • Next message: Jeff Dike: "Re: [PATCH] mutliple root devs (take II)"

    [Tracy Camp]
    > 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?

    Have a static 'char *' somewhere. In the "root=" callback function,
    just set this variable. Do not parse it until you are ready to
    actually mount root, then just parse one dev at a time. No allocation
    needed.

    Note that this approach doesn't support the "multiple root=" feature,
    which brings us to...

    > I guess I can't think of any really good reason why having multiple
    > root= is a necissary feature.

    Agreed, and there *is* good reason not to support this, since it is
    useful to be able to override a root= given in a config file.

    > Yeah you would need to patch lilo as well to handle the new syntax
    > amongst other things.

    Hmm. LILO shouldn't care, but it does, because it has a 'root='
    parameter which it handles specially, by patching the 16-bit device
    number into the kernel image at runtime. Your patch should be fully
    functional, though, as long as people just use 'append="root=..."'
    instead of simply 'root=...'. The append= forces LILO not to treat the
    root dev specially. (This tip brought to you by the devfs docs.)

    Peter
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    Please read the FAQ at http://www.tux.org/lkml/



    This archive was generated by hypermail 2b29 : Fri Dec 01 2000 - 18:51:41 EST