Revision
16903
Author
andrew
Date
2010-03-04 14:17:32 -0600 (Thu, 04 Mar 2010)

Log Message

Tweak some mutexes

Modified Paths

Diff

Modified: freeswitch/trunk/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.c (16902 => 16903)


--- freeswitch/trunk/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.c	2010-03-04 19:09:13 UTC (rev 16902)
+++ freeswitch/trunk/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.c	2010-03-04 20:17:32 UTC (rev 16903)
@@ -245,7 +245,6 @@
 static void close_socket(int *sock)
 #endif
 {
-	switch_mutex_lock(listen_list.sock_mutex);
 	if (*sock) {
 #ifdef WIN32
 		shutdown(*sock, SD_BOTH);
@@ -256,7 +255,6 @@
 #endif
 		sock = NULL;
 	}
-	switch_mutex_unlock(listen_list.sock_mutex);
 }
 
 
@@ -453,7 +451,7 @@
 	if (!p->reply) {
 		p->state = reply_timeout;
 		switch_mutex_unlock(globals.fetch_reply_mutex);
-		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_WARNING, "Timed out after %d milliseconds when waiting for XML fetch response\n", (int) (switch_micro_time_now() - now) / 1000);
+		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_WARNING, "Timed out after %d milliseconds when waiting for XML fetch response for %s\n", (int) (switch_micro_time_now() - now) / 1000, uuid_str);
 		goto cleanup;
 	}
 
@@ -476,7 +474,7 @@
 
 	ei_decode_string_or_binary(rep->buff, &rep->index, size, xmlstr);
 
-	switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, "got data %s after %d milliseconds from %s!\n", xmlstr, (int) (switch_micro_time_now() - now) / 1000, p->winner);
+	switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, "got data %s after %d milliseconds from %s for %s!\n", xmlstr, (int) (switch_micro_time_now() - now) / 1000, p->winner, uuid_str);
 
 	if (zstr(xmlstr)) {
 		switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "No Result\n");
@@ -546,11 +544,11 @@
 	ei_x_encode_tuple_header(&lbuf, 2);
 	ei_x_encode_atom(&lbuf, "call");
 	ei_encode_switch_event(&lbuf, call_event);
-	switch_mutex_lock(listener->sock_mutex);
 	switch_log_printf(SWITCH_CHANNEL_UUID_LOG(session_element->uuid_str), SWITCH_LOG_DEBUG, "Sending initial call event for %s\n",
 					  session_element->uuid_str);
-	result = ei_sendto(listener->ec, listener->sockfd, &session_element->process, &lbuf);
 
+	switch_mutex_lock(listener->sock_mutex);
+	result = ei_sendto(listener->ec, listener->sockfd, &session_element->process, &lbuf);
 	switch_mutex_unlock(listener->sock_mutex);
 
 	if (result) {
@@ -1666,8 +1664,6 @@
 		return SWITCH_STATUS_TERM;
 	}
 
-	switch_mutex_init(&listen_list.sock_mutex, SWITCH_MUTEX_NESTED, pool);
-
 	/* zero out the struct before we use it */
 	memset(&server_addr, 0, sizeof(server_addr));
 

Modified: freeswitch/trunk/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.h (16902 => 16903)


--- freeswitch/trunk/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.h	2010-03-04 19:09:13 UTC (rev 16902)
+++ freeswitch/trunk/src/mod/event_handlers/mod_erlang_event/mod_erlang_event.h	2010-03-04 20:17:32 UTC (rev 16903)
@@ -166,7 +166,6 @@
 #else
 	int sockfd;
 #endif
-	switch_mutex_t *sock_mutex;
 	listener_t *listeners;
 	uint8_t ready;
 };