diff options
| author | Andrea Shepard <andrea@torproject.org> | 2014-04-29 02:18:34 -0700 |
|---|---|---|
| committer | Andrea Shepard <andrea@torproject.org> | 2014-04-29 02:18:34 -0700 |
| commit | 91ff10f6be984c6ca00a7240d32e79ab7db09c7c (patch) | |
| tree | f05158871709b439772283b0828a1110fa8e8513 | |
| parent | cdc64f020c926cd51b037b85b5eba5bf08d66e7d (diff) | |
| download | tor-91ff10f6be984c6ca00a7240d32e79ab7db09c7c.tar.gz tor-91ff10f6be984c6ca00a7240d32e79ab7db09c7c.zip | |
Make --disable-buf-freelists build and pass unit tests
| -rw-r--r-- | src/or/buffers.c | 6 | ||||
| -rw-r--r-- | src/test/test_buffers.c | 13 |
2 files changed, 14 insertions, 5 deletions
diff --git a/src/or/buffers.c b/src/or/buffers.c index fb186081cf..e54751db28 100644 --- a/src/or/buffers.c +++ b/src/or/buffers.c @@ -117,6 +117,9 @@ chunk_repack(chunk_t *chunk) chunk->data = &chunk->mem[0]; } +/** Keep track of total size of allocated chunks for consistency asserts */ +static size_t total_bytes_allocated_in_chunks = 0; + #if defined(ENABLE_BUF_FREELISTS) || defined(RUNNING_DOXYGEN) /** A freelist of chunks. */ typedef struct chunk_freelist_t { @@ -148,9 +151,6 @@ static chunk_freelist_t freelists[] = { * could help with? */ static uint64_t n_freelist_miss = 0; -/** DOCDOC */ -static size_t total_bytes_allocated_in_chunks = 0; - static void assert_freelist_ok(chunk_freelist_t *fl); /** Return the freelist to hold chunks of size <b>alloc</b>, or NULL if diff --git a/src/test/test_buffers.c b/src/test/test_buffers.c index 6dd7715936..c2cfd2700f 100644 --- a/src/test/test_buffers.c +++ b/src/test/test_buffers.c @@ -481,13 +481,22 @@ test_buffer_allocation_tracking(void *arg) fetch_from_buf(junk, 4096, buf1); /* drop a 1k chunk... */ tt_int_op(buf_allocation(buf1), ==, 3*4096); /* now 3 4k chunks */ +#ifdef ENABLE_BUF_FREELISTS tt_int_op(buf_get_total_allocation(), ==, 16384); /* that chunk went onto the freelist. */ +#else + tt_int_op(buf_get_total_allocation(), ==, 12288); /* that chunk was really + freed. */ +#endif write_to_buf(junk, 4000, buf2); tt_int_op(buf_allocation(buf2), ==, 4096); /* another 4k chunk. */ - tt_int_op(buf_get_total_allocation(), ==, 16384); /* that chunk came from - the freelist. */ + /* + * If we're using freelists, size stays at 16384 because we just pulled a + * chunk from the freelist. If we aren't, we bounce back up to 16384 by + * allocating a new chunk. + */ + tt_int_op(buf_get_total_allocation(), ==, 16384); write_to_buf(junk, 4000, buf2); tt_int_op(buf_allocation(buf2), ==, 8192); /* another 4k chunk. */ tt_int_op(buf_get_total_allocation(), ==, 5*4096); /* that chunk was new. */ |