[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.078 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site