ARMmbed
/
mbed-os
mirror of https://github.com/ARMmbed/mbed-os.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Critical tests: use atomics 

Get rid of a volatile, and use atomics to synchronise with the interrupt
routine instead.

Useful as a non-RTOS basic compilation check for the atomics - the
fuller atomic test relies on the RTOS.
pull/10147/head
Kevin Bracey 2019-03-27 16:14:27 +02:00
parent 0fabf37564
commit 4fe06e0949
1 changed files with 16 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
TESTS/mbed_platform/critical_section/main.cpp
View File

@ -22,11 +22,11 @@
using utest::v1::Case;
volatile bool callback_called;
bool callback_called;
void tiemout_callback(void)
{
callback_called = true;
core_util_atomic_store(&callback_called, true);
}
template<int N>
@ -49,7 +49,7 @@ void critical_section_raii_recursive(Timeout &timeout)
wait_us(wait_time_us);
}
TEST_ASSERT_TRUE(core_util_in_critical_section());
TEST_ASSERT_FALSE(callback_called);
TEST_ASSERT_FALSE(core_util_atomic_load(&callback_called));
}
@ -82,31 +82,30 @@ void test_C_API(void)
TEST_ASSERT_FALSE(core_util_in_critical_section());
callback_called = false;
core_util_atomic_store(&callback_called, false);
timeout.attach_us(callback(tiemout_callback), timeout_time_us);
wait_us(wait_time_us);
TEST_ASSERT_TRUE(callback_called);
TEST_ASSERT_TRUE(core_util_atomic_exchange(&callback_called, false));
for (int i = 0; i < N; i++) {
core_util_critical_section_enter();
TEST_ASSERT_TRUE(core_util_in_critical_section());
}
callback_called = false;
timeout.attach_us(callback(tiemout_callback), timeout_time_us);
wait_us(wait_time_us);
TEST_ASSERT_FALSE(callback_called);
TEST_ASSERT_FALSE(core_util_atomic_load(&callback_called));
TEST_ASSERT_TRUE(core_util_in_critical_section());
for (int i = 0; i < N - 1; i++) {
core_util_critical_section_exit();
TEST_ASSERT_TRUE(core_util_in_critical_section());
TEST_ASSERT_FALSE(callback_called);
TEST_ASSERT_FALSE(core_util_atomic_load(&callback_called));
}
core_util_critical_section_exit();
TEST_ASSERT_FALSE(core_util_in_critical_section());
TEST_ASSERT_TRUE(callback_called);
TEST_ASSERT_TRUE(core_util_atomic_load(&callback_called));
}
/** Template for tests
@ -138,16 +137,15 @@ void test_CPP_API_constructor_destructor(void)
TEST_ASSERT_FALSE(core_util_in_critical_section());
callback_called = false;
core_util_atomic_store(&callback_called, false);
timeout.attach_us(callback(tiemout_callback), timeout_time_us);
wait_us(wait_time_us);
TEST_ASSERT_TRUE(callback_called);
TEST_ASSERT_TRUE(core_util_atomic_exchange(&callback_called, false));
callback_called = false;
critical_section_raii_recursive<N>(timeout);
TEST_ASSERT_FALSE(core_util_in_critical_section());
TEST_ASSERT_TRUE(callback_called);
TEST_ASSERT_TRUE(core_util_atomic_load(&callback_called));
}
/** Template for tests
@ -179,31 +177,30 @@ void test_CPP_API_enable_disable(void)
TEST_ASSERT_FALSE(core_util_in_critical_section());
callback_called = false;
core_util_atomic_store(&callback_called, false);
timeout.attach_us(callback(tiemout_callback), timeout_time_us);
wait_us(wait_time_us);
TEST_ASSERT_TRUE(callback_called);
TEST_ASSERT_TRUE(core_util_atomic_exchange(&callback_called, false));
for (int i = 0; i < N; i++) {
CriticalSectionLock::enable();
TEST_ASSERT_TRUE(core_util_in_critical_section());
}
callback_called = false;
timeout.attach_us(callback(tiemout_callback), timeout_time_us);
wait_us(wait_time_us);
TEST_ASSERT_FALSE(callback_called);
TEST_ASSERT_FALSE(core_util_atomic_load(&callback_called));
TEST_ASSERT_TRUE(core_util_in_critical_section());
for (int i = 0; i < N - 1; i++) {
CriticalSectionLock::disable();
TEST_ASSERT_TRUE(core_util_in_critical_section());
TEST_ASSERT_FALSE(callback_called);
TEST_ASSERT_FALSE(core_util_atomic_load(&callback_called));
}
CriticalSectionLock::disable();
TEST_ASSERT_FALSE(core_util_in_critical_section());
TEST_ASSERT_TRUE(callback_called);
TEST_ASSERT_TRUE(core_util_atomic_load(&callback_called));
}