grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH] GRUB2 does not provide Multiboot Legacy memory map


From: Julian Salazar
Subject: RE: [PATCH] GRUB2 does not provide Multiboot Legacy memory map
Date: Sat, 19 Apr 2008 16:09:15 -0600

Attached is the patch. Since I'm not completely familiar with GRUB2's
internals, there might be a more efficient way of implementing this.

Thoughts/improvements?

> -----Original Message-----
> From: address@hidden [mailto:grub-
> address@hidden On Behalf Of Thomas Schwinge
> Sent: April-19-08 3:32 PM
> To: Julian Salazar
> Cc: address@hidden
> Subject: Re: [PATCH] GRUB2 does not provide Multiboot Legacy memory map
> 
> Hello!
> 
> On Sat, Apr 19, 2008 at 03:11:43PM -0600, Julian Salazar wrote:
> > I'm developing a small kernel and I've been using GRUB2 with the Legacy
> > Multiboot Specification for some time now. I've noticed that it does not
> > provide some of the things in the Multiboot Information structure that
> GRUB
> > Legacy does, like the memory map. So I've created a small patch which
> works
> > in QEMU and real hardware so that GRUB2 will pass the memory map to my
> > kernel or any multiboot-compliant kernel for the i386-pc.
> 
> Now this is interesting as I've just today seen an incompatibility
> between booting a multiboot kernel with GRUB legacy and GRUB2.  It might
> actually be the same issue, but I didn't fully investigate yet.
> Therefore I'm quite interested in seeing your patch.
> 
> 
> > Since I've never actually contributed code to a GNU project before, are
> 
> Welcome!
> 
> > there certain rules and regulations of some sort that I have to comply
> with
> 
> Your guessed right: sure there are :-).  See, for example, (parts of) the
> GNU Coding Standards, <http://www.gnu.org/prep/standards/html_node/>.
> Not everything is relevant in your case, though.
> 
> > before I can send my patch here?
> 
> Feel free to simply send it here and we'll tell you what needs to be
> changed.
> 
> 
> Regards,
>  Thomas

Attachment: multiboot_memmap.patch
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]