From b7f35d76c5504d8e8d4f9f99616c022c7872850e Mon Sep 17 00:00:00 2001 From: Jolene Tan <32562011+zxjtan@users.noreply.github.com> Date: Thu, 19 Oct 2023 02:17:53 +0800 Subject: [PATCH] Use CMAKE_USE_PTHREADS_INIT (#1576) --- .CMake/compiler_opts.cmake | 6 ++---- src/oqsconfig.h.cmake | 4 ++-- tests/test_kem.c | 6 +++--- tests/test_sig.c | 6 +++--- 4 files changed, 10 insertions(+), 12 deletions(-) diff --git a/.CMake/compiler_opts.cmake b/.CMake/compiler_opts.cmake index e160bdcfec..98fe0b72b7 100644 --- a/.CMake/compiler_opts.cmake +++ b/.CMake/compiler_opts.cmake @@ -106,8 +106,7 @@ if(CMAKE_C_COMPILER_ID MATCHES "Clang") if(NOT ${OQS_BUILD_ONLY_LIB}) set(THREADS_PREFER_PTHREAD_FLAG ON) - find_package(Threads REQUIRED) - set(OQS_USE_PTHREADS_IN_TESTS 1) + find_package(Threads) endif() if(${OQS_DEBUG_BUILD}) @@ -168,8 +167,7 @@ elseif(CMAKE_C_COMPILER_ID STREQUAL "GNU") if(NOT ${OQS_BUILD_ONLY_LIB}) set(THREADS_PREFER_PTHREAD_FLAG ON) - find_package(Threads REQUIRED) - set(OQS_USE_PTHREADS_IN_TESTS 1) + find_package(Threads) endif() if(${OQS_DEBUG_BUILD}) diff --git a/src/oqsconfig.h.cmake b/src/oqsconfig.h.cmake index 6c26745f51..602788c7dd 100644 --- a/src/oqsconfig.h.cmake +++ b/src/oqsconfig.h.cmake @@ -18,13 +18,13 @@ #cmakedefine USE_SANITIZER "@USE_SANITIZER@" #cmakedefine CMAKE_BUILD_TYPE "@CMAKE_BUILD_TYPE@" +#cmakedefine CMAKE_USE_PTHREADS_INIT 1 + #cmakedefine OQS_USE_OPENSSL 1 #cmakedefine OQS_USE_AES_OPENSSL 1 #cmakedefine OQS_USE_SHA2_OPENSSL 1 #cmakedefine OQS_USE_SHA3_OPENSSL 1 -#cmakedefine OQS_USE_PTHREADS_IN_TESTS 1 - #cmakedefine OQS_USE_ADX_INSTRUCTIONS 1 #cmakedefine OQS_USE_AES_INSTRUCTIONS 1 #cmakedefine OQS_USE_AVX_INSTRUCTIONS 1 diff --git a/tests/test_kem.c b/tests/test_kem.c index b11271509c..07ff68e79e 100644 --- a/tests/test_kem.c +++ b/tests/test_kem.c @@ -7,7 +7,7 @@ #include -#if OQS_USE_PTHREADS_IN_TESTS +#if CMAKE_USE_PTHREADS_INIT #include #endif @@ -197,7 +197,7 @@ static void TEST_KEM_randombytes(uint8_t *random_array, size_t bytes_to_read) { } #endif -#if OQS_USE_PTHREADS_IN_TESTS +#if CMAKE_USE_PTHREADS_INIT struct thread_data { char *alg_name; OQS_STATUS rc; @@ -245,7 +245,7 @@ int main(int argc, char **argv) { #endif OQS_STATUS rc; -#if OQS_USE_PTHREADS_IN_TESTS +#if CMAKE_USE_PTHREADS_INIT #define MAX_LEN_KEM_NAME_ 64 // don't run Classic McEliece in threads because of large stack usage char no_thread_kem_patterns[][MAX_LEN_KEM_NAME_] = {"Classic-McEliece", "HQC-256-"}; diff --git a/tests/test_sig.c b/tests/test_sig.c index 28cecc9275..831e54e272 100644 --- a/tests/test_sig.c +++ b/tests/test_sig.c @@ -10,7 +10,7 @@ #include -#if OQS_USE_PTHREADS_IN_TESTS +#if CMAKE_USE_PTHREADS_INIT #include #endif @@ -174,7 +174,7 @@ static void TEST_SIG_randombytes(uint8_t *random_array, size_t bytes_to_read) { } #endif -#if OQS_USE_PTHREADS_IN_TESTS +#if CMAKE_USE_PTHREADS_INIT struct thread_data { char *alg_name; OQS_STATUS rc; @@ -222,7 +222,7 @@ int main(int argc, char **argv) { #endif OQS_STATUS rc; -#if OQS_USE_PTHREADS_IN_TESTS +#if CMAKE_USE_PTHREADS_INIT #define MAX_LEN_SIG_NAME_ 64 pthread_t thread; struct thread_data td;