> 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.
t.
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 [EMAIL PROTECTED]
Please read the FAQ at http://www.tux.org/lkml/
