Attempts to minimize races, remove unused nni_sock_mtx function.

We still have endpoint related races apparently; we need to examine
the possibility of handling endpoints much like we do pipes, which seem
to be race free.

1 files changed, 0 insertions, 8 deletions

diff --git a/src/core/socket.h b/src/core/socket.h
index c4619b42..76b57f09 100644
--- a/src/core/socket.h
+++ b/src/core/socket.h
@@ -113,14 +113,6 @@ extern nni_msgq *nni_sock_sendq(nni_sock *);
 // inject incoming messages from pipes to it.
 extern nni_msgq *nni_sock_recvq(nni_sock *);
 
-// nni_sock_mtx obtains the socket mutex.  This is for protocols to use
-// from separate threads; they must not hold the lock for extended periods.
-// Additionally, this can only be acquired from separate threads.  The
-// synchronous entry points (excluding the send/recv thread workers) will
-// be called with this lock already held.  We expose the mutex directly
-// here so that protocols can use it to initialize condvars.
-extern nni_mtx *nni_sock_mtx(nni_sock *);
-
 extern nni_duration nni_sock_linger(nni_sock *);
 extern size_t       nni_sock_rcvmaxsz(nni_sock *);
 extern void nni_sock_reconntimes(nni_sock *, nni_duration *, nni_duration *);