diff options
| author | Nicholas Piggin <npiggin@gmail.com> | 2019-11-11 15:44:41 +1000 |
|---|---|---|
| committer | Oliver O'Halloran <oohall@gmail.com> | 2019-11-11 18:35:00 +1100 |
| commit | 3734172b0f40aa4b742f65aa8293476ca26916e5 (patch) | |
| tree | 4dee3229cccd86ecc572bb42a2e4403caa8faa88 /core/mem_region.c | |
| parent | 6b512fceb4210d5cf166912ef72c90cd29caec67 (diff) | |
| download | skiboot-3734172b0f40aa4b742f65aa8293476ca26916e5.zip skiboot-3734172b0f40aa4b742f65aa8293476ca26916e5.tar.gz skiboot-3734172b0f40aa4b742f65aa8293476ca26916e5.tar.bz2 | |
build: fix non-constant build asserts
BUILD_ASSERT can not be used for constants generated by the assembler
or linker. This results in variable length arrays that do not catch
the failure condition. This was caught by sparse.
Remove these and add some equivalent as/ld checks which actually do
the right thing.
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
Diffstat (limited to 'core/mem_region.c')
| -rw-r--r-- | core/mem_region.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/core/mem_region.c b/core/mem_region.c index 06cef1b..1bb8780 100644 --- a/core/mem_region.c +++ b/core/mem_region.c @@ -1058,10 +1058,6 @@ void mem_region_init(void) struct dt_node *i; bool rc; - /* Ensure we have no collision between skiboot core and our heap */ - extern char _end[]; - BUILD_ASSERT(HEAP_BASE >= (uint64_t)_end); - /* * Add associativity properties outside of the lock * to avoid recursive locking caused by allocations |