[U-Boot] malloc: solve dead code issue in memalign()

Tom Rini trini at konsulko.com
Mon Feb 8 21:49:24 CET 2016


On Thu, Feb 04, 2016 at 11:54:20AM -0700, Stephen Warren wrote:

> From: Stephen Warren <swarren at nvidia.com>
> 
> The recent change to memalign() caused the allocation failure detection
> code to be dead code; the "retry" logic is always activated under the same
> condition that the original failure detection code is, and also fully
> handles any possible failures. This patch solves the presence of dead
> code.
> 
> Two alternatives are possible:
> 
> a) Delete the now-dead test, and rely on the "retry" path to handle any
> allocation problems, as it does.
> 
> b) Make the "retry" path fall through to the existing (currently dead)
> failure detection code, thus making it not-dead.
> 
> (b) was chosen since it reduces the diff between U-Boot's and the upstream
> dlmalloc. This should make it marginally easier to import a new version of
> dlmalloc in the future.
> 
> Reported by: Coverity Scan
> Fixes: 4f144a416469 ("malloc: work around some memalign fragmentation issues")
> Signed-off-by: Stephen Warren <swarren at nvidia.com>
> Reviewed-by: Tom Rini <trini at konsulko.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160208/caeb8108/attachment.sig>


More information about the U-Boot mailing list