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/topic_node.cpp |   95 +++++++++++++++++++++++++++++++++++------------
 1 files changed, 71 insertions(+), 24 deletions(-)

diff --git a/src/topic_node.cpp b/src/topic_node.cpp
index d8d6a42..35228b4 100644
--- a/src/topic_node.cpp
+++ b/src/topic_node.cpp
@@ -42,7 +42,6 @@
 TopicNode::TopicNode(SharedMemory &shm) :
     shm_(shm), state_(eStateUnregistered)
 {
-	Init();
 }
 
 TopicNode::~TopicNode()
@@ -57,34 +56,79 @@
 
 	if (Valid()) {
 		return true;
+	} else if (info_.proc_id().empty()) {
+		return false;
 	}
 
 	if (ssn_id_ == 0) {
 		ssn_id_ = ShmMsgQueue::NewId();
 	}
 	LOG_DEBUG() << "Node Init, id " << ssn_id_;
-	MsgI msg;
-	msg.OffsetRef() = ssn_id_;
-	if (ShmMsgQueue::TrySend(shm(), BHInitAddress(), msg)) {
-
-		auto end_time = steady_clock::now() + 3s;
-		do {
-			try {
-				for (int i = eSockStart; i < eSockEnd; ++i) {
-					sockets_.emplace_back(new ShmSocket(shm_, false, ssn_id_ + i, kMqLen));
+	auto NodeInit = [&]() {
+		auto SendInitCmd = [&]() {
+			int64_t init_cmd = ssn_id_ << 4 | EncodeCmd(eCmdNodeInit);
+			auto end_time = steady_clock::now() + 3s;
+			bool r = false;
+			do {
+				r = ShmMsgQueue::TrySend(shm(), BHTopicCenterAddress(), init_cmd);
+			} while (!r && steady_clock::now() < end_time);
+			return r;
+		};
+		if (SendInitCmd()) {
+			LOG_DEBUG() << "node send init ok";
+			auto end_time = steady_clock::now() + 3s;
+			do {
+				try {
+					for (int i = eSockStart; i < eSockEnd; ++i) {
+						sockets_.emplace_back(new ShmSocket(shm_, false, ssn_id_ + i, kMqLen));
+					}
+					break;
+				} catch (...) {
+					sockets_.clear();
+					std::this_thread::sleep_for(100ms);
 				}
-				break;
-			} catch (...) {
-				sockets_.clear();
-				std::this_thread::sleep_for(100ms);
-			}
-		} while (steady_clock::now() < end_time);
-		if (!sockets_.empty()) {
-			// recv msgs to avoid memory leak.
-			auto default_ignore_msg = [](ShmSocket &sock, MsgI &imsg, BHMsgHead &head) { return true; };
-			SockNode().Start(default_ignore_msg);
-			return true;
+			} while (steady_clock::now() < end_time);
 		}
+	};
+	if (sockets_.empty()) {
+		NodeInit();
+	}
+	if (!sockets_.empty()) {
+		LOG_DEBUG() << "node sockets ok";
+		auto onNodeCmd = [this](ShmSocket &socket, int64_t &val) {
+			LOG_DEBUG() << "node recv cmd: " << DecodeCmd(val);
+			switch (DecodeCmd(val)) {
+			case eCmdNodeInitReply: {
+				MsgI msg(val >> 4);
+				DEFER1(msg.Release());
+				MsgProcInit body;
+				auto head = InitMsgHead(GetType(body), info_.proc_id(), ssn_id_);
+				head.add_route()->set_mq_id(ssn_id_);
+				if (msg.Fill(head, body)) {
+					socket.Send(BHTopicCenterAddress(), msg);
+				}
+			} break;
+			default:
+				break;
+			}
+			return true;
+		};
+
+		// recv msgs to avoid memory leak.
+		auto onMsg = [this](ShmSocket &sock, MsgI &imsg, BHMsgHead &head) {
+			LOG_DEBUG() << "node recv type: " << head.type();
+			if (head.type() == kMsgTypeProcInitReply) {
+				LOG_DEBUG() << "got proc init reply";
+				MsgProcInitReply reply;
+				if (imsg.ParseBody(reply)) {
+					SetProcIndex(reply.proc_index());
+				}
+			}
+			return true;
+		};
+		SockNode().Start(1, onMsg, onNodeCmd);
+		LOG_DEBUG() << "sockets ok.";
+		return true;
 	}
 	return false;
 }
@@ -100,7 +144,7 @@
 	} else if (nworker > 16) {
 		nworker = 16;
 	}
-	SockNode().Start();
+	// SockNode().Start();
 	ServerStart(server_cb, nworker);
 	SubscribeStartWorker(sub_cb, nworker);
 	ClientStartWorker(client_cb, nworker);
@@ -114,12 +158,15 @@
 
 bool TopicNode::Register(ProcInfo &proc, MsgCommonReply &reply_body, const int timeout_ms)
 {
+	{
+		std::lock_guard<std::mutex> lk(mutex_);
+		info_ = proc;
+	}
+
 	if (!Init()) {
 		SetLastError(eError, kErrMsgNotInit);
 		return false;
 	}
-
-	info_ = proc;
 
 	auto &sock = SockNode();
 	MsgRegister body;

--
Gitblit v1.8.0