aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--CMakeLists.txt4
-rw-r--r--src/core/aio.c20
2 files changed, 7 insertions, 17 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt
index e8973244..04cc14de 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -132,9 +132,7 @@ endif ()
# Expire threads. This runs the timeout handling, and having more of them
# reduces contention on the common locks used for aio expiration.
-# The default is to allocate up to max(8, ncpu). The upper limit can be
-# overridden here.
-set(NNG_MAX_EXPIRE_THREADS 8 CACHE STRING "Upper bound on expire threads, between 1...256")
+set(NNG_MAX_EXPIRE_THREADS 8 CACHE STRING "Upper bound on expire threads, 0 for no limit")
mark_as_advanced(NNG_MAX_EXPIRE_THREADS)
if (NNG_MAX_EXPIRE_THREADS)
add_definitions(-DNNG_MAX_EXPIRE_THREADS=${NNG_MAX_EXPIRE_THREADS})
diff --git a/src/core/aio.c b/src/core/aio.c
index 2d288a40..564e91a3 100644
--- a/src/core/aio.c
+++ b/src/core/aio.c
@@ -41,7 +41,7 @@ static int nni_aio_expire_q_cnt;
// condition variable, and expiration thread. By default, this is one
// per CPU core present -- the goal being to reduce overall pressure
// caused by a single lock. The number of queues (and threads) can
-// be tuned using the NNG_EXPIRE_THREADS tunable.
+// be tuned using the NNG_NUM_EXPIRE_THREADS tunable.
//
// We will not permit an AIO
// to be marked done if an expiration is outstanding.
@@ -795,24 +795,16 @@ nni_aio_sys_init(void)
{
int num_thr;
- // We create a thread per CPU core for expiration by default.
+#ifndef NNG_NUM_EXPIRE_THREADS
num_thr = nni_plat_ncpu();
-#ifndef NNG_EXPIRE_THREADS
-#ifndef NNG_MAX_EXPIRE_THREADS
-#define NNG_MAX_EXPIRE_THREADS 8
+#else
+ num_thr = NNG_NUM_EXPIRE_THREADS;
#endif
- if ((num_thr > NNG_MAX_EXPIRE_THREADS) && (NNG_MAX_EXPIRE_THREADS > 0)) {
+#if NNG_MAX_EXPIRE_THREADS > 0
+ if (num_thr > NNG_MAX_EXPIRE_THREADS) {
num_thr = NNG_MAX_EXPIRE_THREADS;
}
-#else
- num_thr = NNG_EXPIRE_THREADS;
#endif
- if (num_thr > 256) { // upper limits
- num_thr = 256;
- }
- if (num_thr < 1) {
- num_thr = 1;
- }
nni_aio_expire_q_list =
nni_zalloc(sizeof(nni_aio_expire_q *) * num_thr);