From 1d033484ee1a2ec26d3eead073e7bc0f889ffdf4 Mon Sep 17 00:00:00 2001 From: Garrett D'Amore Date: Tue, 15 May 2018 01:47:12 -0700 Subject: fixes #419 want to nni_aio_stop without blocking (#428) * fixes #419 want to nni_aio_stop without blocking This actually introduces an nni_aio_close() API that causes nni_aio_begin to return NNG_ECLOSED, while scheduling a callback on the AIO to do an NNG_ECLOSED as well. This should be called in non-blocking close() contexts instead of nni_aio_stop(), and the cases where we call nni_aio_fini() multiple times are updated updated to add nni_aio_stop() calls on all "interlinked" aios before finalizing them. Furthermore, we call nni_aio_close() as soon as practical in the close path. This closes an annoying race condition where the callback from a lower subsystem could wind up rescheduling an operation that we wanted to abort. --- src/transport/ipc/ipc.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) (limited to 'src/transport/ipc') diff --git a/src/transport/ipc/ipc.c b/src/transport/ipc/ipc.c index 2347e24c..1740bfcb 100644 --- a/src/transport/ipc/ipc.c +++ b/src/transport/ipc/ipc.c @@ -81,6 +81,10 @@ nni_ipc_pipe_close(void *arg) { nni_ipc_pipe *pipe = arg; + nni_aio_close(pipe->rxaio); + nni_aio_close(pipe->txaio); + nni_aio_close(pipe->negaio); + nni_plat_ipc_pipe_close(pipe->ipp); } @@ -644,7 +648,7 @@ nni_ipc_ep_init(void **epp, nni_url *url, nni_sock *sock, int mode) nni_ipc_ep_fini(ep); return (rv); } - ep->proto = nni_sock_proto(sock); + ep->proto = nni_sock_proto_id(sock); *epp = ep; return (0); @@ -655,11 +659,11 @@ nni_ipc_ep_close(void *arg) { nni_ipc_ep *ep = arg; + nni_aio_close(ep->aio); + nni_mtx_lock(&ep->mtx); nni_plat_ipc_ep_close(ep->iep); nni_mtx_unlock(&ep->mtx); - - nni_aio_stop(ep->aio); } static int @@ -883,7 +887,7 @@ static nni_tran_pipe_option nni_ipc_pipe_options[] = { }, }; -static nni_tran_pipe nni_ipc_pipe_ops = { +static nni_tran_pipe_ops nni_ipc_pipe_ops = { .p_fini = nni_ipc_pipe_fini, .p_start = nni_ipc_pipe_start, .p_send = nni_ipc_pipe_send, @@ -924,7 +928,7 @@ static nni_tran_ep_option nni_ipc_ep_options[] = { }, }; -static nni_tran_ep nni_ipc_ep_ops = { +static nni_tran_ep_ops nni_ipc_ep_ops = { .ep_init = nni_ipc_ep_init, .ep_fini = nni_ipc_ep_fini, .ep_connect = nni_ipc_ep_connect, -- cgit v1.2.3-70-g09d2