fix: fix some bugs

This commit is contained in:
Renge 2022-03-26 11:32:36 -04:00
parent 85fa313fcb
commit c3dbf2030c
3 changed files with 76 additions and 29 deletions

View File

@ -3,5 +3,11 @@
int main(int argc, char const *argv[]) int main(int argc, char const *argv[])
{ {
size_t sz_x = sizeof(int), sz_y = 10, sz_x1 = sizeof(int) * 20;
void *x = sf_malloc(sz_x);
/* void *y = */ sf_malloc(sz_y);
sf_realloc(x, sz_x1);
sf_show_heap();
return EXIT_SUCCESS; return EXIT_SUCCESS;
} }

View File

@ -140,6 +140,9 @@ void *sf_malloc(sf_size_t size)
set_entire_header(last, size, get_block_size(last->header), 1, get_prv_alloc(last->header), 0, 0); set_entire_header(last, size, get_block_size(last->header), 1, get_prv_alloc(last->header), 0, 0);
set_entire_header(epi, 0, 0, 1, get_alloc(epi->prev_footer), 0, 1); set_entire_header(epi, 0, 0, 1, get_alloc(epi->prev_footer), 0, 1);
total_block_size += get_block_size(last->header);
add_payload(size);
return last->body.payload; return last->body.payload;
} }
@ -182,12 +185,15 @@ void *sf_realloc(void *pp, sf_size_t rsize)
sf_size_t new_size = get_min_size(rsize); sf_size_t new_size = get_min_size(rsize);
if (new_size > size) if (new_size > size)
{ {
int payload = (int)get_payload_size(block->header);
add_payload(-1 * payload);
sf_block *new; sf_block *new;
if ((new = sf_malloc(rsize)) == NULL) if ((new = sf_malloc(rsize)) == NULL)
return NULL; return NULL;
new = (sf_block *)(((intptr_t) new) - 2 * sizeof(sf_header)); new = (sf_block *)(((intptr_t) new) - 2 * sizeof(sf_header));
memcpy(new->body.payload, block->body.payload, size); memcpy(new->body.payload, block->body.payload, size);
sf_free(pp); sf_free(pp);
add_payload(payload);
return new->body.payload; return new->body.payload;
} }
else else

View File

@ -95,6 +95,11 @@ Test(sfmm_basecode_suite, malloc_an_int, .timeout = TEST_TIMEOUT)
cr_assert(sf_errno == 0, "sf_errno is not zero!"); cr_assert(sf_errno == 0, "sf_errno is not zero!");
cr_assert(sf_mem_start() + PAGE_SZ == sf_mem_end(), "Allocated more than necessary!"); cr_assert(sf_mem_start() + PAGE_SZ == sf_mem_end(), "Allocated more than necessary!");
cr_assert_eq(sf_internal_fragmentation(), 4 / 32.0, "Wrong number of sf_internal_fragmentation(): (exp=%f, found=%f)",
sf_internal_fragmentation(), 4 / 32.0);
cr_assert_eq(sf_peak_utilization(), 4 / 1024.0, "Wrong number of sf_peak_utilization(): (exp=%f, found=%f)",
sf_peak_utilization(), 4 / 1024.0);
} }
Test(sfmm_basecode_suite, malloc_four_pages, .timeout = TEST_TIMEOUT) Test(sfmm_basecode_suite, malloc_four_pages, .timeout = TEST_TIMEOUT)
@ -106,6 +111,11 @@ Test(sfmm_basecode_suite, malloc_four_pages, .timeout = TEST_TIMEOUT)
assert_quick_list_block_count(0, 0); assert_quick_list_block_count(0, 0);
assert_free_block_count(0, 0); assert_free_block_count(0, 0);
cr_assert(sf_errno == 0, "sf_errno is not 0!"); cr_assert(sf_errno == 0, "sf_errno is not 0!");
cr_assert_eq(sf_internal_fragmentation(), 4032 / 4048.0, "Wrong number of sf_internal_fragmentation(): (exp=%f, found=%f)",
sf_internal_fragmentation(), 4032 / 4048.0);
cr_assert_eq(sf_peak_utilization(), 4032 / 4096.0, "Wrong number of sf_peak_utilization(): (exp=%f, found=%f)",
sf_peak_utilization(), 4032 / 4096.0);
} }
Test(sfmm_basecode_suite, malloc_too_large, .timeout = TEST_TIMEOUT) Test(sfmm_basecode_suite, malloc_too_large, .timeout = TEST_TIMEOUT)
@ -135,6 +145,10 @@ Test(sfmm_basecode_suite, free_quick, .timeout = TEST_TIMEOUT)
assert_free_block_count(0, 1); assert_free_block_count(0, 1);
assert_free_block_count(864, 1); assert_free_block_count(864, 1);
cr_assert(sf_errno == 0, "sf_errno is not zero!"); cr_assert(sf_errno == 0, "sf_errno is not zero!");
cr_assert_eq(sf_internal_fragmentation(), 9 / 64.0, "Wrong number of sf_internal_fragmentation(): (exp=%f, found=%f)",
sf_internal_fragmentation(), 9 / 64.0);
cr_assert_eq(sf_peak_utilization(), 41 / 1024.0, "Wrong number of sf_peak_utilization(): (exp=%f, found=%f)",
sf_peak_utilization(), 41 / 1024.0);
} }
Test(sfmm_basecode_suite, free_no_coalesce, .timeout = TEST_TIMEOUT) Test(sfmm_basecode_suite, free_no_coalesce, .timeout = TEST_TIMEOUT)
@ -153,6 +167,10 @@ Test(sfmm_basecode_suite, free_no_coalesce, .timeout = TEST_TIMEOUT)
assert_free_block_count(704, 1); assert_free_block_count(704, 1);
cr_assert(sf_errno == 0, "sf_errno is not zero!"); cr_assert(sf_errno == 0, "sf_errno is not zero!");
cr_assert_eq(sf_internal_fragmentation(), 9 / 64.0, "Wrong number of sf_internal_fragmentation(): (exp=%f, found=%f)",
sf_internal_fragmentation(), 9 / 64.0);
cr_assert_eq(sf_peak_utilization(), 209 / 1024.0, "Wrong number of sf_peak_utilization(): (exp=%f, found=%f)",
sf_peak_utilization(), 209 / 1024.0);
} }
Test(sfmm_basecode_suite, free_coalesce, .timeout = TEST_TIMEOUT) Test(sfmm_basecode_suite, free_coalesce, .timeout = TEST_TIMEOUT)
@ -173,6 +191,10 @@ Test(sfmm_basecode_suite, free_coalesce, .timeout = TEST_TIMEOUT)
assert_free_block_count(528, 1); assert_free_block_count(528, 1);
cr_assert(sf_errno == 0, "sf_errno is not zero!"); cr_assert(sf_errno == 0, "sf_errno is not zero!");
cr_assert_eq(sf_internal_fragmentation(), 12 / 64.0, "Wrong number of sf_internal_fragmentation(): (exp=%f, found=%f)",
sf_internal_fragmentation(), 12 / 64.0);
cr_assert_eq(sf_peak_utilization(), 512 / 1024.0, "Wrong number of sf_peak_utilization(): (exp=%f, found=%f)",
sf_peak_utilization(), 512 / 1024.0);
} }
Test(sfmm_basecode_suite, freelist, .timeout = TEST_TIMEOUT) Test(sfmm_basecode_suite, freelist, .timeout = TEST_TIMEOUT)
@ -200,6 +222,10 @@ Test(sfmm_basecode_suite, freelist, .timeout = TEST_TIMEOUT)
cr_assert_eq(&bp->header, (char *)y - 8, cr_assert_eq(&bp->header, (char *)y - 8,
"Wrong first block in free list %d: (found=%p, exp=%p)", "Wrong first block in free list %d: (found=%p, exp=%p)",
i, &bp->header, (char *)y - 8); i, &bp->header, (char *)y - 8);
cr_assert_eq(sf_internal_fragmentation(), 550 / 592.0, "Wrong number of sf_internal_fragmentation(): (exp=%f, found=%f)",
sf_internal_fragmentation(), 550 / 592.0);
cr_assert_eq(sf_peak_utilization(), 1000 / 2048.0, "Wrong number of sf_peak_utilization(): (exp=%f, found=%f)",
sf_peak_utilization(), 1000 / 2048.0);
} }
Test(sfmm_basecode_suite, realloc_larger_block, .timeout = TEST_TIMEOUT) Test(sfmm_basecode_suite, realloc_larger_block, .timeout = TEST_TIMEOUT)
@ -223,6 +249,12 @@ Test(sfmm_basecode_suite, realloc_larger_block, .timeout = TEST_TIMEOUT)
assert_quick_list_block_count(32, 1); assert_quick_list_block_count(32, 1);
assert_free_block_count(0, 1); assert_free_block_count(0, 1);
assert_free_block_count(816, 1); assert_free_block_count(816, 1);
cr_assert_eq(sf_internal_fragmentation(), 90 / 128.0, "Wrong number of sf_internal_fragmentation(): (exp=%f, found=%f)",
sf_internal_fragmentation(), 90 / 128.0);
cr_assert_eq(sf_peak_utilization(), 90 / 1024.0, "Wrong number of sf_peak_utilization(): (exp=%f, found=%f)",
sf_peak_utilization(), 90 / 1024.0);
} }
Test(sfmm_basecode_suite, realloc_smaller_block_splinter, .timeout = TEST_TIMEOUT) Test(sfmm_basecode_suite, realloc_smaller_block_splinter, .timeout = TEST_TIMEOUT)
@ -334,7 +366,6 @@ Test(sfmm_basecode_suite, quicklist2, .timeout = TEST_TIMEOUT)
assert_free_block_count(0, 1); assert_free_block_count(0, 1);
assert_free_block_count(816, 1); assert_free_block_count(816, 1);
// First block in list should be the most recently freed block not in quick list. // First block in list should be the most recently freed block not in quick list.
int i = 0; int i = 0;
sf_block *bp = sf_quick_lists[i].first; sf_block *bp = sf_quick_lists[i].first;
@ -343,7 +374,8 @@ Test(sfmm_basecode_suite, quicklist2, .timeout = TEST_TIMEOUT)
i, &bp->header, (char *)x1 - 8); i, &bp->header, (char *)x1 - 8);
} }
Test(sfmm_basecode_suite, realloc1, .timeout = TEST_TIMEOUT) { Test(sfmm_basecode_suite, realloc1, .timeout = TEST_TIMEOUT)
{
char *x1 = sf_malloc(2000 - 8); char *x1 = sf_malloc(2000 - 8);
char *x2 = sf_malloc(10000 - 8); char *x2 = sf_malloc(10000 - 8);
sf_free(x1); sf_free(x1);
@ -354,7 +386,8 @@ Test(sfmm_basecode_suite, realloc1, .timeout = TEST_TIMEOUT) {
assert_free_block_count(528, 1); assert_free_block_count(528, 1);
} }
Test(sfmm_basecode_suite, realloc2, .timeout = TEST_TIMEOUT) { Test(sfmm_basecode_suite, realloc2, .timeout = TEST_TIMEOUT)
{
int *x1 = sf_malloc(sizeof(int)); int *x1 = sf_malloc(sizeof(int));
sf_malloc(sizeof(int)); sf_malloc(sizeof(int));
*x1 = 114514; *x1 = 114514;
@ -371,12 +404,14 @@ Test(sfmm_basecode_suite, realloc2, .timeout = TEST_TIMEOUT) {
114514, (int)*x3); 114514, (int)*x3);
} }
Test(sfmm_basecode_suite, freepro, .signal=SIGABRT, .timeout = TEST_TIMEOUT) { Test(sfmm_basecode_suite, freepro, .signal = SIGABRT, .timeout = TEST_TIMEOUT)
{
sf_malloc(sizeof(int)); sf_malloc(sizeof(int));
sf_free(sf_mem_start()); sf_free(sf_mem_start());
} }
Test(sfmm_basecode_suite, freeepi, .signal=SIGABRT, .timeout = TEST_TIMEOUT) { Test(sfmm_basecode_suite, freeepi, .signal = SIGABRT, .timeout = TEST_TIMEOUT)
{
sf_malloc(sizeof(int)); sf_malloc(sizeof(int));
sf_free(((void *)(intptr_t)sf_mem_end()) - 2 * sizeof(sf_header)); sf_free(((void *)(intptr_t)sf_mem_end()) - 2 * sizeof(sf_header));
} }