[asterisk-commits] kpfleming: trunk r49608 - /trunk/main/threadstorage.c

[asterisk-commits at lists.digium.com]

Author: kpfleming
Date: Thu Jan  4 18:12:10 2007
New Revision: 49608

URL: http://svn.digium.com/view/asterisk?view=rev&rev=49608
Log:
use a rwlock-list for the list of TLS objects

Modified:
    trunk/main/threadstorage.c

Modified: trunk/main/threadstorage.c
URL: http://svn.digium.com/view/asterisk/trunk/main/threadstorage.c?view=diff&rev=49608&r1=49607&r2=49608
==============================================================================
--- trunk/main/threadstorage.c (original)
+++ trunk/main/threadstorage.c Thu Jan  4 18:12:10 2007
@@ -50,7 +50,7 @@
 	AST_LIST_ENTRY(tls_object) entry;
 };
 
-static AST_LIST_HEAD_STATIC(tls_objects, tls_object);
+static AST_RWLIST_HEAD_STATIC(tls_objects, tls_object);
 
 void __ast_threadstorage_object_add(void *key, size_t len, const char *file, const char *function, unsigned int line)
 {
@@ -66,16 +66,16 @@
 	to->line = line;
 	to->thread = pthread_self();
 
-	AST_LIST_LOCK(&tls_objects);
+	AST_RWLIST_WRLOCK(&tls_objects);
 	AST_LIST_INSERT_TAIL(&tls_objects, to, entry);
-	AST_LIST_UNLOCK(&tls_objects);
+	AST_RWLIST_UNLOCK(&tls_objects);
 }
 
 void __ast_threadstorage_object_remove(void *key)
 {
 	struct tls_object *to;
 
-	AST_LIST_LOCK(&tls_objects);
+	AST_RWLIST_WRLOCK(&tls_objects);
 	AST_LIST_TRAVERSE_SAFE_BEGIN(&tls_objects, to, entry) {
 		if (to->key == key) {
 			AST_LIST_REMOVE_CURRENT(&tls_objects, entry);
@@ -83,7 +83,7 @@
 		}
 	}
 	AST_LIST_TRAVERSE_SAFE_END;
-	AST_LIST_UNLOCK(&tls_objects);
+	AST_RWLIST_UNLOCK(&tls_objects);
 	if (to)
 		free(to);
 }
@@ -92,7 +92,7 @@
 {
 	struct tls_object *to;
 
-	AST_LIST_LOCK(&tls_objects);
+	AST_RWLIST_WRLOCK(&tls_objects);
 	AST_LIST_TRAVERSE_SAFE_BEGIN(&tls_objects, to, entry) {
 		if (to->key == key_old) {
 			to->key = key_new;
@@ -101,7 +101,7 @@
 		}
 	}
 	AST_LIST_TRAVERSE_SAFE_END;
-	AST_LIST_UNLOCK(&tls_objects);
+	AST_RWLIST_UNLOCK(&tls_objects);
 }
 
 static int handle_show_allocations(int fd, int argc, char *argv[])
@@ -114,7 +114,7 @@
 	if (argc > 3)
 		fn = argv[3];
 
-	AST_LIST_LOCK(&tls_objects);
+	AST_RWLIST_RDLOCK(&tls_objects);
 
 	AST_LIST_TRAVERSE(&tls_objects, to, entry) {
 		if (fn && strcasecmp(to->file, fn))
@@ -126,7 +126,7 @@
 		count++;
 	}
 
-	AST_LIST_UNLOCK(&tls_objects);
+	AST_RWLIST_UNLOCK(&tls_objects);
 
 	ast_cli(fd, "%10d bytes allocated in %d allocation%s\n", (int) len, count, count > 1 ? "s" : "");
 	
@@ -150,7 +150,7 @@
 	if (argc > 3)
 		fn = argv[3];
 
-	AST_LIST_LOCK(&tls_objects);
+	AST_RWLIST_RDLOCK(&tls_objects);
 
 	AST_LIST_TRAVERSE(&tls_objects, to, entry) {
 		if (fn && strcasecmp(to->file, fn))
@@ -172,7 +172,7 @@
 		file->count++;
 	}
 
-	AST_LIST_UNLOCK(&tls_objects);
+	AST_RWLIST_UNLOCK(&tls_objects);
 	
 	AST_LIST_TRAVERSE(&file_summary, file, entry) {
 		len += file->len;