From: Robert Millan <rmh@aybabtu.com>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: embedding filesystems in core image (Re: moving ata initialisation to a command)
Date: Sat, 22 Dec 2007 13:18:27 +0100 [thread overview]
Message-ID: <20071222121827.GA27503@thorin> (raw)
In-Reply-To: <ca0f59980712212028q7718e771y2ca2057c03bfdec8@mail.gmail.com>
Here's a new patch, with some cleanup. The main difference is that memdisk.c
doesn't include any arch-specific code.
I've spotted a memory management problem. The memdisk image, at the location
that it's usually uncompressed, tends to collide with the payload loading
region (grub_os_area_{addr,size}).
I'm not sure what would be a good solution to this. Perhaps we could copy it
to dynamic memory in grub_memdisk_init() ? However, if user plays with payload
images without booting them, and then loads memdisk.mod manually, things could
break.
Although, embedding a memdisk image without embedding memdisk.mod is kind of
silly; perhaps grub-mkimage shouldn't allow it.
What do you think?
--
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 /.)
next prev parent reply other threads:[~2007-12-22 12:18 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-16 15:30 moving ata initialisation to a command Robert Millan
2007-12-16 15:58 ` Christian Franke
2007-12-16 16:06 ` Vesa Jääskeläinen
2007-12-16 16:19 ` Robert Millan
2007-12-17 15:52 ` Marco Gerards
2007-12-17 16:01 ` Marco Gerards
2007-12-17 18:26 ` Pavel Roskin
2007-12-17 19:01 ` Marco Gerards
2007-12-20 17:38 ` Robert Millan
2007-12-20 17:36 ` Robert Millan
2007-12-21 16:47 ` Marco Gerards
2007-12-21 18:04 ` Robert Millan
2007-12-21 20:54 ` embedding filesystems in core image (Re: moving ata initialisation to a command) Robert Millan
2007-12-22 4:28 ` Bean
2007-12-22 11:24 ` Robert Millan
2007-12-22 12:27 ` Yoshinori K. Okuji
2007-12-22 12:18 ` Robert Millan [this message]
2007-12-22 12:24 ` Yoshinori K. Okuji
2007-12-23 18:44 ` Robert Millan
2008-01-16 20:10 ` Marco Gerards
2008-01-20 23:21 ` Robert Millan
2007-12-24 8:29 ` Bean
2007-12-24 13:09 ` Robert Millan
2007-12-24 13:27 ` Bean
2007-12-24 16:24 ` Robert Millan
2007-12-24 16:44 ` Bean
2008-01-02 19:56 ` Robert Millan
2007-12-26 8:00 ` embedding data in ELF " Robert Millan
2008-01-02 18:21 ` Pavel Roskin
2007-12-26 7:57 ` avoid firmware in grub-mkrescue " Robert Millan
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=20071222121827.GA27503@thorin \
--to=rmh@aybabtu.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 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.