From cb85aa8a8d02a3d6dc16e3f32e78da9e70f9c7f5 Mon Sep 17 00:00:00 2001
From: wangzhengquan <wangzhengquan85@126.com>
Date: 星期二, 02 二月 2021 17:49:21 +0800
Subject: [PATCH] update

---
 src/socket/shm_socket.cpp |  102 +++++++++++++++++++++++++++-----------------------
 1 files changed, 55 insertions(+), 47 deletions(-)

diff --git a/src/socket/shm_socket.cpp b/src/socket/shm_socket.cpp
index 31598d0..9544827 100644
--- a/src/socket/shm_socket.cpp
+++ b/src/socket/shm_socket.cpp
@@ -9,7 +9,7 @@
 
 
 
-static void print_msg(char *head, shm_msg_t &msg) {
+static void print_msg(char *head, shm_packet_t &msg) {
   // err_msg(0, "%s: key=%d, type=%d\n", head, msg.key, msg.type);
 }
 
@@ -20,21 +20,21 @@
 static void _create_socket_key_perthread(void);
 
 //  妫�鏌ey鏄惁宸茬粡琚娇鐢紝 鏈浣跨敤鍒欑粦瀹歬ey
-static LockFreeQueue<shm_msg_t> * shm_socket_bind_queue(int key, bool force) {
+static LockFreeQueue<shm_packet_t> * shm_socket_bind_queue(int key, bool force) {
   hashtable_t *hashtable = mm_get_hashtable();
-  LockFreeQueue<shm_msg_t> *queue;
+  LockFreeQueue<shm_packet_t> *queue;
   hashtable_lock(hashtable);
   void *tmp_ptr = hashtable_get(hashtable, key);
 
 
   if (tmp_ptr == NULL || tmp_ptr == (void *)1  ) {
-    queue = new LockFreeQueue<shm_msg_t>(16);
+    queue = new LockFreeQueue<shm_packet_t>(16);
     hashtable_put(hashtable, key, (void *)queue);
     hashtable_unlock(hashtable);
     return queue;
   } else if(force) {
     hashtable_unlock(hashtable);
-    return (LockFreeQueue<shm_msg_t> *) queue;
+    return (LockFreeQueue<shm_packet_t> *) queue;
   }
   
   hashtable_unlock(hashtable);
@@ -44,8 +44,8 @@
 /**
  * 缁戝畾key鍒伴槦鍒楋紝浣嗘槸骞朵笉浼氬垱寤洪槦鍒椼��
  */
-static LockFreeQueue<shm_msg_t> * shm_socket_attach_queue(int key) {
-  LockFreeQueue<shm_msg_t> * queue;
+static LockFreeQueue<shm_packet_t> * shm_socket_attach_queue(int key) {
+  LockFreeQueue<shm_packet_t> * queue;
   hashtable_t *hashtable = mm_get_hashtable();
   void *tmp_ptr = hashtable_get(hashtable, key);
   if (tmp_ptr == NULL || tmp_ptr == (void *)1) {
@@ -53,7 +53,7 @@
     return NULL;
   }
 
-  queue = ( LockFreeQueue<shm_msg_t> *)tmp_ptr;
+  queue = ( LockFreeQueue<shm_packet_t> *)tmp_ptr;
   // hashtable_unlock(hashtable);
   return queue;
 }
@@ -61,11 +61,11 @@
 
 size_t shm_socket_remove_keys(int keys[], size_t length) {
   hashtable_t *hashtable = mm_get_hashtable();
-  LockFreeQueue<shm_msg_t> *mqueue;
+  LockFreeQueue<shm_packet_t> *mqueue;
   size_t count = 0;
   for(int i = 0; i< length; i++) {
     // 閿�姣佸叡浜唴瀛樼殑queue
-    mqueue = (LockFreeQueue<shm_msg_t> *)hashtable_get(hashtable, keys[i]);
+    mqueue = (LockFreeQueue<shm_packet_t> *)hashtable_get(hashtable, keys[i]);
     delete mqueue;
     hashtable_remove(hashtable, keys[i]);
     count++;
@@ -174,15 +174,14 @@
     return EBUS_SENDTO_SELF;
   }
 
-  LockFreeQueue<shm_msg_t> *remoteQueue;
+  LockFreeQueue<shm_packet_t> *remoteQueue;
   if ((remoteQueue = shm_socket_attach_queue(key)) == NULL) {
     bus_errno = EBUS_CLOSED;
     logger->error("sendto key %d failed, %s", key, bus_strerror(bus_errno));
     return EBUS_CLOSED;
   }
 
-  shm_msg_t dest;
-  dest.type = SHM_COMMON_MSG;
+  shm_packet_t dest;
   dest.key = sockt->key;
   dest.size = size;
   dest.buf = mm_malloc(size);
@@ -228,7 +227,7 @@
   if ((rv = pthread_mutex_unlock(&(sockt->mutex))) != 0)
     err_exit(rv, "shm_recvfrom : pthread_mutex_unlock");
 
-  shm_msg_t src;
+  shm_packet_t src;
  
   rv = sockt->queue->pop(src, timeout, flag);
 
@@ -287,7 +286,6 @@
   //s = pthread_key_create(&_perthread_socket_key_, NULL);
   if (s != 0) {
      logger->error(s, "pthread_key_create");
-     abort(); /* dump core and terminate */ 
      exit(1);
   }
 }
@@ -299,11 +297,13 @@
                     int *recv_size,  const struct timespec *timeout,  int flags) {
   int recv_key;
   int rv;
+  int tryn = 0;
 
   // 鐢╰hread local 淇濊瘉姣忎釜绾跨▼鐢ㄤ竴涓嫭鍗犵殑socket鎺ュ彈瀵规柟杩斿洖鐨勪俊鎭�
   shm_socket_t *tmp_socket;
  
-
+  /* If first call from this thread, allocate buffer for thread, and save its location */
+  // logger->debug("%d create tmp socket\n", pthread_self() );
   rv = pthread_once(&_once_, _create_socket_key_perthread);
   if (rv != 0) {
     logger->error(rv, "shm_sendandrecv pthread_once");
@@ -325,26 +325,31 @@
   }
 
   if ((rv = shm_sendto(tmp_socket, send_buf, send_size, send_key, timeout, flags)) == 0) {
-    rv = shm_recvfrom(tmp_socket, recv_buf, recv_size, &recv_key, timeout, flags);
-    if(rv != 0) {
-      logger->error("_shm_sendandrecv_thread_local : %s\n", bus_strerror(rv));
-    }
-    else if(rv == 0 ) {
-      logger->debug("======%d use tmp_socket %d, send to  %d, receive from  %d\n", shm_socket_get_key(sockt), shm_socket_get_key(tmp_socket), send_key, recv_key);
-    
-      if(recv_key == shm_socket_get_key(sockt)) {
-        logger->debug("=====鏀跺埌浜嗚嚜宸卞彂缁欒嚜宸辩殑娑堟伅\n");
+
+    while(tryn < 3) {
+      tryn++;
+      rv = shm_recvfrom(tmp_socket, recv_buf, recv_size, &recv_key, timeout, flags);
+      if(rv != 0) {
+        logger->error("_shm_sendandrecv_thread_local : %s\n", bus_strerror(rv));
+        return rv;
       }
-      assert( send_key == recv_key);
+     
+       // 瓒呮椂瀵艰嚧鎺ュ彂閫佸璞★紝涓庤繑鍥炲璞′笉瀵瑰簲鐨勬儏鍐�
       if(send_key != recv_key) {
-        logger->error( "_shm_sendandrecv_alloc_new: send key expect to equal to recv key! send key =%d , recv key=%d", send_key, recv_key);
-        exit(1);
+        logger->debug("======%d use tmp_socket %d, send to  %d, receive from  %d\n", shm_socket_get_key(sockt), shm_socket_get_key(tmp_socket), send_key, recv_key);
+        // logger->error( "_shm_sendandrecv_alloc_new: send key expect to equal to recv key! send key =%d , recv key=%d", send_key, recv_key);
+        // exit(1);
+        continue;
+        // return EBUS_RECVFROM_WRONG_END;
       }
+
+      return 0;
     }
-    return rv;
-  } else {
-    return rv;
-  }
+    
+    return EBUS_RECVFROM_WRONG_END;
+  }  
+
+  return rv;
 }
 
 int _shm_sendandrecv_alloc_new(shm_socket_t *sockt, const void *send_buf,
@@ -353,31 +358,34 @@
   int recv_key;
   int rv;
 
-  // 鐢╰hread local 淇濊瘉姣忎釜绾跨▼鐢ㄤ竴涓嫭鍗犵殑socket鎺ュ彈瀵规柟杩斿洖鐨勪俊鎭�
+  int tryn = 0;
   shm_socket_t *tmp_socket;
  
-  /* If first call from this thread, allocate buffer for thread, and save its location */
-  // logger->debug("%d create tmp socket\n", pthread_self() );
+ 
   tmp_socket = shm_open_socket(SHM_SOCKET_DGRAM);
 
   if ((rv = shm_sendto(tmp_socket, send_buf, send_size, send_key, timeout, flags)) == 0) {
     rv = shm_recvfrom(tmp_socket, recv_buf, recv_size, &recv_key, timeout, flags);
    
-    if(rv != 0) {
-      printf("_shm_sendandrecv_alloc_new : %s\n", bus_strerror(rv));
-    }
-    else if(rv == 0 ) {
-      printf("======%d use tmp_socket %d, send to  %d, receive from  %d\n", shm_socket_get_key(sockt), shm_socket_get_key(tmp_socket), send_key, recv_key);
-    
-      if(recv_key == shm_socket_get_key(sockt)) {
-        printf("=====鏀跺埌浜嗚嚜宸卞彂缁欒嚜宸辩殑娑堟伅\n");
-      }
-      assert( send_key == recv_key);
-      if(send_key != recv_key) {
-         err_exit(0, "_shm_sendandrecv_alloc_new: send key expect to equal to recv key! send key =%d , recv key=%d", send_key, recv_key);
+    while(tryn < 3) {
+      tryn++;
+      rv = shm_recvfrom(tmp_socket, recv_buf, recv_size, &recv_key, timeout, flags);
+      if(rv != 0) {
+        logger->error("_shm_sendandrecv_thread_local : %s\n", bus_strerror(rv));
+        return rv;
       }
      
+      // 瓒呮椂瀵艰嚧鎺ュ彂閫佸璞★紝涓庤繑鍥炲璞′笉瀵瑰簲鐨勬儏鍐�
+      if(send_key != recv_key) {
+        // logger->debug("======%d use tmp_socket %d, send to  %d, receive from  %d\n", shm_socket_get_key(sockt), shm_socket_get_key(tmp_socket), send_key, recv_key);
+        // logger->error( "_shm_sendandrecv_alloc_new: send key expect to equal to recv key! send key =%d , recv key=%d", send_key, recv_key);
+        
+        continue;
+      }
+      return 0;
     }
+    
+    return EBUS_RECVFROM_WRONG_END;
   } 
    
   shm_close_socket(tmp_socket);  

--
Gitblit v1.8.0