Re: FreeBSD loader options patch

[Yoshinori K. Okuji]

At Sat, 30 Nov 2002 04:04:21 +0100,
Alexander Langer wrote:
> I only added that line so people don't wonder why 0x4000 is left out
> while all others are used.  Remove that line, if you want.  It's not
> used in GRUB and won't ever.

Ok, I'll remove it. Any macro which is never used shouldn't be defined.

> The RB_* commands were args to the reboot system call before, used from
> within reboot(8).  Here the poweroff flag makes sense.  I can't see a
> real usage for booting a kernel just to power the system off ;-)
> (Neither did I saw a reason to add support for all the other parameters
> I left out - we defenitely don't need that stuff in GRUB)

Thanks for your information. I'll add the rest of the macros.

> PS: There is currently an isse with NetBSD, which uses a different
> struct bootinfo layout than FreeBSD.  I ran into this problem when
> trying to tell the NetBSD kernel to use serial console, which failed
> since it doesn't understand FreeBSD's struct bootinfo - so basically
> GRUB must not share the same sources for FreeBSD and NetBSD there (at
> least that's the case for NetBSD 1.6).  One should fix that.  I might
> be able to do it once I have my diskless workstation running and some
> more time to actually do it ;-)

You are right. I know that. NetBSD support in GRUB really sucks. Thus
I'm looking forward to your contribution.

Thanks,
Okuji