From 77a6c3512a44dfe6540dde71946e6484fe4f173f Mon Sep 17 00:00:00 2001
From: lichao <lichao@aiotlink.com>
Date: 星期一, 10 五月 2021 16:05:28 +0800
Subject: [PATCH] test lock code.
---
src/socket.h | 57 +++++++++++++++++++++++++++++++--------------------------
1 files changed, 31 insertions(+), 26 deletions(-)
diff --git a/src/socket.h b/src/socket.h
index 1ba10cb..981677f 100644
--- a/src/socket.h
+++ b/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,44 +33,40 @@
#include <vector>
using namespace bhome_msg;
-
class ShmSocket : private boost::noncopyable
{
- template <class... T>
- bool SendImpl(const void *valid_remote, T &&...rest)
- {
- send_buffer_.Append(*static_cast<const MQId *>(valid_remote), std::forward<decltype(rest)>(rest)...);
- 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); }
+ static bool Remove(SharedMemory &shm, const MQId id) { return Queue::Remove(shm, id); }
bool Remove() { return Remove(shm(), id()); }
- const MQId &id() const { return mq().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, BHMsgHead &head, Body &body, RecvCB &&cb = RecvCB())
+ bool Send(const MQId remote, BHMsgHead &head, Body &body, RecvCB &&cb = RecvCB())
{
try {
if (!cb) {
- return SendImpl(valid_remote, MsgI::Serialize(head, body));
+ return SendImpl(remote, MsgI::Serialize(head, body));
} else {
std::string msg_id(head.msg_id());
per_msg_cbs_->Store(msg_id, std::move(cb));
@@ -78,7 +74,7 @@
RecvCB cb_no_use;
per_msg_cbs_->Pick(msg_id, cb_no_use);
};
- return SendImpl(valid_remote, MsgI::Serialize(head, body), onExpireRemoveCB);
+ return SendImpl(remote, MsgI::Serialize(head, body), onExpireRemoveCB);
}
} catch (...) {
SetLastError(eError, "Send internal error.");
@@ -86,15 +82,15 @@
}
}
- 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);
template <class Body>
- bool SendAndRecv(const void *remote, BHMsgHead &head, 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,7 +115,7 @@
std::unique_lock<std::mutex> lk(st->mutex);
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;
@@ -144,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 Store(const std::string &id, RecvCB &&cb) { return store_.emplace(id, std::move(cb)).second; }
- bool Pick(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()) {
@@ -169,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
--
Gitblit v1.8.0