diff options
| author | Stewart Smith <stewart@linux.ibm.com> | 2018-11-29 15:28:28 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.ibm.com> | 2018-12-10 22:21:12 -0600 |
| commit | 685f0786a93528b24c73771b643a6183d4fd16dc (patch) | |
| tree | 2ef835eeca4015499ec8e824f7e820025cbbf596 /core/mem_region.c | |
| parent | 88bf48a2a5def0bb40c999dd098500f8cfc6a2b3 (diff) | |
| download | skiboot-685f0786a93528b24c73771b643a6183d4fd16dc.zip skiboot-685f0786a93528b24c73771b643a6183d4fd16dc.tar.gz skiboot-685f0786a93528b24c73771b643a6183d4fd16dc.tar.bz2 | |
mem_region.c: Move ifdef for MEM_POISON to shut up static analysis
The static analysis tool is arguably wrong and should go away.
But... I'm sick of keeping coming back to it and reviewing the false
positives enough to make a slight change to where ifdefs are.
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Diffstat (limited to 'core/mem_region.c')
| -rw-r--r-- | core/mem_region.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/core/mem_region.c b/core/mem_region.c index 7aac4e1..7455192 100644 --- a/core/mem_region.c +++ b/core/mem_region.c @@ -162,8 +162,6 @@ static void mem_poison(struct free_hdr *f) memset(f+1, POISON_MEM_REGION_WITH, poison_size); } -#else -static inline void mem_poison(struct free_hdr *f __unused) { } #endif /* Creates free block covering entire region. */ @@ -178,7 +176,9 @@ static void init_allocatable_region(struct mem_region *region) *tailer(f) = f->hdr.num_longs; list_head_init(®ion->free_list); list_add(®ion->free_list, &f->list); +#if POISON_MEM_REGION == 1 mem_poison(f); +#endif } static void make_free(struct mem_region *region, struct free_hdr *f, @@ -186,8 +186,12 @@ static void make_free(struct mem_region *region, struct free_hdr *f, { struct alloc_hdr *next; +#if POISON_MEM_REGION == 1 if (!skip_poison) mem_poison(f); +#else + (void)skip_poison; +#endif if (f->hdr.prev_free) { struct free_hdr *prev; |