This is based a patch by William Park <[email protected]> included in
2.6.10-mm1, in order to fix booting from usb-storage devices which no longer
make their partitions immediately available.
While William's patch fixed the situation where you boot from a "root=8:1"
parameter (to correspond to /dev/sda1), it does not allow you to use
"root=/dev/sda1" (which apparently worked in 2.6.9 and earlier), because
the name-->dev_t discovery is done before the "retry" loop starts, and is
not retried during the loop.
This patch, which replaces William's, solves the above problems.
Signed-off-by: Daniel Drake <[email protected]>
On Tue, Jan 04, 2005 at 12:37:16PM +0000, Daniel Drake wrote:
> This is based a patch by William Park <[email protected]> included in
> 2.6.10-mm1, in order to fix booting from usb-storage devices which no longer
> make their partitions immediately available.
>
> While William's patch fixed the situation where you boot from a "root=8:1"
> parameter (to correspond to /dev/sda1), it does not allow you to use
> "root=/dev/sda1" (which apparently worked in 2.6.9 and earlier), because
> the name-->dev_t discovery is done before the "retry" loop starts, and is
> not retried during the loop.
>
> This patch, which replaces William's, solves the above problems.
It's funny... Your patch does the opposite. It works for
root=/dev/sda1
from the kernel command line, but not from Lilo or 'root=8:1' on command
line. :-)
--
William Park <[email protected]>
Open Geometry Consulting, Toronto, Canada
Linux solution for data processing.
Hi William,
William Park wrote:
> It's funny... Your patch does the opposite. It works for
> root=/dev/sda1
> from the kernel command line, but not from Lilo or 'root=8:1' on command
> line. :-)
Ah, yes :/
I found a simpler way to do it (but I'm not sure if it is 'clean' enough).
I've attached an incremental patch against your first patch. This works with
root=8:1 and root=/dev/sda1 for me.
Andrew, how does this look? I've also attached the new full patch so that you
can also see the context.
Thanks,
Daniel