lichao
2021-04-06 4deeafbd502dc3c57dab8ad6ca601a38a9e7f074
src/shm_queue.cpp
@@ -21,6 +21,11 @@
#include <boost/uuid/uuid_generators.hpp>
#include <boost/uuid/uuid_io.hpp>
// TODO !!! add garbage collection, and handshake with gc-center.
// smart pointer will NOT work if user crash.
// problem of process release mq: client query success; server exits; client send.
//  client query success; server exits; client send.
namespace bhome_shm
{
using namespace bhome_msg;
@@ -61,53 +66,27 @@
ShmMsgQueue::~ShmMsgQueue()
{
   Remove();
   // It's not safe to remove, others may still holder pointers and write to it.
   // TODO use smart_ptr or garbage collection.
   //Remove();
}
bool ShmMsgQueue::Send(SharedMemory &shm, const MQId &remote_id, const MsgI &msg, const int timeout_ms, OnSend const &onsend)
{
   Queue *remote = Find(shm, MsgQIdToName(remote_id));
   return remote && remote->Write(msg, timeout_ms, [&onsend](const MsgI &msg) { onsend(); msg.AddRef(); });
}
bool ShmMsgQueue::Send(SharedMemory &shm, const MQId &remote_id, const MsgI &msg, const int timeout_ms)
{
   Queue *remote = Find(shm, MsgQIdToName(remote_id));
   return remote && remote->Write(msg, timeout_ms, [](const MsgI &msg) { msg.AddRef(); });
}
// bool ShmMsgQueue::Send(const MQId &remote_id, const MsgI &msg, const int timeout_ms)
// {
//     Queue *remote = Find(MsgQIdToName(remote_id));
//     return remote && remote->Write(msg, timeout_ms, [](const MsgI&msg){msg.AddRef();});
// }
// Test shows that in the 2 cases:
// 1) build msg first, then find remote queue;
// 2) find remote queue first, then build msg;
// 1 is about 50% faster than 2, maybe cache related.
bool ShmMsgQueue::Send(const MQId &remote_id, const BHMsg &data, const int timeout_ms)
{
   MsgI msg;
   if (msg.Make(shm(), data)) {
      if (Send(remote_id, msg, timeout_ms)) {
         return true;
      } else {
         msg.Release(shm());
      }
   }
   return false;
}
/*
bool ShmMsgQueue::Send(const MQId &remote_id, const void *data, const size_t size, const int timeout_ms)
{
    // Test shows that in the 2 cases:
    // 1) build msg first, then find remote queue;
    // 2) find remote queue first, then build msg;
    // 1 is about 50% faster than 2, maybe cache related.
    MsgI msg;
    if(msg.BuildRequest(shm(), Id(), data, size)) {
        if(Send(remote_id, msg, timeout_ms)) {
            return true;
        } else {
            msg.Release(shm());
        }
    }
    return false;
}
//*/
bool ShmMsgQueue::Recv(BHMsg &msg, const int timeout_ms)
{
   MsgI imsg;