Re: "GRUB Loading kernel..." message

2008-07-06 Thread Isaac Dupree

Robert Millan wrote:

On Sat, Jul 05, 2008 at 07:24:56PM -0400, Isaac Dupree wrote:

I thought I remembered somewhere a discussion how the message
"GRUB Loading kernel"
is confusing, because it doesn't say what kernel it's loading, and grub 
loads lots of kernels


I think this is not accurate.  GRUB loads payloads.  It doesn't care if they're
a standalone OS like invaders.elf or the kernel part of an OS that resides on
disk/network/whatever.


okay, I'm getting pedantic, but that is very much from GRUB 
point-of-view; GRUB certainly does load payloads, but from non-GRUB 
point-of-view, those payloads are often also OS kernels.  So if you know 
GRUB terminology, GRUB would never say it was loading a kernel when it 
was loading a payload, because it neither knows nor cares whether anyone 
else would call that payload a "kernel".  Rather convincing.  Although, 
it would still be as accurate for a user to say that "GRUB is loading 
linux kernel" as it would be, if I'm blindly eating food, for someone to 
say I'm eating a banana.  But I don't care so much about philosophy, 
just that "GRUB Loading kernel" confuses lots of people and it seems 
we've found a just-as-good wording that doesn't :-)


Anyway, any change of the message vs. GRUB Legacy means that we can tell 
from the message whether it's GRUB1 or GRUB2 :-D


should I send a patch to change the message from "Loading kernel" to 
"Loading"?


-Isaac


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: "GRUB Loading kernel..." message

2008-07-06 Thread Robert Millan
On Sun, Jul 06, 2008 at 08:33:03PM +0200, Robert Millan wrote:
> On Sat, Jul 05, 2008 at 07:35:15PM -0400, Pavel Roskin wrote:
> > On Sat, 2008-07-05 at 19:24 -0400, Isaac Dupree wrote:
> > 
> > > .  Although, looking at the files, boot/i386/pc/boot.S outputs "GRUB " 
> > > and boot/i386/pc/diskboot.S outputs "Loading kernel", so the parts 
> > > actually mean different things: maybe it's important that it prints 
> > > "GRUB " first in case it never prints anything else?  perhaps there are 
> > > other possible code-paths?  In any case, then maybe it should say
> > > "GRUB Loading GRUB kernel", or, "GRUB Loading itself" :-)
> > 
> > Considering that diskboot.img is limited to 512 bytes,
> 
> But it currently fits, and we don't gain anything by saving space in that
> sector.

Anyway, "GRUB loading" sounds fine to me.

-- 
Robert Millan

 I know my rights; I want my phone call!
 What good is a phone call… if you are unable to speak?
(as seen on /.)


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: "GRUB Loading kernel..." message

2008-07-06 Thread Robert Millan
On Sat, Jul 05, 2008 at 07:24:56PM -0400, Isaac Dupree wrote:
> I thought I remembered somewhere a discussion how the message
> "GRUB Loading kernel"
> is confusing, because it doesn't say what kernel it's loading, and grub 
> loads lots of kernels

I think this is not accurate.  GRUB loads payloads.  It doesn't care if they're
a standalone OS like invaders.elf or the kernel part of an OS that resides on
disk/network/whatever.

-- 
Robert Millan

 I know my rights; I want my phone call!
 What good is a phone call… if you are unable to speak?
(as seen on /.)


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: "GRUB Loading kernel..." message

2008-07-06 Thread Robert Millan
On Sat, Jul 05, 2008 at 07:35:15PM -0400, Pavel Roskin wrote:
> On Sat, 2008-07-05 at 19:24 -0400, Isaac Dupree wrote:
> 
> > .  Although, looking at the files, boot/i386/pc/boot.S outputs "GRUB " 
> > and boot/i386/pc/diskboot.S outputs "Loading kernel", so the parts 
> > actually mean different things: maybe it's important that it prints 
> > "GRUB " first in case it never prints anything else?  perhaps there are 
> > other possible code-paths?  In any case, then maybe it should say
> > "GRUB Loading GRUB kernel", or, "GRUB Loading itself" :-)
> 
> Considering that diskboot.img is limited to 512 bytes,

But it currently fits, and we don't gain anything by saving space in that
sector.

-- 
Robert Millan

 I know my rights; I want my phone call!
 What good is a phone call… if you are unable to speak?
(as seen on /.)


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: "GRUB Loading kernel..." message

2008-07-05 Thread Isaac Dupree

Pavel Roskin wrote:

On Sat, 2008-07-05 at 19:24 -0400, Isaac Dupree wrote:

.  Although, looking at the files, boot/i386/pc/boot.S outputs "GRUB " 
and boot/i386/pc/diskboot.S outputs "Loading kernel", so the parts 
actually mean different things: maybe it's important that it prints 
"GRUB " first in case it never prints anything else?  perhaps there are 
other possible code-paths?  In any case, then maybe it should say

"GRUB Loading GRUB kernel", or, "GRUB Loading itself" :-)


Considering that diskboot.img is limited to 512 bytes, I'd rather go
with "loading" without "kernel".  The result would be "GRUB loading",
which would look better than your proposals.  The saved bytes could be
used for some good purpose eventually.


I like that wording: concise and clear (although maybe so concise that 
it gets hard for a user having a problem, to Google for it).


By the way, currently the "Loading" seems to have the first letter 
capitalized (I don't care either way)


-Isaac


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel


Re: "GRUB Loading kernel..." message

2008-07-05 Thread Pavel Roskin
On Sat, 2008-07-05 at 19:24 -0400, Isaac Dupree wrote:

> .  Although, looking at the files, boot/i386/pc/boot.S outputs "GRUB " 
> and boot/i386/pc/diskboot.S outputs "Loading kernel", so the parts 
> actually mean different things: maybe it's important that it prints 
> "GRUB " first in case it never prints anything else?  perhaps there are 
> other possible code-paths?  In any case, then maybe it should say
> "GRUB Loading GRUB kernel", or, "GRUB Loading itself" :-)

Considering that diskboot.img is limited to 512 bytes, I'd rather go
with "loading" without "kernel".  The result would be "GRUB loading",
which would look better than your proposals.  The saved bytes could be
used for some good purpose eventually.

-- 
Regards,
Pavel Roskin


___
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel