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: [PATCH] Split of the normal mode
Date: Sun, 29 Mar 2009 23:20:38 +0900	[thread overview]
Message-ID: <200903292320.38804.okuji@enbug.org> (raw)
In-Reply-To: <ca0f59980903290659m3d6d6252y6bb68d45b9db9e31@mail.gmail.com>

On Sunday 29 March 2009 22:59:53 Bean wrote:
> On Sun, Mar 29, 2009 at 9:10 PM, Yoshinori K. Okuji <okuji@enbug.org> wrote:
> > On Sunday 29 March 2009 21:09:05 Bean wrote:
> >> Hi,
> >>
> >> Some of my consideration about splitting of normal mode.
> >>
> >> 1, In some environment, we have size limit of the boot image.
> >> Normal.mod is too big, and rescue mode has too little functionality.
> >> Using the split method, we could combine modules in anyway we wanted.
> >
> > In my opinion, you are postponing the decision to the runtime too much.
> > If you have N modules, you have N! combinations, but we don't need to
> > support that many in reality. I bet that you know which portions of the
> > normal mode you would like to select for your own need. Surely, others
> > might have different needs, but the number of modules you added looks
> > overkill to me.
>
> Actually, there is still only one normal configuration. But others
> could be useful in some situation. We could hide the details from
> normal user, but they can do such configuration if required.

My basic attitude is "making it only if it is used in pracice". If your goal 
is virtual, I don't want to accept it.

>
> >> 2, Speaking of linux, it's actually doing the same thing. The kernel
> >> is in vmlinuz, while the initialization script is in initrd.img. We
> >> don't want the user to enter those commands manually, a default
> >> boot.cfg should be used by grub-mkimage.
> >
> > Mmh, I hardly agree on this. The purpose of initrd.img is, although it
> > could be abused, to bootstrap an OS environment for further work, which
> > is analogous to core.img in GRUB. For the rest of the work, ifplugd,
> > udev, etc. take care of loading appropriate modules on demand.
>
> Sometime we need to setup the environment before it can access boot
> media. For example, locating the boot partition using label/uuid,
> setting the network address, etc, this is where boot.cfg can be quite
> useful.

If you use a label, label support should be loaded automatically.
If you use an uuid, uuid support should be loaded automatically.
If you set up a network, network support should be loaded automatically.

So I see no reason to stop automatic loading.

> >> 3. Currently, the structure of normal.mod just mix things together to
> >> a point that make modification difficult, if not impossible. For
> >> example, the current bash script engine is not quite suitable for gui
> >> interaction. With the split mode, we could develop a new parser
> >> without interfere with existing function.
> >
> > I prefer that you replace the existing code with a better implementation
> > in this case. From my point of view, fancy menu support is a key feature
> > in GRUB, thus if the current engine is not good enough, we need to
> > improve it rather than to provide an alternative.
>
> If I were to fix this problem, I'd separate parser and reader code
> anyway. In fact, colin already separate the menu viewer code. The
> problem is to still keep them in a single normal.mod, or to move them
> to logical independent modules. I see no problem with the later.

Views can be separete. I don't deny this. But the scripting engine itself 
should stay in normal mode. What is wrong with this?

Regards,
Okuji



  reply	other threads:[~2009-03-29 14:20 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-29  9:25 [PATCH] Split of the normal mode Bean
2009-03-29  9:40 ` Vesa Jääskeläinen
2009-03-29 10:09   ` Yoshinori K. Okuji
2009-03-29 10:43     ` phcoder
2009-03-29 10:53       ` Vesa Jääskeläinen
2009-03-29 11:33         ` phcoder
2009-03-29 11:51           ` Yoshinori K. Okuji
2009-03-29 12:09             ` Bean
2009-03-29 13:10               ` Yoshinori K. Okuji
2009-03-29 13:59                 ` Bean
2009-03-29 14:20                   ` Yoshinori K. Okuji [this message]
2009-03-29 14:30                     ` Bean
2009-03-29 14:54                       ` Yoshinori K. Okuji
2009-03-29 15:17                         ` Bean
2009-03-30 15:42                           ` Yoshinori K. Okuji
2009-03-30 20:04                 ` Colin D Bennett
2009-03-31  6:11                   ` Bean
2009-03-31 15:02                     ` Colin D Bennett
2009-03-29 13:00             ` phcoder
2009-03-29 14:51               ` Yoshinori K. Okuji
2009-03-29 17:07                 ` phcoder
2009-03-29 20:35                   ` phcoder
2009-03-30 15:49                     ` Yoshinori K. Okuji
2009-03-30 15:59                       ` phcoder
2009-04-01  7:43                         ` phcoder
2009-04-01  8:07                           ` David Miller
2009-04-01  9:22                             ` phcoder
2009-04-01 14:10                               ` Yoshinori K. Okuji
2009-04-01 16:14                                 ` phcoder
2009-03-29 11:29       ` Yoshinori K. Okuji
2009-03-29 11:40         ` David Miller
2009-03-29 12:55           ` Yoshinori K. Okuji
2009-03-29 13:23             ` Vesa Jääskeläinen
2009-03-29 14:49               ` Yoshinori K. Okuji
2009-03-29 15:43                 ` Vesa Jääskeläinen
2009-03-29 20:26               ` David Miller
2009-03-29 20:24             ` David Miller
2009-03-29 20:38               ` phcoder
2009-03-30 15:43                 ` Bean
2009-03-30 16:22                   ` Yoshinori K. Okuji
2009-03-30 16:38                     ` Bean
2009-03-30 16:35                   ` Vesa Jääskeläinen
2009-03-29 11:59         ` General design (was Re: [PATCH] Split of the normal mode) phcoder
2009-03-29 10:48     ` [PATCH] Split of the normal mode Vesa Jääskeläinen
2009-03-29 11:39       ` Yoshinori K. Okuji
2009-03-29 12:17       ` Vesa Jääskeläinen
2009-04-01 13:19     ` Robert Millan
2009-04-01 14:15       ` Yoshinori K. Okuji

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=200903292320.38804.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.