grub-devel.gnu.org archive mirror
 help / color / mirror / Atom feed
From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: Add --memdisk parameter to grub-install
Date: Mon, 25 Feb 2013 23:06:23 +0100	[thread overview]
Message-ID: <512BE05F.2030400@gmail.com> (raw)
In-Reply-To: <20130206174119.Horde.V_FoIpir309REtwPc6vEf6A@imap.linux.ibm.com>

[-- Attachment #1: Type: text/plain, Size: 1368 bytes --]


>It works this way:

>grub rescue> set
>prefix=(mduuid/f897565f27383014091630e267aa4720)/grub
>root=mduuid/f897565f27383014091630e267aa4720
>grub rescue> set root=memdisk
>grub rescue> set prefix=($root)/
>grub rescue> ls /
>powerpc-ieee1275/
>grub rescue> insmod normal
>grub rescue> normal
If you have to type this then obviously patch isn't finished work. You
need to point prefix to right point and have grub.cfg find and source
the definite grub.cfg.
You also need to include other files in memdisk like fonts and
translations. Having multi-prefix (prefix=a:b:c) and helpers to load
files from any of them is a possibility as long as it doesn't increase
core size.
Also you need to make a list of platforms which don't support this
approach. Currently:
Inherently: i386_pc
Due to choice of memory map: i386_coreboot, i386_multiboot,
i386_ieee1275, i386_qemu.
This should also be extended to grub-mknetdir.
It should also probably share more code with grub-mkstandalone.
On 06.02.2013 23:41, Paulo Flabiano Smorigo/Brazil/IBM wrote:

> -fi
> +additional_options=
> +if [ x"$config_opt_file" != x ]; then
> +	additional_options="-c \"${config_opt_file}\""
> +fi
> +if [ x"$memdisk_img" != x ]; then
> +	additional_options="${additional_options} -m ${memdisk_img}"
> +fi
> +

This breaks orribly in presence of whitespaces.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]

      reply	other threads:[~2013-02-25 22:06 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-06 22:41 Add --memdisk parameter to grub-install Paulo Flabiano Smorigo/Brazil/IBM
2013-02-25 22:06 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]

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=512BE05F.2030400@gmail.com \
    --to=phcoder@gmail.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).