All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Yoshinori K. Okuji" <okuji@enbug.org>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: multiboot2
Date: Fri, 10 Apr 2009 08:25:23 +0900	[thread overview]
Message-ID: <200904100825.23968.okuji@enbug.org> (raw)
In-Reply-To: <49DAA9E6.2020909@gmail.com>

On Tuesday 07 April 2009 10:18:30 phcoder wrote:
> Yoshinori K. Okuji wrote:
> >>> 1) double the size of flags. 8 features per category seems to be few.
> >
> > I do not agree on this. As you can see, most bits are still undefined
> > after over 10-year usage of the Multiboot Specification. I do not want to
> > change it without any real issue.
>
> The difference is that multiboot2 is meant to be portable

Yes, but so?

> > There's a good reason to make it optional. If you see GRUB only, you will
> > think that this behavior should be always implemented, but some boot
> > loaders are more nervous about the code size, so they want to skip as
> > many features as they can. In fact, AFAIK, Etherboot didn't implement
> > sorting in its Multiboot support.
>
> What I want is to avoid is the bitrot as with multiboot1 when due to
> different issues some kernels boot only with some booters. Such a
> situation defeats the purpose of the standard

Not really. Even with the most strict spec possible, it is always possible to 
depend on implementation details which are not part of the spec. So, if an OS 
image does boot only with some implementations, it is a fault in the OS 
image, and the OS image should be fixed.

> >>> 6) memory map. "<!> Tags of this type should be omitted on
> >>> architectures where the OS is able to retrieve this information from
> >>> firmware. (Doing do will encourage OS portability across bootloaders,
> >>> and simplify GRUB development and maintenance.) "
> >>> This contradicts the goal of easier OS developement and may result in
> >>> semi-compatible OS and bootloaders. Additionally I think that
> >>> eliminating the necessity of use of firmware from OS is a good thing
> >>> and allows easier porting between architectures differing only by
> >>> firmware
> >
> > It is hard for me to say which is better.
> >
> > In reality, every OS needs to interact with underlying firmware more or
> > less to be functional (power control, interrupt handling, etc.). So
> > giving a memory map does not eliminate the necessity of interactions with
> > firmware anyway.
>
> This isn't entirely true. Most of OS use their own firmware-independent
> drivers for most devices.

For device drivers, yes. For other things, not always. For instance, on Alpha, 
you need to use the firmware to enter the privileged mode. AFAIK, no other 
choice.

From my point of view, the conclusion should be based on whether a boot loader 
may want to provide a memory map different from what firmware thinks. If yes, 
we have a good reason to make it required. If no, not much.

> > Seemingly, someone made a bad change on the draft, so the information is
> > lost:
> >
> > http://grub.enbug.org/MultibootDraft?action=diff&rev2=23&rev1=22
> >
> > Hollis's idea was to use the same format as for modules to give
> > information about an OS image. A part of this change must be reverted. It
> > is wrong to adopt the spec to the implementation.
>
> It's ok with me. Quick look through the code suggests that probably
> kernel tag is created with type MODULE and that it also has an
> additional field type. I will check it tomorrow but it looks like a bug
> somewhere

Hmm.

> And what about encoding?

Fine for me.

Regards,
Okuji



  reply	other threads:[~2009-04-09 23:25 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-03 16:30 multiboot2 phcoder
2009-03-04  3:50 ` multiboot2 Kevin Lacquement
2009-03-04  8:33   ` multiboot2 phcoder
2009-04-05 23:34 ` multiboot2 phcoder
2009-04-07  0:24   ` multiboot2 Yoshinori K. Okuji
2009-04-07  1:18     ` multiboot2 phcoder
2009-04-09 23:25       ` Yoshinori K. Okuji [this message]
2009-04-11 18:56         ` multiboot2 phcoder
2009-07-28 22:36           ` multiboot2 Vladimir 'phcoder' Serbinenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200904100825.23968.okuji@enbug.org \
    --to=okuji@enbug.org \
    --cc=grub-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.