[U-Boot] [PATCH] env_mmc: fix buffer allocation for armv7
Tom Rini
trini at ti.com
Fri Oct 4 19:02:03 CEST 2013
On Fri, Oct 04, 2013 at 03:48:03PM +0200, Markus Niebel wrote:
> From: Markus Niebel <Markus.Niebel at tqs.de>
>
> commit d196bd880347373237d73e0d115b4d51c68cf2ad adds
> redundand environment to mmc. The usage of malloc in
> env_relocate_spec triggers cache errors on armv7.
>
> Tested on a not mainlined i.MX53 board:
>
> Board: TQMa53
> I2C: ready
> DRAM: 512 MiB
> MMC: FSL_SDHC: 0, FSL_SDHC: 1
> ERROR: v7_dcache_inval_range - start address is not aligned - 0x8f57c2d8
> ERROR: v7_dcache_inval_range - stop address is not aligned - 0x8f57e2d8
> ERROR: v7_dcache_inval_range - start address is not aligned - 0x8f57e2e0
> ERROR: v7_dcache_inval_range - stop address is not aligned - 0x8f5802e0
> Using default environment
>
> Signed-off-by: Markus Niebel <Markus.Niebel at tqs.de>
I really don't like this. We're now allocating for example 256KiB on
the stack, rather than malloc. I posted a patch recently to convert the
non-redundant case to malloc instead for this reason. I believe the
answer is we need to be using memalign here, like
common/bouncebuf.c::bounce_buffer_start does. Can you do this? If not,
can you test a patch? Thanks.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131004/c09e8ca6/attachment.pgp>
More information about the U-Boot
mailing list