aboutsummaryrefslogtreecommitdiffstats
path: root/src/prof.c
diff options
context:
space:
mode:
authorJason Evans <jasone@canonware.com>2017-03-17 02:45:12 -0700
committerJason Evans <jasone@canonware.com>2017-03-22 18:33:32 -0700
commit51a2ec92a10691bf5cee78093a0aa9a1fced351d (patch)
treeee9bdd1c4ffe3060cc4da8276a7ac754dbbbde96 /src/prof.c
parent4f341412e5e1370220382f04b1f7d9f70c694884 (diff)
downloadplatform_external_jemalloc_new-51a2ec92a10691bf5cee78093a0aa9a1fced351d.tar.gz
platform_external_jemalloc_new-51a2ec92a10691bf5cee78093a0aa9a1fced351d.tar.bz2
platform_external_jemalloc_new-51a2ec92a10691bf5cee78093a0aa9a1fced351d.zip
Remove extent dereferences from the deallocation fast paths.
Diffstat (limited to 'src/prof.c')
-rw-r--r--src/prof.c33
1 files changed, 12 insertions, 21 deletions
diff --git a/src/prof.c b/src/prof.c
index 13fa20d3..be06555b 100644
--- a/src/prof.c
+++ b/src/prof.c
@@ -582,8 +582,7 @@ prof_gctx_try_destroy(tsd_t *tsd, prof_tdata_t *tdata_self, prof_gctx_t *gctx,
prof_leave(tsd, tdata_self);
/* Destroy gctx. */
malloc_mutex_unlock(tsd_tsdn(tsd), gctx->lock);
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd), gctx), gctx,
- NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), gctx, NULL, true, true);
} else {
/*
* Compensate for increment in prof_tctx_destroy() or
@@ -697,8 +696,7 @@ prof_tctx_destroy(tsd_t *tsd, prof_tctx_t *tctx) {
}
if (destroy_tctx) {
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd), tctx), tctx,
- NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), tctx, NULL, true, true);
}
}
@@ -730,8 +728,8 @@ prof_lookup_global(tsd_t *tsd, prof_bt_t *bt, prof_tdata_t *tdata,
if (ckh_insert(tsd, &bt2gctx, btkey.v, gctx.v)) {
/* OOM. */
prof_leave(tsd, tdata);
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd),
- gctx.v), gctx.v, NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), gctx.v, NULL, true,
+ true);
return true;
}
new_gctx = true;
@@ -755,8 +753,7 @@ prof_lookup_global(tsd_t *tsd, prof_bt_t *bt, prof_tdata_t *tdata,
if (tgctx.v != NULL) {
/* Lost race to insert. */
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd),
- tgctx.v), tgctx.v, NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), tgctx.v, NULL, true, true);
}
}
prof_leave(tsd, tdata);
@@ -828,8 +825,7 @@ prof_lookup(tsd_t *tsd, prof_bt_t *bt) {
if (new_gctx) {
prof_gctx_try_destroy(tsd, tdata, gctx, tdata);
}
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd), ret.v),
- ret.v, NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), ret.v, NULL, true, true);
return NULL;
}
malloc_mutex_lock(tsd_tsdn(tsd), gctx->lock);
@@ -1240,9 +1236,8 @@ prof_gctx_finish(tsd_t *tsd, prof_gctx_tree_t *gctxs) {
to_destroy);
tctx_tree_remove(&gctx->tctxs,
to_destroy);
- idalloctm(tsd_tsdn(tsd),
- iealloc(tsd_tsdn(tsd), to_destroy),
- to_destroy, NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), to_destroy,
+ NULL, true, true);
} else {
next = NULL;
}
@@ -1910,8 +1905,7 @@ prof_tdata_init_impl(tsd_t *tsd, uint64_t thr_uid, uint64_t thr_discrim,
if (ckh_new(tsd, &tdata->bt2tctx, PROF_CKH_MINITEMS, prof_bt_hash,
prof_bt_keycomp)) {
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd), tdata), tdata,
- NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), tdata, NULL, true, true);
return NULL;
}
@@ -1967,12 +1961,10 @@ prof_tdata_destroy_locked(tsd_t *tsd, prof_tdata_t *tdata,
assert(prof_tdata_should_destroy_unlocked(tdata, even_if_attached));
if (tdata->thread_name != NULL) {
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd),
- tdata->thread_name), tdata->thread_name, NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), tdata->thread_name, NULL, true, true);
}
ckh_delete(tsd, &tdata->bt2tctx);
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd), tdata), tdata, NULL,
- true, true);
+ idalloctm(tsd_tsdn(tsd), tdata, NULL, true, true);
}
static void
@@ -2169,8 +2161,7 @@ prof_thread_name_set(tsd_t *tsd, const char *thread_name) {
}
if (tdata->thread_name != NULL) {
- idalloctm(tsd_tsdn(tsd), iealloc(tsd_tsdn(tsd),
- tdata->thread_name), tdata->thread_name, NULL, true, true);
+ idalloctm(tsd_tsdn(tsd), tdata->thread_name, NULL, true, true);
tdata->thread_name = NULL;
}
if (strlen(s) > 0) {