Simplify assert() to reduce memory usage
authorAntonio Nino Diaz <antonio.ninodiaz@arm.com>
Tue, 16 May 2017 08:52:02 +0000 (09:52 +0100)
committerAntonio Nino Diaz <antonio.ninodiaz@arm.com>
Tue, 16 May 2017 08:58:08 +0000 (09:58 +0100)
commit0da2fe7e29380f1b45286ae8a5a999547d250d33
tree9ff074119a21f48cb7c82bef5fadbdf52f64f0d0
parent823cc7a9d28aa77a8329f7581bb91038935b4df8
Simplify assert() to reduce memory usage

The behaviour of assert() now depends on the value of the new optional
platform define `PLAT_LOG_LEVEL_ASSERT`. This defaults to `LOG_LEVEL` if
not defined by the platform.

- If `PLAT_LOG_LEVEL_ASSERT` >= `LOG_LEVEL_VERBOSE`, it prints the file
  name, line and asserted expression.
- If `PLAT_LOG_LEVEL_ASSERT` >= `LOG_LEVEL_INFO`, it prints the file
  name and line.
- If not, it doesn't print anything.

Note the old behaviour was to print the function name whereas now it
prints the file name. This reduces memory usage because the file name is
shared between all assert calls in a given file. Also, the default
behaviour in debug builds is to no longer print the asserted expression,
greatly reducing the string usage.

For FVP debug builds this change saves approximately:

              No TBBR    TBBR
        BL1    1.6 KB   2.2 KB
        BL2    1.7 KB   2.1 KB
        BL31   2.6 KB   3.3 KB

Change-Id: I2947569d593df0b25611dc3c7a6096f42155c115
Signed-off-by: Antonio Nino Diaz <antonio.ninodiaz@arm.com>
docs/porting-guide.md
include/lib/stdlib/assert.h
lib/stdlib/assert.c