diff options
author | Yaniv Kaul <ykaul@redhat.com> | 2019-02-27 15:48:42 +0200 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2019-05-06 13:59:55 +0000 |
commit | ecd1292eaea8873298357ab2d903110f713f7185 (patch) | |
tree | 6a636beb922873af037b160788947a3c8f3159d1 | |
parent | aff76bc71c1910128b04bb151817606a020ef575 (diff) |
mem-pool.{c|h}: minor changes
1. Removed some code that was not needed. It did not really do anything.
2. CALLOC -> MALLOC in one place.
Compile-tested only!
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
Change-Id: I4419161e1bb636158e32b5d33044b06f1eef2449
-rw-r--r-- | libglusterfs/src/mem-pool.c | 37 |
1 files changed, 12 insertions, 25 deletions
diff --git a/libglusterfs/src/mem-pool.c b/libglusterfs/src/mem-pool.c index ab78804012f..ca25ffcfdb8 100644 --- a/libglusterfs/src/mem-pool.c +++ b/libglusterfs/src/mem-pool.c @@ -643,7 +643,7 @@ mem_pool_new_fn(glusterfs_ctx_t *ctx, unsigned long sizeof_type, } pool = &pools[power - POOL_SMALLEST]; - new = GF_CALLOC(sizeof(struct mem_pool), 1, gf_common_mt_mem_pool); + new = GF_MALLOC(sizeof(struct mem_pool), gf_common_mt_mem_pool); if (!new) return NULL; @@ -671,15 +671,7 @@ mem_pool_new_fn(glusterfs_ctx_t *ctx, unsigned long sizeof_type, void * mem_get0(struct mem_pool *mem_pool) { - void *ptr = NULL; - - if (!mem_pool) { - gf_msg_callingfn("mem-pool", GF_LOG_ERROR, EINVAL, LG_MSG_INVALID_ARG, - "invalid argument"); - return NULL; - } - - ptr = mem_get(mem_pool); + void *ptr = mem_get(mem_pool); if (ptr) { #if defined(GF_DISABLE_MEMPOOL) memset(ptr, 0, mem_pool->sizeof_type); @@ -736,12 +728,14 @@ mem_get_pool_list(void) } pooled_obj_hdr_t * -mem_get_from_pool(struct mem_pool *mem_pool, struct mem_pool_shared *pool, - gf_boolean_t *hit) +mem_get_from_pool(struct mem_pool *mem_pool, struct mem_pool_shared *pool) { per_thread_pool_list_t *pool_list; per_thread_pool_t *pt_pool; pooled_obj_hdr_t *retval; +#ifdef DEBUG + gf_boolean_t hit = _gf_true; +#endif pool_list = mem_get_pool_list(); if (!pool_list || pool_list->poison) { @@ -755,10 +749,6 @@ mem_get_from_pool(struct mem_pool *mem_pool, struct mem_pool_shared *pool, pt_pool = &pool_list->pools[pool->power_of_two - POOL_SMALLEST]; } -#ifdef DEBUG - *hit = _gf_true; -#endif - (void)pthread_spin_lock(&pool_list->lock); retval = pt_pool->hot_list; @@ -778,7 +768,7 @@ mem_get_from_pool(struct mem_pool *mem_pool, struct mem_pool_shared *pool, retval = malloc((1 << pt_pool->parent->power_of_two) + sizeof(pooled_obj_hdr_t)); #ifdef DEBUG - *hit = _gf_false; + hit = _gf_false; #endif } } @@ -788,7 +778,7 @@ mem_get_from_pool(struct mem_pool *mem_pool, struct mem_pool_shared *pool, retval->pool = mem_pool; retval->power_of_two = mem_pool->pool->power_of_two; #ifdef DEBUG - if (*hit == _gf_true) + if (hit == _gf_true) GF_ATOMIC_INC(mem_pool->hit); else GF_ATOMIC_INC(mem_pool->miss); @@ -807,19 +797,16 @@ mem_get_from_pool(struct mem_pool *mem_pool, struct mem_pool_shared *pool, void * mem_get(struct mem_pool *mem_pool) { -#if defined(GF_DISABLE_MEMPOOL) - return GF_MALLOC(mem_pool->sizeof_type, gf_common_mt_mem_pool); -#else - pooled_obj_hdr_t *retval; - gf_boolean_t hit; - if (!mem_pool) { gf_msg_callingfn("mem-pool", GF_LOG_ERROR, EINVAL, LG_MSG_INVALID_ARG, "invalid argument"); return NULL; } - retval = mem_get_from_pool(mem_pool, NULL, &hit); +#if defined(GF_DISABLE_MEMPOOL) + return GF_MALLOC(mem_pool->sizeof_type, gf_common_mt_mem_pool); +#else + pooled_obj_hdr_t *retval = mem_get_from_pool(mem_pool, NULL); if (!retval) { return NULL; } |