upstream/bind9
1
0
Fork 0
mirror of https://github.com/isc-projects/bind9.git synced 2026-05-28 04:34:54 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

ISC_RUN_TEST_IMPL should use a static declaration

Browse source 
These functions don't need to be called from multiple places and
by making them static we will detect when they are not added to the
list functions to be tested.

(cherry picked from commit 22d664aa15)
This commit is contained in:
Mark Andrews 2026-01-23 15:57:42 +11:00
parent 47b8ca5ac7
commit 84f955a972
4 changed files with 18 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
tests/include/tests/isc.h
View file

@ -94,7 +94,7 @@ teardown_managers(void **state);
int setup_test_##name(void **state ISC_ATTR_UNUSED);
#define ISC_RUN_TEST_DECLARE(name) \
void run_test_##name(void **state ISC_ATTR_UNUSED);
static void run_test_##name(void **state ISC_ATTR_UNUSED);
#define ISC_TEARDOWN_TEST_DECLARE(name) \
int teardown_test_##name(void **state ISC_ATTR_UNUSED)
@ -123,9 +123,9 @@ teardown_managers(void **state);
int setup_test_##name(void **state ISC_ATTR_UNUSED); \
int setup_test_##name(void **state ISC_ATTR_UNUSED)
#define ISC_RUN_TEST_IMPL(name) \
void run_test_##name(void **state ISC_ATTR_UNUSED); \
void run_test_##name(void **state ISC_ATTR_UNUSED)
#define ISC_RUN_TEST_IMPL(name) \
static void run_test_##name(void **state ISC_ATTR_UNUSED); \
static void run_test_##name(void **state ISC_ATTR_UNUSED)
#define ISC_TEARDOWN_TEST_IMPL(name) \
int teardown_test_##name(void **state ISC_ATTR_UNUSED); \
@ -137,9 +137,9 @@ teardown_managers(void **state);
;
#define ISC_LOOP_TEST_CUSTOM_IMPL(name, setup, teardown) \
void run_test_##name(void **state ISC_ATTR_UNUSED); \
void loop_test_##name(void *arg ISC_ATTR_UNUSED); \
void run_test_##name(void **state ISC_ATTR_UNUSED) { \
static void run_test_##name(void **state ISC_ATTR_UNUSED); \
static void loop_test_##name(void *arg ISC_ATTR_UNUSED); \
static void run_test_##name(void **state ISC_ATTR_UNUSED) { \
isc_job_cb setup_loop = setup; \
isc_job_cb teardown_loop = teardown; \
if (setup_loop != NULL) { \
@ -151,7 +151,7 @@ teardown_managers(void **state);
isc_loop_setup(mainloop, loop_test_##name, state); \
isc_loopmgr_run(loopmgr); \
} \
void loop_test_##name(void *arg ISC_ATTR_UNUSED)
static void loop_test_##name(void *arg ISC_ATTR_UNUSED)
#define ISC_LOOP_TEST_IMPL(name) ISC_LOOP_TEST_CUSTOM_IMPL(name, NULL, NULL)

6
tests/isc/mutex_test.c
View file

@ -77,10 +77,11 @@ ISC_RUN_TEST_IMPL(isc_mutex) {
#define CNT_MIN 800
#define CNT_MAX 1600
static size_t shared_counter = 0;
static size_t expected_counter = SIZE_MAX;
#if !defined(__SANITIZE_THREAD__)
static isc_mutex_t lock;
static pthread_mutex_t mutex;
static size_t expected_counter = SIZE_MAX;
static size_t shared_counter = 0;
static void *
pthread_mutex_thread(void *arg) {
@ -203,6 +204,7 @@ ISC_RUN_TEST_IMPL(isc_mutex_benchmark) {
isc_mem_cput(mctx, threads, workers, sizeof(*threads));
}
#endif
ISC_TEST_LIST_START

4
tests/isc/rwlock_test.c
View file

@ -52,9 +52,11 @@ static isc_barrier_t barrier2;
#define CNT_MIN 800
#define CNT_MAX 1600
#if !defined(__SANITIZE_THREAD__)
static size_t shared_counter = 0;
static size_t expected_counter = SIZE_MAX;
static uint8_t boundary = 0;
#endif
static uint8_t *rnd;
static int
@ -133,6 +135,7 @@ ISC_RUN_TEST_IMPL(isc_rwlock_wrlock) {
/*
* Simple single-threaded lock/tryupgrade/unlock test
*/
#if !defined(__SANITIZE_THREAD__)
ISC_RUN_TEST_IMPL(isc_rwlock_tryupgrade) {
isc_result_t result;
isc_rwlock_lock(&rwlock, isc_rwlocktype_read);
@ -341,6 +344,7 @@ ISC_RUN_TEST_IMPL(isc_rwlock_benchmark) {
isc_mem_cput(mctx, threads, workers, sizeof(*threads));
}
#endif
ISC_TEST_LIST_START

2
tests/isc/spinlock_test.c
View file

@ -82,6 +82,7 @@ ISC_RUN_TEST_IMPL(isc_spinlock) {
#define CNT_MIN 800
#define CNT_MAX 1600
#if !defined(__SANITIZE_THREAD__)
static size_t shared_counter = 0;
static size_t expected_counter = SIZE_MAX;
@ -213,6 +214,7 @@ ISC_RUN_TEST_IMPL(isc_spinlock_benchmark) {
isc_mem_cput(mctx, threads, workers, sizeof(*threads));
}
#endif
ISC_TEST_LIST_START