Re: [patch] set prefix on PPC

[Marco Gerards]

Hollis Blanchard <address@hidden> writes:

> Marco, I believe I addressed all your comments in my mail on 23 Feb
> 2005. I have implemented your suggestions, with the exception of
> checking the device node property lengths, as this requires modifying
> many callers elsewhere in the code.
>
> I will commit this in a couple days if I don't hear any complaints.

There is a problem with the patch, so please don't.  When booting on
the pegasosII I see the following problem:

grub> set
prefix=(/address@hidden/address@hidden,1/address@hidden,0,3)

That is when I am booting using `boot hd:4 grub)'.

The prefix contains an OF device name which will not work in GRUB,
mainly because of the comma's.  Would it be possible to lookup the OF
path in /aliases and set the prefix to the alias?  That will be
something that will work and is more user friendly I think.

Another thing we could do, but which is a bit more ugly, is creating
an alias.  I am not sure if it is possible to create an OF alias from
GRUB, but I assume it will be.

Shouldn't the prefix end with a `/' like you mentioned in one of your
other emails.  So in this case:

prefix=(/address@hidden/address@hidden,1/address@hidden,0,3)/


Thanks,
Marco