From 8284df1d749fa7adb334fe4f43da77bfc9c05a71 Mon Sep 17 00:00:00 2001
From: wangzhengquan <wangzhengquan85@126.com>
Date: 星期四, 24 十二月 2020 11:35:02 +0800
Subject: [PATCH] add error message method
---
src/socket/shm_socket.c | 83 +++++++++++++++++++++++++++++++++--------
1 files changed, 66 insertions(+), 17 deletions(-)
diff --git a/src/socket/shm_socket.c b/src/socket/shm_socket.c
index 9581b69..f53b520 100644
--- a/src/socket/shm_socket.c
+++ b/src/socket/shm_socket.c
@@ -2,14 +2,18 @@
#include "hashtable.h"
#include "logger_factory.h"
#include <map>
+#include "bus_error.h"
static Logger *logger = LoggerFactory::getLogger();
-void print_msg(char *head, shm_msg_t &msg) {
+static void print_msg(char *head, shm_msg_t &msg) {
// err_msg(0, "%s: key=%d, type=%d\n", head, msg.key, msg.type);
}
+
+static pthread_once_t _once_ = PTHREAD_ONCE_INIT;
+static pthread_key_t _tmp_recv_socket_key_;
static void *_server_run_msg_rev(void *_socket);
@@ -18,6 +22,9 @@
static int _shm_close_dgram_socket(shm_socket_t *socket);
static int _shm_close_stream_socket(shm_socket_t *socket, bool notifyRemote);
+
+static void _destrory_tmp_recv_socket_(void *tmp_socket);
+static void _create_tmp_recv_socket_key(void);
// 妫�鏌ey鏄惁宸茬粡琚娇鐢紝鏄繑鍥�0, 鍚﹁繑鍥�1
static inline int _shm_socket_check_key(shm_socket_t *socket) {
@@ -38,6 +45,8 @@
}
shm_socket_t *shm_open_socket(shm_socket_type_t socket_type) {
+
+ logger->debug("shm_open_socket\n");
shm_socket_t *socket = (shm_socket_t *)calloc(1, sizeof(shm_socket_t));
socket->socket_type = socket_type;
socket->key = -1;
@@ -45,12 +54,14 @@
socket->dispatch_thread = 0;
socket->status = SHM_CONN_CLOSED;
socket->mutex = SemUtil::get(IPC_PRIVATE, 1);
+
return socket;
}
int shm_close_socket(shm_socket_t *socket) {
int ret;
+
switch (socket->socket_type) {
case SHM_SOCKET_STREAM:
ret = _shm_close_stream_socket(socket, true);
@@ -61,10 +72,18 @@
default:
break;
}
- SemUtil::remove(socket->mutex);
free(socket);
+ SemUtil::remove(socket->mutex);
+ logger->debug("shm_close_socket\n");
return ret;
}
+
+// int shm_close_socket(shm_socket_t *socket) {
+
+// // _destrory_tmp_recv_socket_((shm_socket_t *)pthread_getspecific(_tmp_recv_socket_key_));
+
+// return shm_close_socket(socket);;
+// }
int shm_socket_bind(shm_socket_t *socket, int key) {
socket->key = key;
@@ -305,8 +324,9 @@
SHMQueue<shm_msg_t> *remoteQueue;
if ((remoteQueue = _attach_remote_queue(key)) == NULL) {
- logger->error( "shm_sendto failed, the other end has been closed, or has not been opened!");
- return SHM_SOCKET_ECONNFAILED;
+ bus_errno = EBUS_CLOSED;
+ logger->error("sendto key %d failed, %s", key, bus_strerror(bus_errno));
+ return EBUS_CLOSED;
}
shm_msg_t dest;
@@ -333,8 +353,16 @@
} else {
delete remoteQueue;
mm_free(dest.buf);
- logger->error(errno, "sendto key %d failed!", key);
- return -1;
+ if(errno == EAGAIN) {
+ bus_errno = EBUS_TIMEOUT;
+ logger->error("sendto key %d failed, %s", key, bus_strerror(bus_errno));
+ return EBUS_TIMEOUT;
+ } else {
+ logger->error(errno, "sendto key %d failed!", key);
+ return -1;
+ }
+
+
}
}
@@ -374,11 +402,18 @@
}
if (rv) {
- void *_buf = malloc(src.size);
- memcpy(_buf, src.buf, src.size);
- *buf = _buf;
- *size = src.size;
- *key = src.key;
+ if(buf != NULL) {
+ void *_buf = malloc(src.size);
+ memcpy(_buf, src.buf, src.size);
+ *buf = _buf;
+ }
+
+ if(size != NULL)
+ *size = src.size;
+
+ if(key != NULL)
+ *key = src.key;
+
mm_free(src.buf);
// printf("shm_recvfrom pop after\n");
return 0;
@@ -387,14 +422,21 @@
}
}
-static pthread_once_t _once_ = PTHREAD_ONCE_INIT;
-static pthread_key_t _tmp_recv_socket_key_;
/* Free thread-specific data buffer */
static void _destrory_tmp_recv_socket_(void *tmp_socket)
{
+ int rv;
+ if(tmp_socket == NULL)
+ return;
+
logger->debug("%d destroy tmp socket\n", pthread_self());
shm_close_socket((shm_socket_t *)tmp_socket);
+ rv = pthread_setspecific(_tmp_recv_socket_key_, NULL);
+ if ( rv != 0) {
+ logger->error(rv, "shm_sendandrecv : pthread_setspecific");
+ exit(1);
+ }
}
/* One-time key creation function */
@@ -405,6 +447,7 @@
/* Allocate a unique thread-specific data key and save the address
of the destructor for thread-specific data buffers */
s = pthread_key_create(&_tmp_recv_socket_key_, _destrory_tmp_recv_socket_);
+ //s = pthread_key_create(&_tmp_recv_socket_key_, NULL);
if (s != 0) {
logger->error(s, "pthread_key_create");
abort(); /* dump core and terminate */
@@ -414,7 +457,7 @@
-int shm_sendandrecv(shm_socket_t *socket, const void *send_buf,
+int shm_sendandrecv_safe(shm_socket_t *socket, const void *send_buf,
const int send_size, const int send_key, void **recv_buf,
int *recv_size, struct timespec *timeout, int flags) {
int recv_key;
@@ -444,8 +487,8 @@
logger->debug("%d create tmp socket\n", pthread_self() );
tmp_socket = shm_open_socket(SHM_SOCKET_DGRAM);
- rv = pthread_setspecific(_tmp_recv_socket_key_, tmp_socket);
- if (rv != 0) {
+ rv = pthread_setspecific(_tmp_recv_socket_key_, tmp_socket);
+ if ( rv != 0) {
logger->error(rv, "shm_sendandrecv : pthread_setspecific");
exit(1);
}
@@ -484,6 +527,12 @@
return -1;
}
+int shm_sendandrecv(shm_socket_t *socket, const void *send_buf,
+ const int send_size, const int send_key, void **recv_buf,
+ int *recv_size, struct timespec *timeout, int flags) {
+ return shm_sendandrecv_unsafe(socket, send_buf, send_size, send_key,recv_buf, recv_size, timeout, flags);
+}
+
// ============================================================================================================
/**
@@ -493,7 +542,7 @@
hashtable_t *hashtable = mm_get_hashtable();
if (hashtable_get(hashtable, key) == NULL) {
- logger->error("shm_socket._remote_queue_attach锛歝onnet at key %d failed!", key);
+ //logger->error("shm_socket._remote_queue_attach锛歝onnet at key %d failed!", key);
return NULL;
}
--
Gitblit v1.8.0