Merge remote branch 'rransom/bug1125-022' into maint-0.2.2

This commit is contained in:
Nick Mathewson 2010-11-12 13:03:00 -05:00
commit 22318e952e

View File

@ -262,6 +262,7 @@ buf_shrink_freelists(int free_all)
{
#ifdef ENABLE_BUF_FREELISTS
int i;
disable_control_logging();
for (i = 0; freelists[i].alloc_size; ++i) {
int slack = freelists[i].slack;
assert_freelist_ok(&freelists[i]);
@ -269,16 +270,12 @@ buf_shrink_freelists(int free_all)
int n_to_free = free_all ? freelists[i].cur_length :
(freelists[i].lowest_length - slack);
int n_to_skip = freelists[i].cur_length - n_to_free;
int orig_length = freelists[i].cur_length;
int orig_n_to_free = n_to_free, n_freed=0;
int orig_n_to_skip = n_to_skip;
int new_length = n_to_skip;
chunk_t **chp = &freelists[i].head;
chunk_t *chunk;
log_info(LD_MM, "Cleaning freelist for %d-byte chunks: length %d, "
"keeping %d, dropping %d.",
(int)freelists[i].alloc_size, freelists[i].cur_length,
n_to_skip, n_to_free);
tor_assert(n_to_skip + n_to_free == freelists[i].cur_length);
while (n_to_skip) {
if (! (*chp)->next) {
log_warn(LD_BUG, "I wanted to skip %d chunks in the freelist for "
@ -313,10 +310,15 @@ buf_shrink_freelists(int free_all)
}
// tor_assert(!n_to_free);
freelists[i].cur_length = new_length;
log_info(LD_MM, "Cleaned freelist for %d-byte chunks: original "
"length %d, kept %d, dropped %d.",
(int)freelists[i].alloc_size, orig_length,
orig_n_to_skip, orig_n_to_free);
}
freelists[i].lowest_length = freelists[i].cur_length;
assert_freelist_ok(&freelists[i]);
}
enable_control_logging();
#else
(void) free_all;
#endif