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

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


[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 : Thu Dec 07 2000 - 21:00:08 EST