[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Handling command line option for core.img and g2ldr
From: |
Yoshinori K. Okuji |
Subject: |
Re: [PATCH] Handling command line option for core.img and g2ldr |
Date: |
Thu, 28 Feb 2008 23:29:14 +0100 |
User-agent: |
KMail/1.9.4 |
On Thursday 28 February 2008 17:58, Robert Millan wrote:
> On Fri, Feb 29, 2008 at 12:14:06AM +0800, Bean wrote:
> > > There are worse situations in fact (in that they're more common), like
> > > RAID and LVM drives (which can be the boot device too).
> > >
> > > But I don't think we need parser code for that. The path part of the
> > > prefix (/boot/grub, but sometimes /grub) is already hardcoded in by
> > > grub-mkimage, so why not extend that to storing the whole path? It's
> > > just a few bytes more (typically 4), which is much less than code to
> > > parse cmdline params.
> >
> > I think this could work, it actually avoid a lot of trouble by storing
> > the device name as well.
>
> One thing I overlooked is that it wouldn't allow for 0xFF effect (i.e. get
> the root drive from bios), which is also desirable (or even is that the
> default?).
>
> > A little setback is that we need to create
> > different core.img for different boot device,
>
> I don't think that's a big problem since core.img is generated every time
> in grub-install (but in most cases, 0xFF setup will handle it, is that
> correct?).
No, no, no. It is one of the most important features in GRUB that you can boot
it, even if a disk is migrated. We must avoid hardcoding whenever possible.
Okuji