Cosmetic improvements.

[invirt/third/libt4.git] / rpc / rpc.h

diff --git a/rpc/rpc.h b/rpc/rpc.h
index 245f277..211c717 100644 (file)
--- a/rpc/rpc.h
+++ b/rpc/rpc.h
@@ -1,269 +1,219 @@
 #ifndef rpc_h
 #define rpc_h
 
 #ifndef rpc_h
 #define rpc_h
 

+#include "types.h"

 #include <sys/socket.h>
 #include <netinet/in.h>
 #include <sys/socket.h>
 #include <netinet/in.h>

-#include <list>
-#include <map>
-#include <stdio.h>

 
 

+#include "rpc_protocol.h"

 #include "thr_pool.h"
 #include "marshall.h"
 #include "thr_pool.h"
 #include "marshall.h"

+#include "marshall_wrap.h"

 #include "connection.h"
 
 #include "connection.h"
 

-#ifdef DMALLOC
-#include "dmalloc.h"
-#endif
+namespace rpc {
+    static constexpr milliseconds to_max{12000};
+    static constexpr milliseconds to_min{100};
+}

 
 

-class rpc_const {
-    public:
-        static const unsigned int bind = 1;   // handler number reserved for bind
-        static const int timeout_failure = -1;
-        static const int unmarshal_args_failure = -2;
-        static const int unmarshal_reply_failure = -3;
-        static const int atmostonce_failure = -4;
-        static const int oldsrv_failure = -5;
-        static const int bind_failure = -6;
-        static const int cancel_failure = -7;
-};
+template<class P, class R, class ...Args> struct is_valid_call : false_type {};

 
 

-struct ReturnOnFailure {
-    static inline int unmarshall_args_failure() {
-        return rpc_const::unmarshal_args_failure;
-    }
-};
+template<class S, class R, class ...Args>
+struct is_valid_call<S(R &, Args...), R, Args...> : true_type {};
+
+template<class P, class F> struct is_valid_registration : false_type {};
+
+template<class S, class R, class ...Args>
+struct is_valid_registration<S(R &, typename decay<Args>::type...), S(R &, Args...)> : true_type {};
+
+template<class P, class C, class S, class R, class ...Args>
+struct is_valid_registration<P, S(C::*)(R &, Args...)> : is_valid_registration<P, S(R &, Args...)> {};

 
 // rpc client endpoint.
 // manages a xid space per destination socket
 // threaded: multiple threads can be sending RPCs,
 
 // rpc client endpoint.
 // manages a xid space per destination socket
 // threaded: multiple threads can be sending RPCs,

-class rpcc : public chanmgr {
-
+class rpcc : private connection_delegate {

     private:
     private:

+        using proc_id_t = rpc_protocol::proc_id_t;
+        template <class S>
+        using proc_t = rpc_protocol::proc_t<S>;
+        using nonce_t = rpc_protocol::nonce_t;
+        using xid_t = rpc_protocol::xid_t;

 
 

-        //manages per rpc info
+        // manages per rpc info

         struct caller {
         struct caller {

-            caller(unsigned int xxid, unmarshall *un);
-            ~caller();
+            caller(xid_t _xid, string *_rep) : xid(_xid), rep(_rep) {}

 
 

-            unsigned int xid;
-            unmarshall *un;
+            int xid;
+            string *rep;

             int intret;
             int intret;

-            bool done;
-            std::mutex m;
-            std::condition_variable c;
+            bool done = false;
+            mutex m;
+            cond c;

         };
 
         };
 

-        void get_refconn(connection **ch);
-        void update_xid_rep(unsigned int xid);
+        void get_latest_connection(shared_ptr<connection> & ch);
+        void update_xid_rep(xid_t xid, lock & m_lock);

 
 
         sockaddr_in dst_;
 
 
         sockaddr_in dst_;

-        unsigned int clt_nonce_;
-        unsigned int srv_nonce_;
-        bool bind_done_;
-        unsigned int xid_;
-        int lossytest_;
-        bool retrans_;
-        bool reachable_;
-
-        connection *chan_;
-
-        std::mutex m_; // protect insert/delete to calls[]
-        std::mutex chan_m_;
-
-        bool destroy_wait_;
-        std::condition_variable destroy_wait_c_;
-
-        std::map<int, caller *> calls_;
-        std::list<unsigned int> xid_rep_window_;
-                
-                struct request {
-                    request() { clear(); }
-                    void clear() { buf.clear(); xid = -1; }
-                    bool isvalid() { return xid != -1; }
-                    std::string buf;
-                    int xid;
-                };
-                struct request dup_req_;
-                int xid_rep_done_;
-    public:
-
-        rpcc(sockaddr_in d, bool retrans=true);
-        ~rpcc();
-
-        struct TO {
-            int to;
-        };
-        static const TO to_max;
-        static const TO to_min;
-        static TO to(int x) { TO t; t.to = x; return t;}
+        nonce_t clt_nonce_;
+        nonce_t srv_nonce_ = 0;
+        bool bind_done_ = false;
+        int lossytest_ = 0;
+        bool reachable_ = true;

 
 

-        unsigned int id() { return clt_nonce_; }
+        shared_ptr<connection> chan_;

 
 

-        int bind(TO to = to_max);
+        mutex m_; // protect insert/delete to calls[]
+        mutex chan_m_;

 
 

-        void set_reachable(bool r) { reachable_ = r; }
+        bool destroy_wait_ = false;
+        cond destroy_wait_c_;

 
 

-        void cancel();
-                
-                int islossy() { return lossytest_ > 0; }
+        map<int, caller *> calls_;

 
 

-        int call1(unsigned int proc, 
-                marshall &req, unmarshall &rep, TO to);
+        // xid starts with 1 and latest received reply starts with 0
+        xid_t xid_ = 1;
+        list<xid_t> xid_rep_window_ = {0};

 
 

-        bool got_pdu(connection *c, char *b, int sz);
+        struct request {
+            void clear() { buf.clear(); xid = -1; }
+            bool isvalid() { return xid != -1; }
+            string buf;
+            xid_t xid = -1;
+        };
+        request dup_req_;
+        int xid_rep_done_ = -1;

 
 

+        int call1(proc_id_t proc, milliseconds to, string & rep, marshall & req);

 
         template<class R>
 
         template<class R>

-            int call_m(unsigned int proc, marshall &req, R & r, TO to);
+        inline int call_m(proc_id_t proc, milliseconds to, R & r, marshall && req) {
+            string rep;
+            int intret = call1(proc, to, rep, req);
+            if (intret < 0) return intret;
+            unmarshall u(rep, true, r);
+            if (u.okdone() != true) {
+                LOG("rpcc::call_m: failed to unmarshall the reply.  You are probably " <<
+                    "calling RPC 0x" << hex << proc << " with the wrong return type.");
+                VERIFY(0);
+                return rpc_protocol::unmarshall_reply_failure;
+            }
+            return intret;
+        }

 
 

-        template<class R, typename ...Args>
-            inline int call(unsigned int proc, R & r, const Args&... args);
+        bool got_pdu(const shared_ptr<connection> & c, const string & b);

 
 

-        template<class R, typename ...Args>
-            inline int call_timeout(unsigned int proc, TO to, R & r, const Args&... args);
-};
+    public:

 
 

-template<class R> int 
-rpcc::call_m(unsigned int proc, marshall &req, R & r, TO to) 
-{
-    unmarshall u;
-    int intret = call1(proc, req, u, to);
-    if (intret < 0) return intret;
-    u >> r;
-    if (u.okdone() != true) {
-        fprintf(stderr, "rpcc::call_m: failed to unmarshall the reply."
-                "You are probably calling RPC 0x%x with wrong return "
-                "type.\n", proc);
-        VERIFY(0);
-        return rpc_const::unmarshal_reply_failure;
-    }
-    return intret;
-}
+        rpcc(const string & d);
+        ~rpcc();

 
 

-template<class R, typename... Args> inline int
-rpcc::call(unsigned int proc, R & r, const Args&... args)
-{
-    return call_timeout(proc, rpcc::to_max, r, args...);
-}
+        nonce_t id() { return clt_nonce_; }

 
 

-template<class R, typename... Args> inline int
-rpcc::call_timeout(unsigned int proc, const rpcc::TO to, R & r, const Args&... args)
-{
-    marshall m{args...};
-    return call_m(proc, m, r, to);
-}
+        int bind(milliseconds to = rpc::to_max);

 
 

-bool operator<(const sockaddr_in &a, const sockaddr_in &b);
+        void set_reachable(bool r) { reachable_ = r; }

 
 

-// rpc server endpoint.
-class rpcs : public chanmgr {
+        void cancel();

 
 

-    typedef enum {
-        NEW,  // new RPC, not a duplicate
-        INPROGRESS, // duplicate of an RPC we're still processing
-        DONE, // duplicate of an RPC we already replied to (have reply)
-        FORGOTTEN,  // duplicate of an old RPC whose reply we've forgotten
-    } rpcstate_t;
+        template<class P, class R, typename ...Args>
+        inline int call(proc_t<P> proc, R & r, const Args & ... args) {
+            return call_timeout(proc, rpc::to_max, r, args...);
+        }

 
 

+        template<class P, class R, typename ...Args>
+        inline int call_timeout(proc_t<P> proc, milliseconds to, R & r, const Args & ... args) {
+            static_assert(is_valid_call<P, R, Args...>::value, "RPC called with incorrect argument types");
+            return call_m(proc.id, to, r, forward<marshall>(marshall(args...)));
+        }
+};
+
+// rpc server endpoint.
+class rpcs : private connection_delegate {

     private:
     private:

+        using proc_id_t = rpc_protocol::proc_id_t;
+        template <class S>
+        using proc_t = rpc_protocol::proc_t<S>;
+        using nonce_t = rpc_protocol::nonce_t;
+        using xid_t = rpc_protocol::xid_t;
+
+        typedef enum {
+            NEW,  // new RPC, not a duplicate
+            INPROGRESS, // duplicate of an RPC we're still processing
+            DONE, // duplicate of an RPC we already replied to (have reply)
+            FORGOTTEN,  // duplicate of an old RPC whose reply we've forgotten
+        } rpcstate_t;

 
         // state about an in-progress or completed RPC, for at-most-once.
         // if cb_present is true, then the RPC is complete and a reply
         // has been sent; in that case buf points to a copy of the reply,
         // and sz holds the size of the reply.
 
         // state about an in-progress or completed RPC, for at-most-once.
         // if cb_present is true, then the RPC is complete and a reply
         // has been sent; in that case buf points to a copy of the reply,
         // and sz holds the size of the reply.

-    struct reply_t {
-        reply_t (unsigned int _xid) {
-            xid = _xid;
-            cb_present = false;
-            buf = NULL;
-            sz = 0;
-        }
-        reply_t (unsigned int _xid, char *_buf, int _sz) {
-            xid = _xid;
-            cb_present = true;
-            buf = _buf;
-            sz = _sz;
-        }
-        unsigned int xid;
-        bool cb_present; // whether the reply buffer is valid
-        char *buf;      // the reply buffer
-        int sz;         // the size of reply buffer
-    };
+        struct reply_t {
+            reply_t (xid_t _xid) : xid(_xid), cb_present(false) {}
+            reply_t (xid_t _xid, const string & _buf) : xid(_xid), cb_present(true), buf(_buf) {}
+            xid_t xid;
+            bool cb_present; // whether the reply buffer is valid
+            string buf;      // the reply buffer
+        };

 
 

-    int port_;
-    unsigned int nonce_;
+        in_port_t port_;
+        nonce_t nonce_;

 
 

-    // provide at most once semantics by maintaining a window of replies
-    // per client that that client hasn't acknowledged receiving yet.
+        // provide at most once semantics by maintaining a window of replies
+        // per client that that client hasn't acknowledged receiving yet.

         // indexed by client nonce.
         // indexed by client nonce.

-    std::map<unsigned int, std::list<reply_t> > reply_window_;
-
-    void free_reply_window(void);
-    void add_reply(unsigned int clt_nonce, unsigned int xid, char *b, int sz);
+        map<nonce_t, list<reply_t>> reply_window_;

 
 

-    rpcstate_t checkduplicate_and_update(unsigned int clt_nonce, 
-            unsigned int xid, unsigned int rep_xid,
-            char **b, int *sz);
+        void add_reply(nonce_t clt_nonce, xid_t xid, const string & b);

 
 

-    void updatestat(unsigned int proc);
+        rpcstate_t check_duplicate_and_update(nonce_t clt_nonce, xid_t xid,
+                xid_t rep_xid, string & b);

 
 

-    // latest connection to the client
-    std::map<unsigned int, connection *> conns_;
+        // latest connection to the client
+        map<nonce_t, shared_ptr<connection>> conns_;

 
 

-    // counting
-    const int counting_;
-    int curr_counts_;
-    std::map<int, int> counts_;
+        bool reachable_ = true;

 
 

-    int lossytest_; 
-    bool reachable_;
+        // map proc # to function
+        map<proc_id_t, handler *> procs_;

 
 

-    // map proc # to function
-    std::map<int, handler *> procs_;
+        mutex procs_m_; // protect insert/delete to procs[]
+        mutex reply_window_m_; // protect reply window et al
+        mutex conns_m_; // protect conns_

 
 

-    std::mutex procs_m_; // protect insert/delete to procs[]
-    std::mutex count_m_;  //protect modification of counts
-    std::mutex reply_window_m_; // protect reply window et al
-    std::mutex conss_m_; // protect conns_
+        void dispatch(shared_ptr<connection> c, const string & buf);

 
 

+        unique_ptr<thread_pool> dispatchpool_{new thread_pool(6, false)};
+        unique_ptr<connection_listener> listener_;

 
 

-    protected:
+        // RPC handler for clients binding
+        rpc_protocol::status rpcbind(nonce_t & r);

 
 

-    struct djob_t {
-        djob_t (connection *c, char *b, int bsz):buf(b),sz(bsz),conn(c) {}
-        char *buf;
-        int sz;
-        connection *conn;
-    };
-    void dispatch(djob_t *);
-
-    // internal handler registration
-    void reg1(unsigned int proc, handler *);
-
-    ThrPool* dispatchpool_;
-    tcpsconn* listener_;
+        bool got_pdu(const shared_ptr<connection> & c, const string & b);

 
     public:
 
     public:

-    rpcs(unsigned int port, int counts=0);
-    ~rpcs();
-    inline int port() { return listener_->port(); }
-    //RPC handler for clients binding
-    int rpcbind(int &r, int a);
-
-    void set_reachable(bool r) { reachable_ = r; }

 
 

-    bool got_pdu(connection *c, char *b, int sz);
+        rpcs(in_port_t port);
+        ~rpcs();

 
 

-    template<class F, class C=void> void reg(unsigned int proc, F f, C *c=nullptr);
-};
+        void set_reachable(bool r) { reachable_ = r; }

 
 

-template<class F, class C> void rpcs::reg(unsigned int proc, F f, C *c) {
-    reg1(proc, marshalled_func<F, F, ReturnOnFailure>::wrap(f, c));
-}
+        template<class P, class F, class C=void> inline void reg(proc_t<P> proc, F f, C *c=nullptr) {
+            static_assert(is_valid_registration<P, F>::value, "RPC handler registered with incorrect argument types");
+            struct ReturnOnFailure {
+                static inline int unmarshall_args_failure() {
+                    return rpc_protocol::unmarshall_args_failure;
+                }
+            };
+            lock pl(procs_m_);
+            VERIFY(procs_.count(proc.id) == 0);
+            procs_[proc.id] = marshalled_func<F, ReturnOnFailure>::wrap(f, c);
+            VERIFY(procs_.count(proc.id) >= 1);
+        }

 
 

-void make_sockaddr(const char *hostandport, struct sockaddr_in *dst);
-void make_sockaddr(const char *host, const char *port,
-        struct sockaddr_in *dst);
+        void start();
+};

 
 #endif
 
 #endif