Re: [PATCH] memdisk plus lnxboot extension

[Robert Millan]

On Wed, Jan 23, 2008 at 12:22:32AM +0800, Bean wrote:
>       * util/i386/pc/grub-mkimage.c (generate_image) : Add is_linux parameter.
>       If it's set, use lnxboot.img as header instead of diskboot.img.
>       (options): Add "linux"|'x' option
>       (main): Parse --linux|-x option, and pass its value to generate_image.

I don't like this very much.  We don't have grub-mkimage options to concatenate
it with boot.img, so why with lnxboot.img ?

Also, I can only think of very specific situations in which this interface
would be useful (that is, when firmware has only a linux loader).  It makes
sense to me as a compatibility layer, yes.

But it seems you want it as a general-purpose option; for that, why not make
it saner like multiboot?  I don't think it's a good idea to compromise our
boot semantics because of the ones legacy Linux has.

-- 
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 /.)