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