public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Chin Liang See <clsee@altera.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] malloc_simple: Add simple malloc free function
Date: Wed, 3 Aug 2016 15:30:24 +0800	[thread overview]
Message-ID: <1470209424.1886.5.camel@altera.com> (raw)
In-Reply-To: <47474ef4-5fff-364e-4c2a-ba19f77db7b5@denx.de>

Hi Marek,

On Wed, 2016-08-03 at 08:58 +0200, Marek Vasut wrote:
> On 08/03/2016 05:24 AM, Chin Liang See wrote:
> > Enable a simple malloc implementation which will minimize
> > memory usage prior relocation. This is essential as memory
> > available prior location is internal memory and limited in
> > size.
> > 
> > This implementation will stored last 2 usage of malloc. When
> > free is invoked and the free address matched, we shall revert
> > to previous value of gd->malloc_ptr that we stored.
> 
> This looks unnecessarily convoluted and fragile design.
> What problem do you observe and on what platform ?

Actually this for our Arria10 SoC device. In order to get DDR working,
we need to program FPGA. To improve the usability, we put the FPGA
programming file (RBF) into FAT partition.

In that sense, we would need to use FAT driver prior relocation / DDR
available. Due to that, the malloc usage is high and memory available
is limited prior DDR available. 

The simple malloc helps but without the free, its consumed way too much
memory than saving. Hence this simple malloc free patch help. So I
believe this would benefits those who are executing complex operation
prior relocation :) 

Thanks
Chin Liang

  reply	other threads:[~2016-08-03  7:30 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-03  3:24 [U-Boot] [PATCH] malloc_simple: Add simple malloc free function Chin Liang See
2016-08-03  6:58 ` Marek Vasut
2016-08-03  7:30   ` Chin Liang See [this message]
2016-08-03  7:53     ` Marek Vasut
2016-08-03 13:41       ` Chin Liang See
2016-08-03 13:57         ` Marek Vasut
2016-08-03 15:22           ` Chin Liang See
2016-08-04  5:30             ` Marek Vasut
2016-08-04 15:12               ` Chin Liang See
2016-08-04 15:26                 ` Marek Vasut
2016-08-04  1:17 ` Simon Glass
2016-08-04 15:09   ` Chin Liang See

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=1470209424.1886.5.camel@altera.com \
    --to=clsee@altera.com \
    --cc=u-boot@lists.denx.de \
    /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