From 1ad7f88e0c58285c9e1cad9448d5a1fcfa7a07f1 Mon Sep 17 00:00:00 2001
From: Garrett D'Amore <garrett@damore.org>
Date: Thu, 26 Dec 2024 14:10:09 -0800
Subject: aio: nng_aio_defer replaced by nng_aio_start

This represents an API change, and we remove the nng_aio_begin
function as well, introducing the lightweight nng_aio_reset instead.
---
 src/core/aio.c      | 63 -----------------------------------------------------
 src/core/aio.h      |  5 -----
 src/core/aio_test.c |  3 +--
 3 files changed, 1 insertion(+), 70 deletions(-)

(limited to 'src/core')

diff --git a/src/core/aio.c b/src/core/aio.c
index 87c79d0d..fa6eb76f 100644
--- a/src/core/aio.c
+++ b/src/core/aio.c
@@ -427,69 +427,6 @@ nni_aio_schedule(nni_aio *aio, nni_aio_cancel_fn cancel, void *data)
 	return (0);
 }
 
-bool
-nni_aio_defer(nni_aio *aio, nni_aio_cancel_fn cancel, void *data)
-{
-	nni_aio_expire_q *eq      = aio->a_expire_q;
-	bool              timeout = false;
-
-	if (!aio->a_sleep && !aio->a_use_expire) {
-		// Convert the relative timeout to an absolute timeout.
-		switch (aio->a_timeout) {
-		case NNG_DURATION_ZERO:
-			timeout = true;
-			break;
-		case NNG_DURATION_INFINITE:
-		case NNG_DURATION_DEFAULT:
-			aio->a_expire = NNI_TIME_NEVER;
-			break;
-		default:
-			aio->a_expire = nni_clock() + aio->a_timeout;
-			break;
-		}
-	} else if (aio->a_use_expire && aio->a_expire <= nni_clock()) {
-		timeout = true;
-	}
-
-	nni_mtx_lock(&eq->eq_mtx);
-	if (aio->a_stop || eq->eq_stop) {
-		aio->a_stop    = true;
-		aio->a_sleep   = false;
-		aio->a_result  = NNG_ESTOPPED;
-		aio->a_stopped = true;
-		nni_mtx_unlock(&eq->eq_mtx);
-		nni_task_dispatch(&aio->a_task);
-		return (false);
-	}
-	if (aio->a_abort) {
-		aio->a_sleep = false;
-		aio->a_abort = false;
-		nni_mtx_unlock(&eq->eq_mtx);
-		nni_task_dispatch(&aio->a_task);
-		return (false);
-	}
-	if (timeout) {
-		aio->a_sleep  = false;
-		aio->a_result = aio->a_expire_ok ? 0 : NNG_ETIMEDOUT;
-		aio->a_abort  = false;
-		nni_mtx_unlock(&eq->eq_mtx);
-		nni_task_dispatch(&aio->a_task);
-		return (false);
-	}
-
-	NNI_ASSERT(aio->a_cancel_fn == NULL);
-	aio->a_cancel_fn  = cancel;
-	aio->a_cancel_arg = data;
-
-	// We only schedule expiration if we have a way for the expiration
-	// handler to actively cancel it.
-	if ((aio->a_expire != NNI_TIME_NEVER) && (cancel != NULL)) {
-		nni_aio_expire_add(aio);
-	}
-	nni_mtx_unlock(&eq->eq_mtx);
-	return (true);
-}
-
 bool
 nni_aio_start(nni_aio *aio, nni_aio_cancel_fn cancel, void *data)
 {
diff --git a/src/core/aio.h b/src/core/aio.h
index d909853f..5346d19b 100644
--- a/src/core/aio.h
+++ b/src/core/aio.h
@@ -170,11 +170,6 @@ extern void         nni_aio_bump_count(nni_aio *, size_t);
 // cannot be canceled, which can lead to apparent hangs.
 extern int nni_aio_schedule(nni_aio *, nni_aio_cancel_fn, void *);
 
-// nni_aio_defer is just like nni_io_schedule, but it also calls the callback
-// automatically if the operation cannot be started because the AIO is stopped
-// or was canceled before this call (but after nni_aio_begin).
-extern bool nni_aio_defer(nni_aio *, nni_aio_cancel_fn, void *);
-
 // nni_aio_reset is called by providers before doing any work -- it resets
 // counts other fields to their initial state.  It will not reset the closed
 // state if the aio has been stopped or closed.
diff --git a/src/core/aio_test.c b/src/core/aio_test.c
index 66682460..a41f608d 100644
--- a/src/core/aio_test.c
+++ b/src/core/aio_test.c
@@ -130,8 +130,7 @@ test_provider_cancel(void)
 	int      rv = 0;
 	// We fake an empty provider that does not do anything.
 	NUTS_PASS(nng_aio_alloc(&aio, NULL, NULL));
-	NUTS_TRUE(nng_aio_begin(aio) == true);
-	nng_aio_defer(aio, cancel, &rv);
+	nng_aio_start(aio, cancel, &rv);
 	nng_aio_cancel(aio);
 	nng_aio_wait(aio);
 	NUTS_TRUE(rv == NNG_ECANCELED);
-- 
cgit v1.2.3-70-g09d2