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.cpp |   61 +++++++++++++++++++++++++-----
 1 files changed, 50 insertions(+), 11 deletions(-)

diff --git a/src/socket.cpp b/src/socket.cpp
index 1315474..6231579 100644
--- a/src/socket.cpp
+++ b/src/socket.cpp
@@ -24,13 +24,13 @@
 using namespace bhome_msg;
 using namespace bhome_shm;
 
-namespace
-{
-
-} // namespace
-
-ShmSocket::ShmSocket(Shm &shm, const MQId &id, const int len) :
+ShmSocket::ShmSocket(Shm &shm, const MQId id, const int len) :
     run_(false), mq_(id, shm, len)
+{
+	Start();
+}
+ShmSocket::ShmSocket(Shm &shm, const bool create_or_else_find, const MQId id, const int len) :
+    run_(false), mq_(id, create_or_else_find, shm, len)
 {
 	Start();
 }
@@ -42,7 +42,45 @@
 
 ShmSocket::~ShmSocket()
 {
-	Stop(); //TODO should stop in sub class, incase thread access sub class data.
+	Stop();
+}
+
+bool ShmSocket::Start(const RawRecvCB &onData, const IdleCB &onIdle, int nworker)
+{
+	auto ioProc = [this, onData, onIdle]() {
+		auto DoSend = [this]() { return send_buffer_.TrySend(mq()); };
+		auto DoRecv = [=] {
+			// do not recv if no cb is set.
+			if (!onData) {
+				return false;
+			}
+			auto onMsg = [&](MsgI &imsg) {
+				DEFER1(imsg.Release());
+				onData(*this, imsg);
+			};
+			MsgI imsg;
+			return mq().TryRecv(imsg) ? (onMsg(imsg), true) : false;
+		};
+
+		try {
+			bool more_to_send = DoSend();
+			bool more_to_recv = DoRecv();
+			if (onIdle) { onIdle(*this); }
+			if (!more_to_send && !more_to_recv) {
+				robust::QuickSleep();
+			}
+		} catch (...) {
+		}
+	};
+
+	std::lock_guard<std::mutex> lock(mutex_);
+	StopNoLock();
+
+	run_.store(true);
+	for (int i = 0; i < nworker; ++i) {
+		workers_.emplace_back([this, ioProc]() { while (run_) { ioProc(); } });
+	}
+	return true;
 }
 
 bool ShmSocket::Start(int nworker, const RecvCB &onData, const IdleCB &onIdle)
@@ -64,13 +102,14 @@
 				return false;
 			}
 			auto onMsg = [&](MsgI &imsg) {
-				DEFER1(imsg.Release(shm()));
+				DEFER1(imsg.Release());
 				BHMsgHead head;
 				if (imsg.ParseHead(head)) {
 					onRecvWithPerMsgCB(*this, imsg, head);
 				}
 			};
-			return mq().TryRecvAll(onMsg) > 0; // this will recv all msgs.
+			MsgI imsg;
+			return mq().TryRecv(imsg) ? (onMsg(imsg), true) : false;
 		};
 
 		try {
@@ -78,7 +117,7 @@
 			bool more_to_recv = DoRecv();
 			if (onIdle) { onIdle(*this); }
 			if (!more_to_send && !more_to_recv) {
-				std::this_thread::yield();
+				robust::QuickSleep();
 			}
 		} catch (...) {
 		}
@@ -123,7 +162,7 @@
 		if (msg.ParseHead(head)) {
 			return true;
 		} else {
-			msg.Release(shm());
+			msg.Release();
 		}
 	}
 	return false;

--
Gitblit v1.8.0