Protocol initialization restructuring.

1 files changed, 44 insertions, 31 deletions

diff --git a/src/protocol/reqrep/req.c b/src/protocol/reqrep/req.c
index ba7d4f6b..c26d66e8 100644
--- a/src/protocol/reqrep/req.c
+++ b/src/protocol/reqrep/req.c
@@ -31,7 +31,6 @@ struct nni_req_sock {
 	nni_thr		resender;
 	int		raw;
 	int		closing;
-	nni_list	pipes;
 	nni_msg *	reqmsg;
 	uint32_t	nextid;         // next id
 	uint8_t		reqid[4];       // outstanding request ID (big endian)
@@ -45,12 +44,10 @@ struct nni_req_pipe {
 	nni_list_node	node;
 };
 
-static void nni_req_receiver(void *);
-static void nni_req_sender(void *);
 static void nni_req_resender(void *);
 
 static int
-nni_req_create(void **reqp, nni_sock *sock)
+nni_req_init(void **reqp, nni_sock *sock)
 {
 	nni_req_sock *req;
 	int rv;
@@ -74,7 +71,6 @@ nni_req_create(void **reqp, nni_sock *sock)
 	req->reqmsg = NULL;
 	req->raw = 0;
 	req->resend = NNI_TIME_ZERO;
-	NNI_LIST_INIT(&req->pipes, nni_req_pipe, node);
 
 	req->uwq = nni_sock_sendq(sock);
 	req->urq = nni_sock_recvq(sock);
@@ -93,7 +89,7 @@ nni_req_create(void **reqp, nni_sock *sock)
 
 
 static void
-nni_req_destroy(void *arg)
+nni_req_fini(void *arg)
 {
 	nni_req_sock *req = arg;
 
@@ -112,37 +108,49 @@ nni_req_destroy(void *arg)
 
 
 static int
-nni_req_add_pipe(void *arg, nni_pipe *pipe, void *data)
+nni_req_pipe_init(void **rpp, nni_pipe *pipe, void *rsock)
 {
-	nni_req_sock *req = arg;
-	nni_req_pipe *rp = data;
-	int rv;
+	nni_req_pipe *rp;
 
+	if ((rp = NNI_ALLOC_STRUCT(rp)) == NULL) {
+		return (NNG_ENOMEM);
+	}
 	rp->pipe = pipe;
 	rp->sigclose = 0;
-	rp->req = req;
-
-	nni_mtx_lock(&req->mx);
-	nni_list_append(&req->pipes, rp);
-	nni_mtx_unlock(&req->mx);
+	rp->req = rsock;
 	return (0);
 }
 
 
 static void
-nni_req_rem_pipe(void *arg, void *data)
+nni_req_pipe_fini(void *arg)
 {
-	nni_req_sock *req = arg;
-	nni_req_pipe *rp = data;
+	nni_req_pipe *rp = arg;
 
-	nni_mtx_lock(&req->mx);
-	nni_list_remove(&req->pipes, rp);
-	nni_mtx_unlock(&req->mx);
+	NNI_FREE_STRUCT(rp);
+}
+
+
+static int
+nni_req_pipe_add(void *arg)
+{
+	// We have nothing to do, since we don't need to maintain a global
+	// list of related pipes.
+	NNI_ARG_UNUSED(arg);
+	return (0);
+}
+
+
+static void
+nni_req_pipe_rem(void *arg)
+{
+	// As with add, nothing to do here.
+	NNI_ARG_UNUSED(arg);
 }
 
 
 static void
-nni_req_sender(void *arg)
+nni_req_pipe_send(void *arg)
 {
 	nni_req_pipe *rp = arg;
 	nni_req_sock *req = rp->req;
@@ -169,7 +177,7 @@ nni_req_sender(void *arg)
 
 
 static void
-nni_req_receiver(void *arg)
+nni_req_pipe_recv(void *arg)
 {
 	nni_req_pipe *rp = arg;
 	nni_req_sock *req = rp->req;
@@ -397,19 +405,24 @@ nni_req_recvfilter(void *arg, nni_msg *msg)
 
 // This is the global protocol structure -- our linkage to the core.
 // This should be the only global non-static symbol in this file.
-struct nni_protocol nni_req_protocol = {
+static nni_proto_pipe nni_req_proto_pipe = {
+	.pipe_init	= nni_req_pipe_init,
+	.pipe_fini	= nni_req_pipe_fini,
+	.pipe_add	= nni_req_pipe_add,
+	.pipe_rem	= nni_req_pipe_rem,
+	.pipe_send	= nni_req_pipe_send,
+	.pipe_recv	= nni_req_pipe_recv,
+};
+
+nni_proto nni_req_protocol = {
 	.proto_self		= NNG_PROTO_REQ,
 	.proto_peer		= NNG_PROTO_REP,
 	.proto_name		= "req",
-	.proto_create		= nni_req_create,
-	.proto_destroy		= nni_req_destroy,
-	.proto_add_pipe		= nni_req_add_pipe,
-	.proto_rem_pipe		= nni_req_rem_pipe,
+	.proto_pipe		= &nni_req_proto_pipe,
+	.proto_init		= nni_req_init,
+	.proto_fini		= nni_req_fini,
 	.proto_setopt		= nni_req_setopt,
 	.proto_getopt		= nni_req_getopt,
-	.proto_pipe_size	= sizeof (nni_req_pipe),
-	.proto_pipe_send	= nni_req_sender,
-	.proto_pipe_recv	= nni_req_receiver,
 	.proto_recv_filter	= nni_req_recvfilter,
 	.proto_send_filter	= nni_req_sendfilter,
 };