From db322f33ba13592f2492317e3f1a070454c97059 Mon Sep 17 00:00:00 2001
From: lichao <lichao@aiotlink.com>
Date: 星期四, 13 五月 2021 19:34:46 +0800
Subject: [PATCH] center alloc all msgs.

---
 src/shm_msg_queue.h |   59 ++++++++++++++++++++++++++++++++++++++++++-----------------
 1 files changed, 42 insertions(+), 17 deletions(-)

diff --git a/src/shm_msg_queue.h b/src/shm_msg_queue.h
index 1970803..4b7aed8 100644
--- a/src/shm_msg_queue.h
+++ b/src/shm_msg_queue.h
@@ -24,19 +24,25 @@
 using namespace bhome_shm;
 using namespace bhome_msg;
 
+#define BH_USE_ATOMIC_Q
+
 class ShmMsgQueue : public StaticDataRef<std::atomic<uint64_t>, ShmMsgQueue>
 {
-	// typedef ShmObject<SharedQ63<4>> Shmq;
-	typedef ShmObject<SharedQueue<int64_t>> Shmq;
-	typedef Shmq::ShmType ShmType;
-	typedef Shmq::Data Queue;
-	typedef std::function<void()> OnSend;
-	typedef robust::FMutex Mutex;
-	// typedef robust::SemMutex Mutex;
-	// typedef robust::NullMutex Mutex;
-	typedef robust::Guard<Mutex> Guard;
-
 public:
+	typedef int64_t RawData;
+
+#ifdef BH_USE_ATOMIC_Q
+	typedef ShmObject<SharedQ63<0>> Shmq;
+#else
+	typedef ShmObject<SharedQueue<RawData>> Shmq;
+	// typedef robust::FMutex Mutex;
+	// typedef robust::SemMutex Mutex;
+	typedef robust::NullMutex Mutex;
+	typedef robust::Guard<Mutex> Guard;
+#endif
+
+	typedef Shmq::Data Queue;
+	typedef Shmq::ShmType ShmType;
 	typedef uint64_t MQId;
 
 	static MQId NewId();
@@ -45,26 +51,45 @@
 	ShmMsgQueue(const MQId id, const bool create_or_else_find, ShmType &segment, const int len);
 	ShmMsgQueue(ShmType &segment, const int len);
 	~ShmMsgQueue();
-	static bool Remove(SharedMemory &shm, const MQId id);
+	static bool Remove(ShmType &shm, const MQId id);
 	MQId Id() const { return id_; }
 	ShmType &shm() const { return queue_.shm(); }
 
-	bool Recv(MsgI &msg, const int timeout_ms)
+	bool Recv(RawData &val, const int timeout_ms)
 	{
+#ifndef BH_USE_ATOMIC_Q
 		Guard lock(GetMutex(Id()));
-		return queue().Read(msg.OffsetRef(), timeout_ms);
+#endif
+		return queue().Read(val, timeout_ms);
 	}
-	bool TryRecv(MsgI &msg)
+
+	bool TryRecv(RawData &val)
 	{
+#ifndef BH_USE_ATOMIC_Q
 		Guard lock(GetMutex(Id()));
-		return queue().TryRead(msg.OffsetRef());
+#endif
+		return queue().TryRead(val);
 	}
-	static Queue *Find(SharedMemory &shm, const MQId remote_id);
-	static bool TrySend(SharedMemory &shm, const MQId remote_id, MsgI msg);
+
+	bool Recv(MsgI &msg, const int timeout_ms) { return Recv(msg.OffsetRef(), timeout_ms); }
+	bool TryRecv(MsgI &msg) { return TryRecv(msg.OffsetRef()); }
+	static Queue *Find(ShmType &shm, const MQId remote_id);
+	static bool TrySend(ShmType &shm, const MQId remote_id, const RawData val);
+	static bool TrySend(ShmType &shm, const MQId remote_id, MsgI msg)
+	{
+		bool r = false;
+		msg.AddRef(); // TODO check if we could avoid addref here.
+		DEFER1(if (!r) { msg.Release(); });
+		r = TrySend(shm, remote_id, msg.Offset());
+		return r;
+	}
 	bool TrySend(const MQId remote_id, const MsgI &msg) { return TrySend(shm(), remote_id, msg); }
+	bool TrySend(const MQId remote_id, const RawData val) { return TrySend(shm(), remote_id, val); }
 
 private:
+#ifndef BH_USE_ATOMIC_Q
 	static Mutex &GetMutex(const MQId id);
+#endif
 	MQId id_;
 	Queue &queue() { return *queue_.data(); }
 	Shmq queue_;

--
Gitblit v1.8.0