[rtems commit] libtest: Add T_get_one_clock_tick_busy()
Sebastian Huber
sebh at rtems.org
Thu Jul 23 08:57:41 UTC 2020
Module: rtems
Branch: master
Commit: af9266541fda63cb02cf54154d54e6b5504cef74
Changeset: http://git.rtems.org/rtems/commit/?id=af9266541fda63cb02cf54154d54e6b5504cef74
Author: Sebastian Huber <sebastian.huber at embedded-brains.de>
Date: Fri Jul 17 14:49:42 2020 +0200
libtest: Add T_get_one_clock_tick_busy()
Update #3199.
---
cpukit/Makefile.am | 1 +
cpukit/include/rtems/test-info.h | 6 -
cpukit/include/rtems/test.h | 2 +
cpukit/libtest/t-test-busy-tick.c | 121 +++++++++++++++++++++
cpukit/libtest/testbusy.c | 73 -------------
.../sptests/spintrcritical_support/intrcritical.c | 2 +-
6 files changed, 125 insertions(+), 80 deletions(-)
diff --git a/cpukit/Makefile.am b/cpukit/Makefile.am
index 7f89f6a..a119055 100644
--- a/cpukit/Makefile.am
+++ b/cpukit/Makefile.am
@@ -1858,6 +1858,7 @@ librtemstest_a_SOURCES += libtest/testparallel.c
librtemstest_a_SOURCES += libtest/testwrappers.c
librtemstest_a_SOURCES += libtest/t-test.c
librtemstest_a_SOURCES += libtest/t-test-busy.c
+librtemstest_a_SOURCES += libtest/t-test-busy-tick.c
librtemstest_a_SOURCES += libtest/t-test-checks.c
librtemstest_a_SOURCES += libtest/t-test-checks-eno.c
librtemstest_a_SOURCES += libtest/t-test-checks-psx.c
diff --git a/cpukit/include/rtems/test-info.h b/cpukit/include/rtems/test-info.h
index 5f73e92..42f5235 100644
--- a/cpukit/include/rtems/test-info.h
+++ b/cpukit/include/rtems/test-info.h
@@ -304,12 +304,6 @@ void rtems_test_parallel(
*/
void rtems_test_busy_cpu_usage(time_t seconds, long nanoseconds);
-/**
- * @brief Returns a count value for T_busy() which yields roughly a
- * duration of one clock tick.
- */
-uint_fast32_t rtems_test_get_one_tick_busy_count(void);
-
/** @} */
#ifdef __cplusplus
diff --git a/cpukit/include/rtems/test.h b/cpukit/include/rtems/test.h
index 4c14e8d..b68f303 100644
--- a/cpukit/include/rtems/test.h
+++ b/cpukit/include/rtems/test.h
@@ -2263,6 +2263,8 @@ void T_case_body_##name(void)
void T_busy(uint_fast32_t);
+uint_fast32_t T_get_one_clock_tick_busy(void);
+
void T_report_hash_sha256(T_event, const char *);
void T_check_heap(T_event, const char *);
diff --git a/cpukit/libtest/t-test-busy-tick.c b/cpukit/libtest/t-test-busy-tick.c
new file mode 100644
index 0000000..3613f8e
--- /dev/null
+++ b/cpukit/libtest/t-test-busy-tick.c
@@ -0,0 +1,121 @@
+/* SPDX-License-Identifier: BSD-2-Clause */
+
+/**
+ * @file
+ *
+ * @ingroup RTEMSTestFrameworkImpl
+ *
+ * @brief Implementation of T_get_one_clock_tick_busy().
+ */
+
+/*
+ * Copyright (C) 2014, 2020 embedded brains GmbH (http://www.embedded-brains.de)
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
+ * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <rtems/test.h>
+
+#include <rtems.h>
+
+static uint_fast32_t
+T_estimate_busy_loop_maximum(void)
+{
+ uint_fast32_t initial;
+ uint_fast32_t units;
+
+ initial = rtems_clock_get_ticks_since_boot();
+ units = 0;
+
+ while (initial == rtems_clock_get_ticks_since_boot()) {
+ ++units;
+ }
+
+ return units;
+}
+
+static uint_fast32_t
+T_wait_for_tick_change(void)
+{
+ uint_fast32_t initial;
+ uint_fast32_t now;
+
+ initial = rtems_clock_get_ticks_since_boot();
+
+ do {
+ now = rtems_clock_get_ticks_since_boot();
+ } while (now == initial);
+
+ return now;
+}
+
+uint_fast32_t
+T_get_one_clock_tick_busy(void)
+{
+ uint_fast32_t last;
+ uint_fast32_t now;
+ uint_fast32_t a;
+ uint_fast32_t b;
+ uint_fast32_t m;
+
+ /* Choose a lower bound */
+ a = 1;
+
+ /* Estimate an upper bound */
+
+ T_wait_for_tick_change();
+ b = 2 * T_estimate_busy_loop_maximum();
+
+ while (true) {
+ last = T_wait_for_tick_change();
+ T_busy(b);
+ now = rtems_clock_get_ticks_since_boot();
+
+ if (now != last) {
+ break;
+ }
+
+ b *= 2;
+ last = now;
+ }
+
+ /* Find a good value */
+ do {
+ m = (a + b) / 2;
+
+ last = T_wait_for_tick_change();
+ T_busy(m);
+ now = rtems_clock_get_ticks_since_boot();
+
+ if (now != last) {
+ b = m;
+ } else {
+ a = m;
+ }
+ } while (b - a > 1);
+
+ return m;
+}
diff --git a/cpukit/libtest/testbusy.c b/cpukit/libtest/testbusy.c
index bf11b11..c1d4427 100644
--- a/cpukit/libtest/testbusy.c
+++ b/cpukit/libtest/testbusy.c
@@ -18,81 +18,8 @@
#endif
#include <rtems/test-info.h>
-#include <rtems.h>
-#include <rtems/test.h>
#include <rtems/score/threadimpl.h>
-static uint_fast32_t estimate_busy_loop_maximum( void )
-{
- uint_fast32_t units = 0;
- uint_fast32_t initial = rtems_clock_get_ticks_since_boot();
-
- while ( initial == rtems_clock_get_ticks_since_boot() ) {
- ++units;
- }
-
- return units;
-}
-
-static uint_fast32_t wait_for_tick_change( void )
-{
- uint_fast32_t initial = rtems_clock_get_ticks_since_boot();
- uint_fast32_t now;
-
- do {
- now = rtems_clock_get_ticks_since_boot();
- } while ( now == initial );
-
- return now;
-}
-
-uint_fast32_t rtems_test_get_one_tick_busy_count( void )
-{
- uint_fast32_t last;
- uint_fast32_t now;
- uint_fast32_t a;
- uint_fast32_t b;
- uint_fast32_t m;
-
- /* Choose a lower bound */
- a = 1;
-
- /* Estimate an upper bound */
-
- wait_for_tick_change();
- b = 2 * estimate_busy_loop_maximum();
-
- while ( true ) {
- last = wait_for_tick_change();
- T_busy( b );
- now = rtems_clock_get_ticks_since_boot();
-
- if ( now != last ) {
- break;
- }
-
- b *= 2;
- last = now;
- }
-
- /* Find a good value */
- do {
- m = ( a + b ) / 2;
-
- last = wait_for_tick_change();
- T_busy( m );
- now = rtems_clock_get_ticks_since_boot();
-
- if ( now != last ) {
- b = m;
- } else {
- a = m;
- }
- } while ( b - a > 1 );
-
- return m;
-}
-
void rtems_test_busy_cpu_usage( time_t seconds, long nanoseconds )
{
Thread_Control *executing;
diff --git a/testsuites/sptests/spintrcritical_support/intrcritical.c b/testsuites/sptests/spintrcritical_support/intrcritical.c
index 9bce230..b6c6dad 100644
--- a/testsuites/sptests/spintrcritical_support/intrcritical.c
+++ b/testsuites/sptests/spintrcritical_support/intrcritical.c
@@ -71,7 +71,7 @@ void interrupt_critical_section_test_support_initialize(
rtems_test_assert( sc == RTEMS_SUCCESSFUL );
}
- m = rtems_test_get_one_tick_busy_count();
+ m = T_get_one_clock_tick_busy();
interrupt_critical.minimum = 0;
interrupt_critical.maximum = m;
More information about the vc
mailing list