Re: [PATCH] memdisk + grub-mkrescue

[Robert Millan]

On Mon, Jan 21, 2008 at 12:47:06PM -0500, Pavel Roskin wrote:
> On Mon, 2008-01-21 at 17:52 +0100, Robert Millan wrote:
> > These two were predestined to get together..  attached patch for their
> > wedding.
> > 
> > Comments?
> 
> I really like the ability to add more things than just modules
> (especially grub.cfg and fonts).  Is that the main motivation for your
> patch?  Or you have some other things in mind?

Yes, that's it.  Specially for situations in which there's no other way
(like netboot images, or in the future coreboot when it works on ELF).

> I don't quite understand why "biosdisk" needs to be mandatory.  I guess
> it's not initialized by default, so that it doesn't conflict with the
> ata module.  But ata can load on top of biosdisk (but not the other way
> around).

I just added it because we don't seem to have a way to define which modules
should be autoloaded (other than grub.cfg itself).  And I'm afraid not loading
it by default will confuse users.  I don't know how to improve this, but ideas
are welcome, of course.

-- 
Robert Millan

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