[U-Boot] [PATCH v4 05/13] Move memcpy(), memset() into new lib/membasic.c

Simon Glass sjg at chromium.org
Tue Mar 6 07:34:24 CET 2012


Hi Mike,

On Mon, Mar 5, 2012 at 9:05 PM, Mike Frysinger <vapier at gentoo.org> wrote:
> On Monday 20 February 2012 20:32:47 Simon Glass wrote:
>> These basic functions are needed by relocation. To avoid bringing in all
>> string.c functions (and the resulting code bloat for architectures where
>> -ffunction-sections is not used), move these into their own file.
>
> seems like fixing -f{data,function}-sections would be a more useful goal.  i
> don't know why arm doesn't support this today ... it's fairly easy to do.

I did enable it for Tegra at one point with no ill effects except for
breaking warmboot, which relied on function ordering. I notice the
no-toplevel-reorder flag on x86 but not with ARM. But I assumed there
must be some reason it is not used and thought this patch safer, since
it is just a minor tweak on a series that has has some review.

IXP does enable it, but not ARM in general. Most archs use
function-sections although not all. Perhaps there are some boards
which don't work with this flag?

Albert do you know?

Regards,
Simon

> -mike


More information about the U-Boot mailing list