mirror of
https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git
synced 2025-11-03 09:01:50 -05:00
make pa_mempool_stat thread-safe/lock-free
git-svn-id: file:///home/lennart/svn/public/pulseaudio/trunk@1343 fefdeb5f-60dc-0310-8127-8f9354f1896f
This commit is contained in:
parent
327e0cd8e1
commit
5264d235d2
5 changed files with 74 additions and 69 deletions
|
|
@ -112,39 +112,40 @@ static void stat_add(pa_memblock*b) {
|
|||
assert(b);
|
||||
assert(b->pool);
|
||||
|
||||
b->pool->stat.n_allocated ++;
|
||||
b->pool->stat.n_accumulated ++;
|
||||
b->pool->stat.allocated_size += b->length;
|
||||
b->pool->stat.accumulated_size += b->length;
|
||||
AO_fetch_and_add1_release_write(&b->pool->stat.n_allocated);
|
||||
AO_fetch_and_add_release_write(&b->pool->stat.allocated_size, (AO_t) b->length);
|
||||
|
||||
AO_fetch_and_add1_release_write(&b->pool->stat.n_accumulated);
|
||||
AO_fetch_and_add_release_write(&b->pool->stat.accumulated_size, (AO_t) b->length);
|
||||
|
||||
if (b->type == PA_MEMBLOCK_IMPORTED) {
|
||||
b->pool->stat.n_imported++;
|
||||
b->pool->stat.imported_size += b->length;
|
||||
AO_fetch_and_add1_release_write(&b->pool->stat.n_imported);
|
||||
AO_fetch_and_add_release_write(&b->pool->stat.imported_size, (AO_t) b->length);
|
||||
}
|
||||
|
||||
b->pool->stat.n_allocated_by_type[b->type]++;
|
||||
b->pool->stat.n_accumulated_by_type[b->type]++;
|
||||
AO_fetch_and_add1_release_write(&b->pool->stat.n_allocated_by_type[b->type]);
|
||||
AO_fetch_and_add1_release_write(&b->pool->stat.n_accumulated_by_type[b->type]);
|
||||
}
|
||||
|
||||
static void stat_remove(pa_memblock *b) {
|
||||
assert(b);
|
||||
assert(b->pool);
|
||||
|
||||
assert(b->pool->stat.n_allocated > 0);
|
||||
assert(b->pool->stat.allocated_size >= b->length);
|
||||
assert(AO_load_acquire_read(&b->pool->stat.n_allocated) > 0);
|
||||
assert(AO_load_acquire_read(&b->pool->stat.allocated_size) >= (AO_t) b->length);
|
||||
|
||||
b->pool->stat.n_allocated --;
|
||||
b->pool->stat.allocated_size -= b->length;
|
||||
AO_fetch_and_sub1_release_write(&b->pool->stat.n_allocated);
|
||||
AO_fetch_and_add_release_write(&b->pool->stat.allocated_size, (AO_t) (-b->length));
|
||||
|
||||
if (b->type == PA_MEMBLOCK_IMPORTED) {
|
||||
assert(b->pool->stat.n_imported > 0);
|
||||
assert(b->pool->stat.imported_size >= b->length);
|
||||
assert(AO_load_acquire_read(&b->pool->stat.n_imported) > 0);
|
||||
assert(AO_load_acquire_read(&b->pool->stat.imported_size) >= (AO_t) b->length);
|
||||
|
||||
b->pool->stat.n_imported --;
|
||||
b->pool->stat.imported_size -= b->length;
|
||||
AO_fetch_and_sub1_release_write(&b->pool->stat.n_imported);
|
||||
AO_fetch_and_add_release_write(&b->pool->stat.imported_size, (AO_t) (-b->length));
|
||||
}
|
||||
|
||||
b->pool->stat.n_allocated_by_type[b->type]--;
|
||||
AO_fetch_and_sub1_release_write(&b->pool->stat.n_allocated_by_type[b->type]);
|
||||
}
|
||||
|
||||
static pa_memblock *memblock_new_appended(pa_mempool *p, size_t length);
|
||||
|
|
@ -190,7 +191,7 @@ static struct mempool_slot* mempool_allocate_slot(pa_mempool *p) {
|
|||
slot = (struct mempool_slot*) ((uint8_t*) p->memory.ptr + (p->block_size * p->n_init++));
|
||||
else {
|
||||
pa_log_debug("Pool full");
|
||||
p->stat.n_pool_full++;
|
||||
AO_fetch_and_add1_release_write(&p->stat.n_pool_full);
|
||||
return NULL;
|
||||
}
|
||||
|
||||
|
|
@ -247,7 +248,7 @@ pa_memblock *pa_memblock_new_pool(pa_mempool *p, size_t length) {
|
|||
b->data = mempool_slot_data(slot);
|
||||
} else {
|
||||
pa_log_debug("Memory block too large for pool: %u > %u", length, p->block_size - sizeof(struct mempool_slot));
|
||||
p->stat.n_too_large_for_pool++;
|
||||
AO_fetch_and_add1_release_write(&p->stat.n_too_large_for_pool);
|
||||
return NULL;
|
||||
}
|
||||
|
||||
|
|
@ -371,7 +372,7 @@ void pa_memblock_unref(pa_memblock*b) {
|
|||
static void memblock_make_local(pa_memblock *b) {
|
||||
assert(b);
|
||||
|
||||
b->pool->stat.n_allocated_by_type[b->type]--;
|
||||
AO_fetch_and_sub1_release_write(&b->pool->stat.n_allocated_by_type[b->type]);
|
||||
|
||||
if (b->length <= b->pool->block_size - sizeof(struct mempool_slot)) {
|
||||
struct mempool_slot *slot;
|
||||
|
|
@ -397,8 +398,8 @@ static void memblock_make_local(pa_memblock *b) {
|
|||
b->data = pa_xmemdup(b->data, b->length);
|
||||
|
||||
finish:
|
||||
b->pool->stat.n_allocated_by_type[b->type]++;
|
||||
b->pool->stat.n_accumulated_by_type[b->type]++;
|
||||
AO_fetch_and_add1_release_write(&b->pool->stat.n_allocated_by_type[b->type]);
|
||||
AO_fetch_and_add1_release_write(&b->pool->stat.n_accumulated_by_type[b->type]);
|
||||
}
|
||||
|
||||
void pa_memblock_unref_fixed(pa_memblock *b) {
|
||||
|
|
@ -418,10 +419,10 @@ static void memblock_replace_import(pa_memblock *b) {
|
|||
assert(b);
|
||||
assert(b->type == PA_MEMBLOCK_IMPORTED);
|
||||
|
||||
assert(b->pool->stat.n_imported > 0);
|
||||
assert(b->pool->stat.imported_size >= b->length);
|
||||
b->pool->stat.n_imported --;
|
||||
b->pool->stat.imported_size -= b->length;
|
||||
assert(AO_load_acquire_read(&b->pool->stat.n_imported) > 0);
|
||||
assert(AO_load_acquire_read(&b->pool->stat.imported_size) >= (AO_t) b->length);
|
||||
AO_fetch_and_sub1_release_write(&b->pool->stat.n_imported);
|
||||
AO_fetch_and_add_release_write(&b->pool->stat.imported_size, (AO_t) - b->length);
|
||||
|
||||
seg = b->per_type.imported.segment;
|
||||
assert(seg);
|
||||
|
|
@ -486,7 +487,7 @@ void pa_mempool_free(pa_mempool *p) {
|
|||
while (p->exports)
|
||||
pa_memexport_free(p->exports);
|
||||
|
||||
if (p->stat.n_allocated > 0)
|
||||
if (AO_load_acquire_read(&p->stat.n_allocated) > 0)
|
||||
pa_log_warn("WARNING! Memory pool destroyed but not all memory blocks freed!");
|
||||
|
||||
pa_shm_free(&p->memory);
|
||||
|
|
@ -685,11 +686,11 @@ int pa_memexport_process_release(pa_memexport *e, uint32_t id) {
|
|||
|
||||
/* pa_log("Processing release for %u", id); */
|
||||
|
||||
assert(e->pool->stat.n_exported > 0);
|
||||
assert(e->pool->stat.exported_size >= e->slots[id].block->length);
|
||||
assert(AO_load_acquire_read(&e->pool->stat.n_exported) > 0);
|
||||
assert(AO_load_acquire_read(&e->pool->stat.exported_size) >= (AO_t) e->slots[id].block->length);
|
||||
|
||||
e->pool->stat.n_exported --;
|
||||
e->pool->stat.exported_size -= e->slots[id].block->length;
|
||||
AO_fetch_and_sub1_release_write(&e->pool->stat.n_exported);
|
||||
AO_fetch_and_add_release_write(&e->pool->stat.exported_size, (AO_t) -e->slots[id].block->length);
|
||||
|
||||
pa_memblock_unref(e->slots[id].block);
|
||||
e->slots[id].block = NULL;
|
||||
|
|
@ -786,8 +787,8 @@ int pa_memexport_put(pa_memexport *e, pa_memblock *b, uint32_t *block_id, uint32
|
|||
*offset = (uint8_t*) b->data - (uint8_t*) memory->ptr;
|
||||
*size = b->length;
|
||||
|
||||
e->pool->stat.n_exported ++;
|
||||
e->pool->stat.exported_size += b->length;
|
||||
AO_fetch_and_add1_release_write(&e->pool->stat.n_exported);
|
||||
AO_fetch_and_add_release_write(&e->pool->stat.exported_size, (AO_t) b->length);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue