projects
/
fio.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
fio.1: update description for 'buffer_pattern' and 'verify_pattern'
[fio.git]
/
smalloc.c
diff --git
a/smalloc.c
b/smalloc.c
index fc6ac526c33d95cac6c88f2bfe371f724f0d426c..8412e7518464d6dbe3e7f218e33e47e855e8e26f 100644
(file)
--- a/
smalloc.c
+++ b/
smalloc.c
@@
-8,41
+8,46
@@
#include <assert.h>
#include <string.h>
#include <unistd.h>
#include <assert.h>
#include <string.h>
#include <unistd.h>
+#include <inttypes.h>
#include <sys/types.h>
#include <limits.h>
#include <sys/types.h>
#include <limits.h>
+#include <fcntl.h>
#include "mutex.h"
#include "arch/arch.h"
#include "mutex.h"
#include "arch/arch.h"
+#include "os/os.h"
+#include "smalloc.h"
+#include "log.h"
-#define MP_SAFE /* define to make thread safe */
#define SMALLOC_REDZONE /* define to detect memory corruption */
#define SMALLOC_BPB 32 /* block size, bytes-per-bit in bitmap */
#define SMALLOC_BPI (sizeof(unsigned int) * 8)
#define SMALLOC_BPL (SMALLOC_BPB * SMALLOC_BPI)
#define SMALLOC_REDZONE /* define to detect memory corruption */
#define SMALLOC_BPB 32 /* block size, bytes-per-bit in bitmap */
#define SMALLOC_BPI (sizeof(unsigned int) * 8)
#define SMALLOC_BPL (SMALLOC_BPB * SMALLOC_BPI)
-#define INITIAL_SIZE 1
024*1024
/* new pool size */
-#define MAX_POOLS
128
/* maximum number of pools to setup */
+#define INITIAL_SIZE 1
6*1024*1024
/* new pool size */
+#define MAX_POOLS
8
/* maximum number of pools to setup */
#define SMALLOC_PRE_RED 0xdeadbeefU
#define SMALLOC_POST_RED 0x5aa55aa5U
unsigned int smalloc_pool_size = INITIAL_SIZE;
#define SMALLOC_PRE_RED 0xdeadbeefU
#define SMALLOC_POST_RED 0x5aa55aa5U
unsigned int smalloc_pool_size = INITIAL_SIZE;
+#ifdef SMALLOC_REDZONE
+static const int int_mask = sizeof(int) - 1;
+#endif
struct pool {
struct fio_mutex *lock; /* protects this pool */
void *map; /* map of blocks */
unsigned int *bitmap; /* blocks free/busy map */
struct pool {
struct fio_mutex *lock; /* protects this pool */
void *map; /* map of blocks */
unsigned int *bitmap; /* blocks free/busy map */
- unsigned int free_blocks; /* free blocks */
- unsigned int nr_blocks; /* total blocks */
- unsigned int next_non_full;
- int fd; /* memory backing fd */
- char file[PATH_MAX]; /* filename for fd */
- unsigned int mmap_size;
+ size_t free_blocks; /* free blocks */
+ size_t nr_blocks; /* total blocks */
+ size_t next_non_full;
+ size_t mmap_size;
};
struct block_hdr {
};
struct block_hdr {
-
unsigned in
t size;
+
size_
t size;
#ifdef SMALLOC_REDZONE
unsigned int prered;
#endif
#ifdef SMALLOC_REDZONE
unsigned int prered;
#endif
@@
-51,42
+56,36
@@
struct block_hdr {
static struct pool mp[MAX_POOLS];
static unsigned int nr_pools;
static unsigned int last_pool;
static struct pool mp[MAX_POOLS];
static unsigned int nr_pools;
static unsigned int last_pool;
-static struct fio_
mutex
*lock;
+static struct fio_
rwlock
*lock;
static inline void pool_lock(struct pool *pool)
{
static inline void pool_lock(struct pool *pool)
{
- if (pool->lock)
- fio_mutex_down(pool->lock);
+ fio_mutex_down(pool->lock);
}
static inline void pool_unlock(struct pool *pool)
{
}
static inline void pool_unlock(struct pool *pool)
{
- if (pool->lock)
- fio_mutex_up(pool->lock);
+ fio_mutex_up(pool->lock);
}
static inline void global_read_lock(void)
{
}
static inline void global_read_lock(void)
{
- if (lock)
- fio_mutex_down_read(lock);
+ fio_rwlock_read(lock);
}
static inline void global_read_unlock(void)
{
}
static inline void global_read_unlock(void)
{
- if (lock)
- fio_mutex_up_read(lock);
+ fio_rwlock_unlock(lock);
}
static inline void global_write_lock(void)
{
}
static inline void global_write_lock(void)
{
- if (lock)
- fio_mutex_down_write(lock);
+ fio_rwlock_write(lock);
}
static inline void global_write_unlock(void)
{
}
static inline void global_write_unlock(void)
{
- if (lock)
- fio_mutex_up_write(lock);
+ fio_rwlock_unlock(lock);
}
static inline int ptr_valid(struct pool *pool, void *ptr)
}
static inline int ptr_valid(struct pool *pool, void *ptr)
@@
-96,13
+95,13
@@
static inline int ptr_valid(struct pool *pool, void *ptr)
return (ptr >= pool->map) && (ptr < pool->map + pool_size);
}
return (ptr >= pool->map) && (ptr < pool->map + pool_size);
}
-static inline
unsigned int size_to_blocks(unsigned in
t size)
+static inline
size_t size_to_blocks(size_
t size)
{
return (size + SMALLOC_BPB - 1) / SMALLOC_BPB;
}
static int blocks_iter(struct pool *pool, unsigned int pool_idx,
{
return (size + SMALLOC_BPB - 1) / SMALLOC_BPB;
}
static int blocks_iter(struct pool *pool, unsigned int pool_idx,
- unsigned int idx,
unsigned in
t nr_blocks,
+ unsigned int idx,
size_
t nr_blocks,
int (*func)(unsigned int *map, unsigned int mask))
{
int (*func)(unsigned int *map, unsigned int mask))
{
@@
-157,19
+156,19
@@
static int mask_set(unsigned int *map, unsigned int mask)
}
static int blocks_free(struct pool *pool, unsigned int pool_idx,
}
static int blocks_free(struct pool *pool, unsigned int pool_idx,
- unsigned int idx,
unsigned in
t nr_blocks)
+ unsigned int idx,
size_
t nr_blocks)
{
return blocks_iter(pool, pool_idx, idx, nr_blocks, mask_cmp);
}
static void set_blocks(struct pool *pool, unsigned int pool_idx,
{
return blocks_iter(pool, pool_idx, idx, nr_blocks, mask_cmp);
}
static void set_blocks(struct pool *pool, unsigned int pool_idx,
- unsigned int idx,
unsigned in
t nr_blocks)
+ unsigned int idx,
size_
t nr_blocks)
{
blocks_iter(pool, pool_idx, idx, nr_blocks, mask_set);
}
static void clear_blocks(struct pool *pool, unsigned int pool_idx,
{
blocks_iter(pool, pool_idx, idx, nr_blocks, mask_set);
}
static void clear_blocks(struct pool *pool, unsigned int pool_idx,
- unsigned int idx,
unsigned in
t nr_blocks)
+ unsigned int idx,
size_
t nr_blocks)
{
blocks_iter(pool, pool_idx, idx, nr_blocks, mask_clear);
}
{
blocks_iter(pool, pool_idx, idx, nr_blocks, mask_clear);
}
@@
-177,19
+176,15
@@
static void clear_blocks(struct pool *pool, unsigned int pool_idx,
static int find_next_zero(int word, int start)
{
assert(word != -1U);
static int find_next_zero(int word, int start)
{
assert(word != -1U);
- word >>=
(start + 1)
;
- return ffz(word) + start
+ 1
;
+ word >>=
start
;
+ return ffz(word) + start;
}
static int add_pool(struct pool *pool, unsigned int alloc_size)
{
}
static int add_pool(struct pool *pool, unsigned int alloc_size)
{
+ int bitmap_blocks;
+ int mmap_flags;
void *ptr;
void *ptr;
- int fd, bitmap_blocks;
-
- strcpy(pool->file, "/tmp/.fio_smalloc.XXXXXX");
- fd = mkstemp(pool->file);
- if (fd < 0)
- goto out_close;
#ifdef SMALLOC_REDZONE
alloc_size += sizeof(unsigned int);
#ifdef SMALLOC_REDZONE
alloc_size += sizeof(unsigned int);
@@
-203,57
+198,55
@@
static int add_pool(struct pool *pool, unsigned int alloc_size)
bitmap_blocks = alloc_size / SMALLOC_BPL;
alloc_size += bitmap_blocks * sizeof(unsigned int);
pool->mmap_size = alloc_size;
bitmap_blocks = alloc_size / SMALLOC_BPL;
alloc_size += bitmap_blocks * sizeof(unsigned int);
pool->mmap_size = alloc_size;
-
+
pool->nr_blocks = bitmap_blocks;
pool->free_blocks = bitmap_blocks * SMALLOC_BPB;
pool->nr_blocks = bitmap_blocks;
pool->free_blocks = bitmap_blocks * SMALLOC_BPB;
- if (ftruncate(fd, alloc_size) < 0)
- goto out_unlink;
+ mmap_flags = OS_MAP_ANON;
+#ifdef CONFIG_ESX
+ mmap_flags |= MAP_PRIVATE;
+#else
+ mmap_flags |= MAP_SHARED;
+#endif
+ ptr = mmap(NULL, alloc_size, PROT_READ|PROT_WRITE, mmap_flags, -1, 0);
- ptr = mmap(NULL, alloc_size, PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
if (ptr == MAP_FAILED)
if (ptr == MAP_FAILED)
- goto out_
unlink
;
+ goto out_
fail
;
memset(ptr, 0, alloc_size);
pool->map = ptr;
pool->bitmap = (void *) ptr + (pool->nr_blocks * SMALLOC_BPL);
memset(ptr, 0, alloc_size);
pool->map = ptr;
pool->bitmap = (void *) ptr + (pool->nr_blocks * SMALLOC_BPL);
-#ifdef MP_SAFE
- pool->lock = fio_mutex_init(1);
+ pool->lock = fio_mutex_init(FIO_MUTEX_UNLOCKED);
if (!pool->lock)
if (!pool->lock)
- goto out_unlink;
-#endif
-
- /*
- * Unlink pool file now. It wont get deleted until the fd is closed,
- * which happens both for cleanup or unexpected quit. This way we
- * don't leave temp files around in case of a crash.
- */
- pool->fd = fd;
- unlink(pool->file);
+ goto out_fail;
nr_pools++;
return 0;
nr_pools++;
return 0;
-out_
unlink
:
-
fprintf(stderr,
"smalloc: failed adding pool\n");
+out_
fail
:
+
log_err(
"smalloc: failed adding pool\n");
if (pool->map)
munmap(pool->map, pool->mmap_size);
if (pool->map)
munmap(pool->map, pool->mmap_size);
- unlink(pool->file);
-out_close:
- if (fd >= 0)
- close(fd);
return 1;
}
void sinit(void)
{
return 1;
}
void sinit(void)
{
- int ret;
+ int
i,
ret;
-#ifdef MP_SAFE
- lock = fio_mutex_rw_init();
-#endif
- ret = add_pool(&mp[0], INITIAL_SIZE);
- assert(!ret);
+ lock = fio_rwlock_init();
+
+ for (i = 0; i < MAX_POOLS; i++) {
+ ret = add_pool(&mp[i], INITIAL_SIZE);
+ if (ret)
+ break;
+ }
+
+ /*
+ * If we added at least one pool, we should be OK for most
+ * cases.
+ */
+ assert(i);
}
static void cleanup_pool(struct pool *pool)
}
static void cleanup_pool(struct pool *pool)
@@
-262,7
+255,6
@@
static void cleanup_pool(struct pool *pool)
* This will also remove the temporary file we used as a backing
* store, it was already unlinked
*/
* This will also remove the temporary file we used as a backing
* store, it was already unlinked
*/
- close(pool->fd);
munmap(pool->map, pool->mmap_size);
if (pool->lock)
munmap(pool->map, pool->mmap_size);
if (pool->lock)
@@
-277,13
+269,23
@@
void scleanup(void)
cleanup_pool(&mp[i]);
if (lock)
cleanup_pool(&mp[i]);
if (lock)
- fio_
mutex
_remove(lock);
+ fio_
rwlock
_remove(lock);
}
#ifdef SMALLOC_REDZONE
}
#ifdef SMALLOC_REDZONE
+static void *postred_ptr(struct block_hdr *hdr)
+{
+ uintptr_t ptr;
+
+ ptr = (uintptr_t) hdr + hdr->size - sizeof(unsigned int);
+ ptr = (ptr + int_mask) & ~int_mask;
+
+ return (void *) ptr;
+}
+
static void fill_redzone(struct block_hdr *hdr)
{
static void fill_redzone(struct block_hdr *hdr)
{
- unsigned int *postred =
(void *) hdr + hdr->size - sizeof(unsigned int
);
+ unsigned int *postred =
postred_ptr(hdr
);
hdr->prered = SMALLOC_PRE_RED;
*postred = SMALLOC_POST_RED;
hdr->prered = SMALLOC_PRE_RED;
*postred = SMALLOC_POST_RED;
@@
-291,17
+293,17
@@
static void fill_redzone(struct block_hdr *hdr)
static void sfree_check_redzone(struct block_hdr *hdr)
{
static void sfree_check_redzone(struct block_hdr *hdr)
{
- unsigned int *postred =
(void *) hdr + hdr->size - sizeof(unsigned int
);
+ unsigned int *postred =
postred_ptr(hdr
);
if (hdr->prered != SMALLOC_PRE_RED) {
if (hdr->prered != SMALLOC_PRE_RED) {
- fprintf(stderr, "smalloc pre redzone destroyed!\n");
-
fprintf(stderr, "
ptr=%p, prered=%x, expected %x\n",
+ log_err("smalloc pre redzone destroyed!\n"
+
"
ptr=%p, prered=%x, expected %x\n",
hdr, hdr->prered, SMALLOC_PRE_RED);
assert(0);
}
if (*postred != SMALLOC_POST_RED) {
hdr, hdr->prered, SMALLOC_PRE_RED);
assert(0);
}
if (*postred != SMALLOC_POST_RED) {
- fprintf(stderr, "smalloc post redzone destroyed!\n");
-
fprintf(stderr,
" ptr=%p, postred=%x, expected %x\n",
+ log_err("smalloc post redzone destroyed!\n"
+
" ptr=%p, postred=%x, expected %x\n",
hdr, *postred, SMALLOC_POST_RED);
assert(0);
}
hdr, *postred, SMALLOC_POST_RED);
assert(0);
}
@@
-363,13
+365,17
@@
void sfree(void *ptr)
global_read_unlock();
global_read_unlock();
- assert(pool);
- sfree_pool(pool, ptr);
+ if (pool) {
+ sfree_pool(pool, ptr);
+ return;
+ }
+
+ log_err("smalloc: ptr %p not from smalloc pool\n", ptr);
}
}
-static void *__smalloc_pool(struct pool *pool,
unsigned in
t size)
+static void *__smalloc_pool(struct pool *pool,
size_
t size)
{
{
-
unsigned in
t nr_blocks;
+
size_
t nr_blocks;
unsigned int i;
unsigned int offset;
unsigned int last_idx;
unsigned int i;
unsigned int offset;
unsigned int last_idx;
@@
-422,13
+428,18
@@
fail:
return ret;
}
return ret;
}
-static void *smalloc_pool(struct pool *pool,
unsigned in
t size)
+static void *smalloc_pool(struct pool *pool,
size_
t size)
{
{
-
unsigned in
t alloc_size = size + sizeof(struct block_hdr);
+
size_
t alloc_size = size + sizeof(struct block_hdr);
void *ptr;
void *ptr;
+ /*
+ * Round to int alignment, so that the postred pointer will
+ * be naturally aligned as well.
+ */
#ifdef SMALLOC_REDZONE
alloc_size += sizeof(unsigned int);
#ifdef SMALLOC_REDZONE
alloc_size += sizeof(unsigned int);
+ alloc_size = (alloc_size + int_mask) & ~int_mask;
#endif
ptr = __smalloc_pool(pool, alloc_size);
#endif
ptr = __smalloc_pool(pool, alloc_size);
@@
-445,15
+456,19
@@
static void *smalloc_pool(struct pool *pool, unsigned int size)
return ptr;
}
return ptr;
}
-void *smalloc(
unsigned in
t size)
+void *smalloc(
size_
t size)
{
{
- unsigned int i;
+ unsigned int i, end_pool;
+
+ if (size != (unsigned int) size)
+ return NULL;
global_write_lock();
i = last_pool;
global_write_lock();
i = last_pool;
+ end_pool = nr_pools;
do {
do {
- for (; i <
nr_pools
; i++) {
+ for (; i <
end_pool
; i++) {
void *ptr = smalloc_pool(&mp[i], size);
if (ptr) {
void *ptr = smalloc_pool(&mp[i], size);
if (ptr) {
@@
-463,29
+478,29
@@
void *smalloc(unsigned int size)
}
}
if (last_pool) {
}
}
if (last_pool) {
- last_pool = 0;
+ end_pool = last_pool;
+ last_pool = i = 0;
continue;
}
continue;
}
- if (nr_pools + 1 > MAX_POOLS)
- break;
- else {
- i = nr_pools;
- if (add_pool(&mp[nr_pools], size))
- goto out;
- }
+ break;
} while (1);
} while (1);
-out:
global_write_unlock();
return NULL;
}
global_write_unlock();
return NULL;
}
+void *scalloc(size_t nmemb, size_t size)
+{
+ return smalloc(nmemb * size);
+}
+
char *smalloc_strdup(const char *str)
{
char *smalloc_strdup(const char *str)
{
- char *ptr;
+ char *ptr
= NULL
;
ptr = smalloc(strlen(str) + 1);
ptr = smalloc(strlen(str) + 1);
- strcpy(ptr, str);
+ if (ptr)
+ strcpy(ptr, str);
return ptr;
}
return ptr;
}