All of lore.kernel.org
 help / color / mirror / Atom feed
From: Robert Millan <rmh@aybabtu.com>
To: The development of GRUB 2 <grub-devel@gnu.org>
Cc: Sven Luther <sven@powerlinux.fr>
Subject: Re: [PATCH] efika memory issues
Date: Tue, 31 Jul 2007 21:42:32 +0200	[thread overview]
Message-ID: <20070731194232.GA19385@aragorn> (raw)
In-Reply-To: <fb412d760707310855n31bdeea5j6e1201d2c9d6e9fe@mail.gmail.com>

On Tue, Jul 31, 2007 at 10:55:09AM -0500, Hollis Blanchard wrote:
> On 7/31/07, Robert Millan <rmh@aybabtu.com> wrote:
> > On Mon, Jul 30, 2007 at 05:35:28PM -0500, Hollis Blanchard wrote:
> > > >         When GRUB_IEEE1275_FLAG_EFIKA_SECRET_AVAILABLE_REGION was set,
> > > >         release hardcoded 0x4000:0xffc000 region.
> > >
> > > Hmm, does this actually work? Since GRUB itself falls within that
> > > regions, the grub_ieee1275_release() call will mark all the memory
> > > that the GRUB kernel and modules occupy as free, which means they will
> > > be clobbered by heap usage.
> >
> > Didn't think of this.. (by chance it worked here, though).  How about using
> > max(_end,0x4000) instead of 0x4000 ?
> >
> > (and adjusting region length not to overflow 0x1000000)
> 
> That won't quite work because the modules are loaded in the area
> immediately following the kernel. However, because they are added in a
> post-processing step, we don't know their end address at link time.
> 
> I think the simplest solution will be to allocate Efika's heap high,
> as we talked about earlier. If the first "available" region is above
> 4MB, just reserve 4MB and don't worry about where it is. In general we
> should try to keep 4MB free, but at the end of the day it's Efika's
> fault and probably won't be a big deal for them anyways.

Ok, let's do that.  But what do we do if the first available region is
just a few kBs below 4MB?  Or if it's just too small?  I think we need to
determine what is the minimum heap size we want to accept, such that the
ability to keep 4MB free still compensates having a small heap.

Any suggestion?

-- 
Robert Millan

My spam trap is honeypot@aybabtu.com.  Note: this address is only intended
for spam harvesters.  Writing to it will get you added to my black list.



  reply	other threads:[~2007-07-31 19:39 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-02 18:16 another regression on Efika Robert Millan
2007-07-02 21:14 ` Robert Millan
2007-07-04 16:18 ` Hollis Blanchard
2007-07-10 10:04   ` Jordi Mallach
2007-07-10 14:08     ` memory management issue (Re: another regression on Efika) Robert Millan
2007-07-10 20:32       ` Hollis Blanchard
2007-07-13 22:08         ` Robert Millan
2007-07-24 19:59           ` Robert Millan
2007-07-25 17:51             ` Hollis Blanchard
2007-07-25 20:08               ` Robert Millan
2007-07-25 23:25                 ` Hollis Blanchard
2007-07-27  8:05                   ` Robert Millan
     [not found]                     ` <20070727103310.GA1539@powerlinux.fr>
2007-07-27 19:15                       ` Robert Millan
2007-07-29 19:30                         ` Robert Millan
2007-07-30 20:11                           ` [PATCH] efika memory issues Robert Millan
2007-07-30 22:35                             ` Hollis Blanchard
2007-07-31 14:38                               ` Robert Millan
2007-07-31 15:55                                 ` Hollis Blanchard
2007-07-31 19:42                                   ` Robert Millan [this message]
2007-08-01 17:34                                     ` Hollis Blanchard
2007-08-01 18:27                                       ` Robert Millan
2007-09-30 20:10                                         ` [PATCH] fix memory management on efika/pegasos Robert Millan
2007-10-01 18:18                                           ` Marcin Kurek
2007-10-01 22:25                                           ` Marcin Kurek
2007-10-02 19:42                                             ` Robert Millan
2007-10-03 23:38                                               ` Marcin Kurek
2007-10-04 20:47                                                 ` Robert Millan
2007-07-10 14:10     ` OF disk naming scheme (Re: another regression on Efika) Robert Millan
2007-07-10 19:26       ` Jordi Mallach
2007-07-13 22:16         ` Robert Millan
2007-07-22  9:33           ` Robert Millan
2007-07-22 14:37             ` Marco Gerards
2007-07-22 20:25               ` Robert Millan
2007-08-12 14:56                 ` proposed solution using "ofpathname -a" (Re: OF disk naming scheme) Robert Millan
2007-08-05 10:03 ` updated regression diff (Re: another regression on Efika) 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=20070731194232.GA19385@aragorn \
    --to=rmh@aybabtu.com \
    --cc=grub-devel@gnu.org \
    --cc=sven@powerlinux.fr \
    /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.