diff options
author | Jason Evans <jasone@FreeBSD.org> | 2006-01-19 18:37:30 +0000 |
---|---|---|
committer | Jason Evans <jasone@FreeBSD.org> | 2006-01-19 18:37:30 +0000 |
commit | a3bb22bc8e6ffaabf85eb590eed1d049dc23482c (patch) | |
tree | e98ec63db8e8e270780b9f21da05eaf9dc434f63 /lib | |
parent | 5f9cc6261de9cdb1a570b632a2a85a5a9e97fee7 (diff) | |
download | src-a3bb22bc8e6ffaabf85eb590eed1d049dc23482c.tar.gz src-a3bb22bc8e6ffaabf85eb590eed1d049dc23482c.zip |
Remove loops in arena_coalesce(). They are no longer necessary, now that
internal allocation does not rely on recursive arena use (base_arena was
removed in revision 1.95).
Notes
Notes:
svn path=/head/; revision=154555
Diffstat (limited to 'lib')
-rw-r--r-- | lib/libc/stdlib/malloc.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/lib/libc/stdlib/malloc.c b/lib/libc/stdlib/malloc.c index 75edd2fce0b3..32497305b798 100644 --- a/lib/libc/stdlib/malloc.c +++ b/lib/libc/stdlib/malloc.c @@ -1789,7 +1789,7 @@ arena_coalesce(arena_t *arena, region_t **reg, size_t size) assert(region_prev_free_get(&next->sep)); assert(region_next_size_get(&treg->sep) == next->prev.size); - while (region_prev_free_get(&treg->sep)) { + if (region_prev_free_get(&treg->sep)) { prev_size = treg->prev.size; prev = (region_t *)&((char *)treg)[-prev_size]; assert(region_next_free_get(&prev->sep)); @@ -1801,13 +1801,13 @@ arena_coalesce(arena_t *arena, region_t **reg, size_t size) treg = prev; #ifdef MALLOC_STATS - if (ret == false) - arena->stats.ncoalesce++; + arena->stats.ncoalesce++; #endif ret = true; } + assert(region_prev_free_get(&treg->sep) == false); - while (region_next_free_get(&next->sep)) { + if (region_next_free_get(&next->sep)) { next_size = region_next_size_get(&next->sep); nextnext = (region_t *)&((char *)next)[next_size]; assert(region_prev_free_get(&nextnext->sep)); @@ -1828,6 +1828,7 @@ arena_coalesce(arena_t *arena, region_t **reg, size_t size) next = (region_t *)&((char *)treg)[tsize]; } + assert(region_next_free_get(&next->sep) == false); /* Update header/footer. */ if (ret) { |