test lock code.

lichao

2021-05-10 77a6c3512a44dfe6540dde71946e6484fe4f173f

 src/socket.h

@@ -22,7 +22,7 @@
#include "bh_util.h"
#include "defs.h"
#include "sendq.h"
#include "shm_queue.h"
#include "shm_msg_queue.h"
#include <atomic>
#include <boost/noncopyable.hpp>
#include <condition_variable>
@@ -33,70 +33,64 @@
#include <vector>

using namespace bhome_msg;

class ShmSocket : private boost::noncopyable
{
   bool SendImpl(const void *valid_remote, MsgI const &imsg, SendQ::OnMsgEvent onExpire = SendQ::OnMsgEvent())
   {
      // if (!mq().TrySend(*(MQId *) valid_remote, imsg)) {
      send_buffer_.Append(*static_cast<const MQId *>(valid_remote), imsg, onExpire);
      // }
      return true;
   }

protected:
   typedef bhome_shm::ShmMsgQueue Queue;
   typedef ShmMsgQueue Queue;

public:
   typedef ShmMsgQueue::MQId MQId;
   typedef bhome_shm::SharedMemory Shm;
   typedef std::function<void(ShmSocket &sock, MsgI &imsg)> RawRecvCB;
   typedef std::function<void(ShmSocket &sock, MsgI &imsg, BHMsgHead &head)> RecvCB;
   typedef std::function<bool(ShmSocket &sock, MsgI &imsg, BHMsgHead &head)> PartialRecvCB;
   typedef std::function<void(ShmSocket &sock)> IdleCB;

   ShmSocket(Shm &shm, const MQId &id, const int len);
   ShmSocket(Shm &shm, const MQId id, const int len);
   ShmSocket(Shm &shm, const bool create_or_else_find, const MQId id, const int len);
   ShmSocket(Shm &shm, const int len = 12);
   ~ShmSocket();
   static bool Remove(SharedMemory &shm, const MQId &id) { return Queue::Remove(shm, id); }
   const MQId &id() const { return mq().Id(); }
   static bool Remove(SharedMemory &shm, const MQId id) { return Queue::Remove(shm, id); }
   bool Remove() { return Remove(shm(), id()); }
   MQId id() const { return mq().Id(); }
   // start recv.
   bool Start(const RawRecvCB &onData, const IdleCB &onIdle, int nworker = 1);
   bool Start(int nworker = 1, const RecvCB &onData = RecvCB(), const IdleCB &onIdle = IdleCB());
   bool Start(const RecvCB &onData, const IdleCB &onIdle, int nworker = 1) { return Start(nworker, onData, onIdle); }
   bool Start(const RecvCB &onData, int nworker = 1) { return Start(nworker, onData); }
   bool Stop();
   size_t Pending() const { return mq().Pending(); }

   template <class Body>
   bool Send(const void *valid_remote, const BHMsgHead &head, const Body &body, const RecvCB &cb = RecvCB())
   bool Send(const MQId remote, BHMsgHead &head, Body &body, RecvCB &&cb = RecvCB())
   {
      MsgI msg;
      if (msg.Make(shm(), head, body)) {
         DEFER1(if (msg.IsCounted()) { msg.Release(shm()); });
         std::string msg_id(head.msg_id());
      try {
         if (!cb) {
            return SendImpl(valid_remote, msg);
            return SendImpl(remote, MsgI::Serialize(head, body));
         } else {
            per_msg_cbs_->Add(msg_id, cb);
            auto onExpireRemoveCB = [this, msg_id](MsgI const &msg) {
            std::string msg_id(head.msg_id());
            per_msg_cbs_->Store(msg_id, std::move(cb));
            auto onExpireRemoveCB = [this, msg_id](SendQ::Data const &msg) {
               RecvCB cb_no_use;
               per_msg_cbs_->Find(msg_id, cb_no_use);
               per_msg_cbs_->Pick(msg_id, cb_no_use);
            };
            return SendImpl(valid_remote, msg, onExpireRemoveCB);
            return SendImpl(remote, MsgI::Serialize(head, body), onExpireRemoveCB);
         }
      } else {
         SetLastError(ENOMEM, "Out of mem");
      } catch (...) {
         SetLastError(eError, "Send internal error.");
         return false;
      }
      return false;
   }

   bool Send(const void *valid_remote, const MsgI &imsg)
   bool Send(const MQId remote, const MsgI &imsg)
   {
      return SendImpl(valid_remote, imsg);
      return SendImpl(remote, imsg);
   }

   bool SyncRecv(MsgI &msg, bhome::msg::BHMsgHead &head, const int timeout_ms);
   bool SyncRecv(MsgI &msg, bhome_msg::BHMsgHead &head, const int timeout_ms);

   template <class Body>
   bool SendAndRecv(const void *remote, const BHMsgHead &head, const Body &body, MsgI &reply, BHMsgHead &reply_head, const int timeout_ms)
   bool SendAndRecv(const MQId remote, BHMsgHead &head, Body &body, MsgI &reply, BHMsgHead &reply_head, const int timeout_ms)
   {
      struct State {
         std::mutex mutex;
@@ -119,14 +113,15 @@
         };

         std::unique_lock<std::mutex> lk(st->mutex);
         bool sendok = Send(remote, head, body, OnRecv);
         bool sendok = Send(remote, head, body, std::move(OnRecv));
         if (!sendok) {
            printf("send timeout\n");
            LOG_DEBUG() << "send timeout";
         }
         if (sendok && st->cv.wait_until(lk, endtime) == std::cv_status::no_timeout) {
            return true;
         } else {
            st->canceled = true;
            SetLastError(ETIMEDOUT, "timeout");
            return false;
         }
      } catch (...) {
@@ -145,19 +140,28 @@
   bool StopNoLock();
   bool RunningNoLock() { return !workers_.empty(); }

   template <class... Rest>
   bool SendImpl(const MQId remote, Rest &&...rest)
   {
      // TODO send alloc request, and pack later, higher bit means alloc?
      send_buffer_.Append(remote, std::forward<decltype(rest)>(rest)...);
      return true;
   }

   std::vector<std::thread> workers_;
   std::mutex mutex_;
   std::atomic<bool> run_;

   Queue mq_;
   class AsyncCBs
   template <class Key>
   class CallbackRecords
   {
      std::unordered_map<std::string, RecvCB> store_;
      std::unordered_map<Key, RecvCB> store_;

   public:
      bool empty() const { return store_.empty(); }
      bool Add(const std::string &id, const RecvCB &cb) { return store_.emplace(id, cb).second; }
      bool Find(const std::string &id, RecvCB &cb)
      bool Store(const Key &id, RecvCB &&cb) { return store_.emplace(id, std::move(cb)).second; }
      bool Pick(const Key &id, RecvCB &cb)
      {
         auto pos = store_.find(id);
         if (pos != store_.end()) {
@@ -170,9 +174,9 @@
      }
   };

   Synced<AsyncCBs> per_msg_cbs_;
   Synced<CallbackRecords<std::string>> per_msg_cbs_;

   SendQ send_buffer_;
   // Synced<SendQ> send_buffer_;
};

#endif // end of include guard: SOCKET_GWTJHBPO