[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 toactually mount root, then just parse one dev at a time. No allocationneeded.

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 isuseful 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 devicenumber into the kernel image at runtime. Your patch should be fullyfunctional, though, as long as people just use 'append="root=..."'instead of simply 'root=...'. The append= forces LILO not to treat theroot dev specially. (This tip brought to you by the devfs docs.)

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