From c9c89f7b85c82d5b7d2c094e2aa1488d865d85de Mon Sep 17 00:00:00 2001
From: wangzhengquan <wangzhengquan85@126.com>
Date: 星期一, 25 一月 2021 09:42:56 +0800
Subject: [PATCH] update

---
 src/queue/shm_queue.h |  162 ++++++++++++++++++++++++++++++++---------------------
 1 files changed, 97 insertions(+), 65 deletions(-)

diff --git a/src/queue/shm_queue.h b/src/queue/shm_queue.h
index 7c7b89b..9e64992 100644
--- a/src/queue/shm_queue.h
+++ b/src/queue/shm_queue.h
@@ -1,14 +1,19 @@
+/**
+ * encapsulate lock_free_queue, populate in userspace
+ */
+
 #ifndef __SHM_QUEUE_H__
 #define __SHM_QUEUE_H__
 
 #include "hashtable.h"
-#include "lock_free_queue.h"
+ 
 #include "logger_factory.h"
 #include "sem_util.h"
 #include "shm_allocator.h"
 #include "usg_common.h"
-// default Queue size
-// #define LOCK_FREE_Q_DEFAULT_SIZE 16
+#include "array_lock_free_sem_queue.h"
+#include "lock_free_queue.h"
+#include "bus_error.h"
 
 template <typename ELEM_T> class SHMQueue {
 
@@ -17,26 +22,25 @@
 
 public:
   /// @brief constructor of the class
-  SHMQueue(int key = 0, size_t qsize = LOCK_FREE_Q_DEFAULT_SIZE);
+  SHMQueue(int key = 0, size_t qsize = 16);
 
   ~SHMQueue();
 
   void force_destroy();
 
-  inline uint32_t size();
+  uint32_t size();
 
-  inline bool full();
-  inline bool empty();
+  bool full();
+  bool empty();
 
-  inline bool push(const ELEM_T &a_data);
-  inline bool push_nowait(const ELEM_T &a_data);
-  inline bool push_timeout(const ELEM_T &a_data,
-                           const struct timespec *timeout);
-  inline bool pop(ELEM_T &a_data);
-  inline bool pop_nowait(ELEM_T &a_data);
-  inline bool pop_timeout(ELEM_T &a_data, struct timespec *timeout);
+  int push(const ELEM_T &a_data);
+  int push_nowait(const ELEM_T &a_data);
+  int push_timeout(const ELEM_T &a_data, const struct timespec *timeout);
+  int pop(ELEM_T &a_data);
+  int pop_nowait(ELEM_T &a_data);
+  int pop_timeout(ELEM_T &a_data, struct timespec *timeout);
 
-  inline ELEM_T &operator[](unsigned i);
+  ELEM_T &operator[](unsigned i);
 
  // @deprecate
   static size_t remove_queues_exclude(int keys[], size_t length);
@@ -47,6 +51,7 @@
 protected:
   /// @brief the actual queue-> methods are forwarded into the real
   ///        implementation
+
   LockFreeQueue<ELEM_T, SHM_Allocator> *queue;
 
 private:
@@ -114,96 +119,123 @@
     queue = new LockFreeQueue<ELEM_T, SHM_Allocator>(qsize);
     hashtable_put(hashtable, key, (void *)queue);
   }
-  queue->reference++;
+  // queue->reference++;
   // LoggerFactory::getLogger()->debug("SHMQueue constructor reference===%d", queue->reference.load());
 }
 
 template <typename ELEM_T> SHMQueue<ELEM_T>::~SHMQueue() {
-  if(queue == NULL) {
-     // queue宸茬粡閿�姣�
-    return;
-  }
-
-  SemUtil::dec(queue->mutex);
-  queue->reference--;
-  // LoggerFactory::getLogger()->debug("SHMQueue destructor  reference===%d",
-  if (queue->reference.load() == 0) {
-      delete queue;
-      queue = NULL;
-      hashtable_t *hashtable = mm_get_hashtable();
-      hashtable_remove(hashtable, KEY);
-      // 姝ゆ椂queue宸茬粡閿�姣侊紝鏃犻渶 SemUtil::inc(queue->mutex)
-      // printf("SHMQueue destructor delete queue\n");
-  } else {
-      SemUtil::inc(queue->mutex);
-  }
-  
-}
-
-template <typename ELEM_T> void SHMQueue<ELEM_T>::force_destroy() {
-  if(queue == NULL) {
-    // queue宸茬粡閿�姣�
-    return;
-  }
-
-  SemUtil::dec(queue->mutex);
+  LoggerFactory::getLogger()->debug("SHMQueue destroy");
   delete queue;
   queue = NULL;
   hashtable_t *hashtable = mm_get_hashtable();
   hashtable_remove(hashtable, KEY);
-  // 姝ゆ椂queue宸茬粡閿�姣侊紝鏃犻渶 SemUtil::inc(queue->mutex)
+  
 }
 
-template <typename ELEM_T> inline uint32_t SHMQueue<ELEM_T>::size() {
+template <typename ELEM_T> uint32_t SHMQueue<ELEM_T>::size() {
   return queue->size();
 }
 
-template <typename ELEM_T> inline bool SHMQueue<ELEM_T>::full() {
+template <typename ELEM_T> bool SHMQueue<ELEM_T>::full() {
   return queue->full();
 }
 
-template <typename ELEM_T> inline bool SHMQueue<ELEM_T>::empty() {
+template <typename ELEM_T> bool SHMQueue<ELEM_T>::empty() {
   return queue->empty();
 }
 
 template <typename ELEM_T>
-inline bool SHMQueue<ELEM_T>::push(const ELEM_T &a_data) {
-  return queue->push(a_data);
+int SHMQueue<ELEM_T>::push(const ELEM_T &a_data) {
+  int rv = queue->push(a_data);
+  if(rv == -1) {
+    return errno;
+  } else {
+    return 0;
+  }
 }
 
 template <typename ELEM_T>
-inline bool SHMQueue<ELEM_T>::push_nowait(const ELEM_T &a_data) {
-  return queue->push_nowait(a_data);
+int SHMQueue<ELEM_T>::push_nowait(const ELEM_T &a_data) {
+  int rv =  queue->push(a_data, NULL, BUS_NOWAIT_FLAG);
+  if(rv == -1) {
+    if (errno == EAGAIN)
+      return EAGAIN;
+    else {
+        err_msg(errno, "LockFreeQueue push_nowait");
+        return errno;
+    }
+  }
+  return 0;
 }
 
 template <typename ELEM_T>
-inline bool SHMQueue<ELEM_T>::push_timeout(const ELEM_T &a_data,
-                                           const struct timespec *timeout) {
+int SHMQueue<ELEM_T>::push_timeout(const ELEM_T &a_data, const struct timespec *timeout) {
 
-  return queue->push_timeout(a_data, timeout);
+  int rv = queue->push(a_data, timeout, BUS_TIMEOUT_FLAG);
+  if(rv == -1) {
+    if(errno == ETIMEDOUT)
+        return EBUS_TIMEOUT;
+    else {
+       LoggerFactory::getLogger()->error(errno, "LockFreeQueue push_timeout");
+       return errno;
+    }
+  }
+  return 0;
 }
 
-template <typename ELEM_T> inline bool SHMQueue<ELEM_T>::pop(ELEM_T &a_data) {
-  // printf("SHMQueue pop before\n");
+template <typename ELEM_T> 
+int SHMQueue<ELEM_T>::pop(ELEM_T &a_data) {
+  LoggerFactory::getLogger()->debug("SHMQueue pop before\n");
   int rv = queue->pop(a_data);
-  // printf("SHMQueue after before\n");
-  return rv;
+
+  LoggerFactory::getLogger()->debug("SHMQueue pop before\n");
+  if(rv == -1) {
+
+    return errno;
+  } else {
+    return 0;
+  }
 }
 
 template <typename ELEM_T>
-inline bool SHMQueue<ELEM_T>::pop_nowait(ELEM_T &a_data) {
-  return queue->pop_nowait(a_data);
+int SHMQueue<ELEM_T>::pop_nowait(ELEM_T &a_data) {
+  int rv = queue->pop(a_data, NULL, BUS_NOWAIT_FLAG);
+
+  if(rv == -1) {
+    if (errno == EAGAIN)
+      return errno;
+    else {
+        LoggerFactory::getLogger()->error(errno, " SHMQueue pop_nowait");
+        return errno;
+    }
+  }
+  return 0;
+  
 }
 
 template <typename ELEM_T>
-inline bool SHMQueue<ELEM_T>::pop_timeout(ELEM_T &a_data,
-                                          struct timespec *timeout) {
-  return queue->pop_timeout(a_data, timeout);
+int SHMQueue<ELEM_T>::pop_timeout(ELEM_T &a_data, struct timespec *timeout) {
+
+  int rv;
+  rv = queue->pop(a_data, timeout, BUS_TIMEOUT_FLAG);
+  if(rv == -1) {
+    if (errno == ETIMEDOUT) {
+      return EBUS_TIMEOUT;
+    } else {
+      LoggerFactory::getLogger()->error(errno, " SHMQueue pop_timeout");
+      return errno;
+    }
+  }
+  return 0;
+  
 }
 
 template <typename ELEM_T>
-inline ELEM_T &SHMQueue<ELEM_T>::operator[](unsigned i) {
+ELEM_T &SHMQueue<ELEM_T>::operator[](unsigned i) {
   return queue->operator[](i);
 }
 
+
+
+
 #endif

--
Gitblit v1.8.0