From 8922abcc79e5c7755c77ca8481d2dcccff37de72 Mon Sep 17 00:00:00 2001 From: Michal Demin Date: Tue, 19 Mar 2024 12:04:51 +0100 Subject: [PATCH 1/2] Remove warnings about unused function arguments --- src/common/bench_interrupt_latency_test.c | 2 ++ src/common/bench_malloc_free_test.c | 2 +- src/common/bench_message_queue_test.c | 1 + src/common/bench_mutex_lock_unlock_test.c | 2 ++ src/common/bench_sem_context_switch_test.c | 1 + src/common/bench_sem_signal_release_test.c | 1 + src/common/bench_thread_switch_yield_test.c | 1 + src/common/bench_thread_test.c | 1 + 8 files changed, 10 insertions(+), 1 deletion(-) diff --git a/src/common/bench_interrupt_latency_test.c b/src/common/bench_interrupt_latency_test.c index 64e3f75..ce70add 100644 --- a/src/common/bench_interrupt_latency_test.c +++ b/src/common/bench_interrupt_latency_test.c @@ -57,6 +57,7 @@ __weak void bench_exit_timer_isr(void) */ static void irq_latency_isr(void *arg) { + ARG_UNUSED(arg); bench_isr_cycles = bench_timer_cycles_get(); diff_cycles = bench_timer_cycles_diff(bench_trigger_cycles, @@ -126,6 +127,7 @@ static void bench_thread_low(void *args) */ void bench_interrupt_latency_test(void *arg) { + ARG_UNUSED(arg); uint32_t i; bench_stats_reset(&latency_times); diff --git a/src/common/bench_malloc_free_test.c b/src/common/bench_malloc_free_test.c index b2bcdec..e09fe23 100644 --- a/src/common/bench_malloc_free_test.c +++ b/src/common/bench_malloc_free_test.c @@ -82,4 +82,4 @@ int main(void) return 0; } -#endif \ No newline at end of file +#endif diff --git a/src/common/bench_message_queue_test.c b/src/common/bench_message_queue_test.c index 3ad9ed0..30726db 100644 --- a/src/common/bench_message_queue_test.c +++ b/src/common/bench_message_queue_test.c @@ -161,6 +161,7 @@ static void gather_receive_context_switch_stats(int priority, int iteration) */ void bench_message_queue_init(void *arg) { + ARG_UNUSED(arg); #if RTOS_HAS_MESSAGE_QUEUE int i; diff --git a/src/common/bench_mutex_lock_unlock_test.c b/src/common/bench_mutex_lock_unlock_test.c index 0a9e63d..b3f5208 100644 --- a/src/common/bench_mutex_lock_unlock_test.c +++ b/src/common/bench_mutex_lock_unlock_test.c @@ -255,6 +255,7 @@ static void bench_pend_low(void *args) */ static void bench_pend_high(void *args) { + ARG_UNUSED(args); /* Step 4 */ helper_start = bench_timing_counter_get(); @@ -373,6 +374,7 @@ static void gather_pend_inheritance_stats(int priority, uint32_t iteration) */ void bench_mutex_lock_unlock_test(void *arg) { + ARG_UNUSED(arg); uint32_t i; bench_mutex_create(MUTEX_ID); diff --git a/src/common/bench_sem_context_switch_test.c b/src/common/bench_sem_context_switch_test.c index 2a06e92..5f92d25 100644 --- a/src/common/bench_sem_context_switch_test.c +++ b/src/common/bench_sem_context_switch_test.c @@ -66,6 +66,7 @@ void bench_sem_context_switch_low_prio_give(int priority, int iteration) */ void bench_sem_context_switch_init(void *arg) { + ARG_UNUSED(arg); int i; bench_timing_init(); diff --git a/src/common/bench_sem_signal_release_test.c b/src/common/bench_sem_signal_release_test.c index 7374a85..ecef2af 100644 --- a/src/common/bench_sem_signal_release_test.c +++ b/src/common/bench_sem_signal_release_test.c @@ -64,6 +64,7 @@ void bench_sem_signal_release() */ void bench_sem_signal_release_init(void *param) { + ARG_UNUSED(param); bench_timing_init(); bench_sem_create(0, 0, ITERATIONS); diff --git a/src/common/bench_thread_switch_yield_test.c b/src/common/bench_thread_switch_yield_test.c index dee1844..15c6231 100644 --- a/src/common/bench_thread_switch_yield_test.c +++ b/src/common/bench_thread_switch_yield_test.c @@ -144,6 +144,7 @@ static void gather_set1_stats(int priority, uint32_t iteration) */ void bench_thread_yield(void *arg) { + ARG_UNUSED(arg); uint32_t i; bench_timing_init(); diff --git a/src/common/bench_thread_test.c b/src/common/bench_thread_test.c index 19c5614..0501f71 100644 --- a/src/common/bench_thread_test.c +++ b/src/common/bench_thread_test.c @@ -280,6 +280,7 @@ static void gather_set1_stats(int priority, uint32_t iteration) */ void bench_basic_thread_ops(void *arg) { + ARG_UNUSED(arg); uint32_t i; bench_timing_init(); From 0c37f4c78ff0fb50c1f721724a6ff77b36e21f51 Mon Sep 17 00:00:00 2001 From: Michal Demin Date: Tue, 19 Mar 2024 12:05:22 +0100 Subject: [PATCH 2/2] Fix yield test not cleaning threads --- src/common/bench_thread_switch_yield_test.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/common/bench_thread_switch_yield_test.c b/src/common/bench_thread_switch_yield_test.c index 15c6231..87a47c1 100644 --- a/src/common/bench_thread_switch_yield_test.c +++ b/src/common/bench_thread_switch_yield_test.c @@ -165,6 +165,7 @@ void bench_thread_yield(void *arg) for (i = 1; i <= ITERATIONS; i++) { gather_set1_stats(MAIN_PRIORITY, i); + bench_collect_resources(); } bench_stats_report_line("Yield (no context switch)", &time_to_yield); @@ -173,6 +174,7 @@ void bench_thread_yield(void *arg) for (i = 1; i < ITERATIONS; i++) { gather_set2_stats(MAIN_PRIORITY, i); + bench_collect_resources(); } bench_timing_stop();