summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2014-03-18 11:49:38 +0100
committerSebastian Huber <sebastian.huber@embedded-brains.de>2014-03-19 08:34:26 +0100
commitff25926701a7c4e8cf0858bc8f41133e7d7dd45a (patch)
treec3ca823580022af719f01fc554407cfbda9dd3f2
parentposix: Use interal mutex for once implementation (diff)
downloadrtems-ff25926701a7c4e8cf0858bc8f41133e7d7dd45a.tar.bz2
score: Delete _Assert_Thread_dispatching_repressed
Add _Debug_Is_thread_dispatching_allowed(). This makes it possible to assert the opposite. Use _ISR_Disable_without_giant()/_ISR_Enable_without_giant() to avoid misleading secondary assertion failures.
-rw-r--r--cpukit/score/Makefile.am2
-rw-r--r--cpukit/score/include/rtems/score/assert.h8
-rw-r--r--cpukit/score/src/debugisthreaddispatchingallowed.c (renamed from cpukit/score/src/assertthreaddispatchingrepressed.c)14
-rw-r--r--cpukit/score/src/smp.c2
4 files changed, 12 insertions, 14 deletions
diff --git a/cpukit/score/Makefile.am b/cpukit/score/Makefile.am
index d747c91094..7c87d1a761 100644
--- a/cpukit/score/Makefile.am
+++ b/cpukit/score/Makefile.am
@@ -331,7 +331,7 @@ libscore_a_SOURCES += src/apiext.c src/chain.c src/chainappend.c \
src/chainextract.c src/chainget.c src/chaininsert.c \
src/chainappendempty.c src/chainprependempty.c src/chaingetempty.c \
src/chainnodecount.c \
- src/assertthreaddispatchingrepressed.c \
+ src/debugisthreaddispatchingallowed.c \
src/interr.c src/isr.c src/wkspace.c src/wkstringduplicate.c
libscore_a_SOURCES += src/profilingisrentryexit.c
diff --git a/cpukit/score/include/rtems/score/assert.h b/cpukit/score/include/rtems/score/assert.h
index 4856eae8b4..0248b3c4e5 100644
--- a/cpukit/score/include/rtems/score/assert.h
+++ b/cpukit/score/include/rtems/score/assert.h
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2013 embedded brains GmbH. All rights reserved.
+ * Copyright (c) 2013-2014 embedded brains GmbH. All rights reserved.
*
* embedded brains GmbH
* Dornierstr. 4
@@ -39,15 +39,13 @@ extern "C" {
#endif
/**
- * @brief Asserts that thread dispatching is repressed.
+ * @brief Returns true if thread dispatching is allowed.
*
* Thread dispatching can be repressed via _Thread_Disable_dispatch() or
* _ISR_Disable().
*/
#if defined( RTEMS_DEBUG )
- void _Assert_Thread_dispatching_repressed( void );
-#else
- #define _Assert_Thread_dispatching_repressed() ( ( void ) 0 )
+ bool _Debug_Is_thread_dispatching_allowed( void );
#endif
/**
diff --git a/cpukit/score/src/assertthreaddispatchingrepressed.c b/cpukit/score/src/debugisthreaddispatchingallowed.c
index 0d586f8c74..fa5b502471 100644
--- a/cpukit/score/src/assertthreaddispatchingrepressed.c
+++ b/cpukit/score/src/debugisthreaddispatchingallowed.c
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2013 embedded brains GmbH. All rights reserved.
+ * Copyright (c) 2013-2014 embedded brains GmbH. All rights reserved.
*
* embedded brains GmbH
* Dornierstr. 4
@@ -21,17 +21,17 @@
#include <rtems/score/threaddispatch.h>
#if defined( RTEMS_DEBUG )
- void _Assert_Thread_dispatching_repressed( void )
+ bool _Debug_Is_thread_dispatching_allowed( void )
{
- bool dispatch_is_disabled;
+ bool dispatch_allowed;
ISR_Level level;
Per_CPU_Control *per_cpu;
- _ISR_Disable( level );
+ _ISR_Disable_without_giant( level );
per_cpu = _Per_CPU_Get_by_index( _SMP_Get_current_processor() );
- dispatch_is_disabled = per_cpu->thread_dispatch_disable_level != 0;
- _ISR_Enable( level );
+ dispatch_allowed = per_cpu->thread_dispatch_disable_level == 0;
+ _ISR_Enable_without_giant( level );
- _Assert( dispatch_is_disabled || _ISR_Get_level() != 0 );
+ return dispatch_allowed && _ISR_Get_level() == 0;
}
#endif
diff --git a/cpukit/score/src/smp.c b/cpukit/score/src/smp.c
index 389a4d6c99..8e57ed003f 100644
--- a/cpukit/score/src/smp.c
+++ b/cpukit/score/src/smp.c
@@ -100,7 +100,7 @@ void _SMP_Broadcast_message( uint32_t message )
uint32_t ncpus = _SMP_Get_processor_count();
uint32_t cpu;
- _Assert_Thread_dispatching_repressed();
+ _Assert( _Debug_Is_thread_dispatching_allowed() );
for ( cpu = 0 ; cpu < ncpus ; ++cpu ) {
if ( cpu != self ) {