[rtems commit] score: Hide SMP lock profiling impl if disabled
Sebastian Huber
sebh at rtems.org
Fri Jun 26 09:23:24 UTC 2015
Module: rtems
Branch: master
Commit: d811daca691e1fe9e8166fe20118148442f266a1
Changeset: http://git.rtems.org/rtems/commit/?id=d811daca691e1fe9e8166fe20118148442f266a1
Author: Sebastian Huber <sebastian.huber at embedded-brains.de>
Date: Thu Jun 25 11:24:44 2015 +0200
score: Hide SMP lock profiling impl if disabled
The problem is that empty structures have a different size in C and C++.
---
cpukit/score/include/rtems/score/percpu.h | 17 ++-
cpukit/score/include/rtems/score/smplock.h | 175 ++++++++++++++++++-----------
cpukit/score/src/profilingsmplock.c | 18 +--
cpukit/score/src/smp.c | 3 +-
testsuites/sptests/sp37/init.c | 4 +-
5 files changed, 133 insertions(+), 84 deletions(-)
diff --git a/cpukit/score/include/rtems/score/percpu.h b/cpukit/score/include/rtems/score/percpu.h
index a6f7a25..088c31f 100644
--- a/cpukit/score/include/rtems/score/percpu.h
+++ b/cpukit/score/include/rtems/score/percpu.h
@@ -319,10 +319,17 @@ typedef struct Per_CPU_Control {
*/
SMP_ticket_lock_Control Lock;
- /**
- * @brief Lock statistics context for the per-CPU lock.
- */
- SMP_lock_Stats_context Lock_stats_context;
+ #if defined( RTEMS_PROFILING )
+ /**
+ * @brief Lock statistics for the per-CPU lock.
+ */
+ SMP_lock_Stats Lock_stats;
+
+ /**
+ * @brief Lock statistics context for the per-CPU lock.
+ */
+ SMP_lock_Stats_context Lock_stats_context;
+ #endif
/**
* @brief Context for the Giant lock acquire and release pair of this
@@ -385,6 +392,7 @@ extern Per_CPU_Control_envelope _Per_CPU_Information[] CPU_STRUCTURE_ALIGNMENT;
#define _Per_CPU_Acquire( cpu ) \
_SMP_ticket_lock_Acquire( \
&( cpu )->Lock, \
+ &( cpu )->Lock_stats, \
&( cpu )->Lock_stats_context \
)
#else
@@ -398,6 +406,7 @@ extern Per_CPU_Control_envelope _Per_CPU_Information[] CPU_STRUCTURE_ALIGNMENT;
#define _Per_CPU_Release( cpu ) \
_SMP_ticket_lock_Release( \
&( cpu )->Lock, \
+ &( cpu )->Lock_stats, \
&( cpu )->Lock_stats_context \
)
#else
diff --git a/cpukit/score/include/rtems/score/smplock.h b/cpukit/score/include/rtems/score/smplock.h
index 50a0662..da510d1 100644
--- a/cpukit/score/include/rtems/score/smplock.h
+++ b/cpukit/score/include/rtems/score/smplock.h
@@ -58,6 +58,8 @@ extern "C" {
* @{
*/
+#if defined( RTEMS_PROFILING )
+
/**
* @brief Count of lock contention counters for lock statistics.
*/
@@ -82,7 +84,6 @@ extern "C" {
* instant and the lock acquire instant.
*/
typedef struct {
-#if defined( RTEMS_PROFILING )
/**
* @brief Node for SMP lock statistics chain.
*/
@@ -142,33 +143,25 @@ typedef struct {
* @brief The lock name.
*/
const char *name;
-#endif /* defined( RTEMS_PROFILING ) */
} SMP_lock_Stats;
/**
* @brief Local context for SMP lock statistics.
*/
typedef struct {
-#if defined( RTEMS_PROFILING )
/**
* @brief The last lock acquire instant in CPU counter ticks.
*
* This value is used to measure the lock section time.
*/
CPU_Counter_ticks acquire_instant;
-#endif
} SMP_lock_Stats_context;
/**
* @brief SMP lock statistics initializer for static initialization.
*/
-#if defined( RTEMS_PROFILING )
#define SMP_LOCK_STATS_INITIALIZER( name ) \
{ { NULL, NULL }, 0, 0, 0, 0, { 0, 0, 0, 0 }, 0, name }
-#else
-#define SMP_LOCK_STATS_INITIALIZER( name ) \
- { }
-#endif
/**
* @brief Initializes an SMP lock statistics block.
@@ -190,14 +183,14 @@ static inline void _SMP_lock_Stats_initialize(
/**
* @brief Destroys an SMP lock statistics block.
*
- * @param[in,out] stats The SMP lock statistics block.
+ * @param[in] stats The SMP lock statistics block.
*/
static inline void _SMP_lock_Stats_destroy( SMP_lock_Stats *stats );
/**
* @brief Destroys an SMP lock statistics block.
*
- * @param[in,out] stats The SMP lock statistics block.
+ * @param[in] stats The SMP lock statistics block.
* @param[in] stats_context The SMP lock statistics context.
*/
static inline void _SMP_lock_Stats_release_update(
@@ -205,23 +198,29 @@ static inline void _SMP_lock_Stats_release_update(
const SMP_lock_Stats_context *stats_context
);
+#else /* RTEMS_PROFILING */
+
+#define _SMP_lock_Stats_initialize( stats, name ) do { } while ( 0 )
+
+#define _SMP_lock_Stats_destroy( stats ) do { } while ( 0 )
+
+#endif /* RTEMS_PROFILING */
+
/**
* @brief SMP ticket lock control.
*/
typedef struct {
Atomic_Uint next_ticket;
Atomic_Uint now_serving;
- SMP_lock_Stats Stats;
} SMP_ticket_lock_Control;
/**
* @brief SMP ticket lock control initializer for static initialization.
*/
-#define SMP_TICKET_LOCK_INITIALIZER( name ) \
+#define SMP_TICKET_LOCK_INITIALIZER \
{ \
ATOMIC_INITIALIZER_UINT( 0U ), \
- ATOMIC_INITIALIZER_UINT( 0U ), \
- SMP_LOCK_STATS_INITIALIZER( name ) \
+ ATOMIC_INITIALIZER_UINT( 0U ) \
}
/**
@@ -229,18 +228,16 @@ typedef struct {
*
* Concurrent initialization leads to unpredictable results.
*
- * @param[in,out] lock The SMP ticket lock control.
+ * @param[in] lock The SMP ticket lock control.
* @param[in] name The name for the SMP ticket lock. This name must be
* persistent throughout the life time of this lock.
*/
static inline void _SMP_ticket_lock_Initialize(
- SMP_ticket_lock_Control *lock,
- const char *name
+ SMP_ticket_lock_Control *lock
)
{
_Atomic_Init_uint( &lock->next_ticket, 0U );
_Atomic_Init_uint( &lock->now_serving, 0U );
- _SMP_lock_Stats_initialize( &lock->Stats, name );
}
/**
@@ -248,33 +245,26 @@ static inline void _SMP_ticket_lock_Initialize(
*
* Concurrent destruction leads to unpredictable results.
*
- * @param[in,out] lock The SMP ticket lock control.
+ * @param[in] lock The SMP ticket lock control.
*/
static inline void _SMP_ticket_lock_Destroy( SMP_ticket_lock_Control *lock )
{
- _SMP_lock_Stats_destroy( &lock->Stats );
+ (void) lock;
}
-/**
- * @brief Acquires an SMP ticket lock.
- *
- * This function will not disable interrupts. The caller must ensure that the
- * current thread of execution is not interrupted indefinite once it obtained
- * the SMP ticket lock.
- *
- * @param[in,out] lock The SMP ticket lock control.
- * @param[out] stats_context The SMP lock statistics context.
- */
-static inline void _SMP_ticket_lock_Acquire(
- SMP_ticket_lock_Control *lock,
+static inline void _SMP_ticket_lock_Do_acquire(
+ SMP_ticket_lock_Control *lock
+#if defined( RTEMS_PROFILING )
+ ,
+ SMP_lock_Stats *stats,
SMP_lock_Stats_context *stats_context
+#endif
)
{
unsigned int my_ticket;
unsigned int now_serving;
#if defined( RTEMS_PROFILING )
- SMP_lock_Stats *stats = &lock->Stats;
CPU_Counter_ticks first;
CPU_Counter_ticks second;
CPU_Counter_ticks delta;
@@ -318,36 +308,71 @@ static inline void _SMP_ticket_lock_Acquire(
initial_queue_length = SMP_LOCK_STATS_CONTENTION_COUNTS - 1;
}
++stats->contention_counts[initial_queue_length];
-#else
- (void) stats_context;
#endif
}
/**
- * @brief Releases an SMP ticket lock.
+ * @brief Acquires an SMP ticket lock.
*
- * @param[in,out] lock The SMP ticket lock control.
- * @param[in] stats_context The SMP lock statistics context.
+ * This function will not disable interrupts. The caller must ensure that the
+ * current thread of execution is not interrupted indefinite once it obtained
+ * the SMP ticket lock.
+ *
+ * @param[in] lock The SMP ticket lock control.
+ * @param[in] stats The SMP lock statistics.
+ * @param[out] stats_context The SMP lock statistics context.
*/
-static inline void _SMP_ticket_lock_Release(
- SMP_ticket_lock_Control *lock,
+#if defined( RTEMS_PROFILING )
+ #define _SMP_ticket_lock_Acquire( lock, stats, stats_context ) \
+ _SMP_ticket_lock_Do_acquire( lock, stats, stats_context )
+#else
+ #define _SMP_ticket_lock_Acquire( lock, stats, stats_context ) \
+ _SMP_ticket_lock_Do_acquire( lock )
+#endif
+
+static inline void _SMP_ticket_lock_Do_release(
+ SMP_ticket_lock_Control *lock
+#if defined( RTEMS_PROFILING )
+ ,
+ SMP_lock_Stats *stats,
const SMP_lock_Stats_context *stats_context
+#endif
)
{
unsigned int current_ticket =
_Atomic_Load_uint( &lock->now_serving, ATOMIC_ORDER_RELAXED );
unsigned int next_ticket = current_ticket + 1U;
- _SMP_lock_Stats_release_update( &lock->Stats, stats_context );
+#if defined( RTEMS_PROFILING )
+ _SMP_lock_Stats_release_update( stats, stats_context );
+#endif
_Atomic_Store_uint( &lock->now_serving, next_ticket, ATOMIC_ORDER_RELEASE );
}
/**
+ * @brief Releases an SMP ticket lock.
+ *
+ * @param[in] lock The SMP ticket lock control.
+ * @param[in] stats The SMP lock statistics.
+ * @param[in] stats_context The SMP lock statistics context.
+ */
+#if defined( RTEMS_PROFILING )
+ #define _SMP_ticket_lock_Release( lock, stats, stats_context ) \
+ _SMP_ticket_lock_Do_release( lock, stats, stats_context )
+#else
+ #define _SMP_ticket_lock_Release( lock, stats, stats_context ) \
+ _SMP_ticket_lock_Do_release( lock )
+#endif
+
+/**
* @brief SMP lock control.
*/
typedef struct {
- SMP_ticket_lock_Control ticket_lock;
+ SMP_ticket_lock_Control Ticket_lock;
+#if defined( RTEMS_PROFILING )
+ SMP_lock_Stats Stats;
+#endif
} SMP_lock_Control;
/**
@@ -355,20 +380,27 @@ typedef struct {
*/
typedef struct {
ISR_Level isr_level;
+#if defined( RTEMS_PROFILING )
SMP_lock_Stats_context Stats_context;
+#endif
} SMP_lock_Context;
/**
* @brief SMP lock control initializer for static initialization.
*/
-#define SMP_LOCK_INITIALIZER( name ) { SMP_TICKET_LOCK_INITIALIZER( name ) }
+#if defined( RTEMS_PROFILING )
+ #define SMP_LOCK_INITIALIZER( name ) \
+ { SMP_TICKET_LOCK_INITIALIZER, SMP_LOCK_STATS_INITIALIZER( name ) }
+#else
+ #define SMP_LOCK_INITIALIZER( name ) { SMP_TICKET_LOCK_INITIALIZER }
+#endif
/**
* @brief Initializes an SMP lock.
*
* Concurrent initialization leads to unpredictable results.
*
- * @param[in,out] lock The SMP lock control.
+ * @param[in] lock The SMP lock control.
* @param[in] name The name for the SMP lock statistics. This name must be
* persistent throughout the life time of this statistics block.
*/
@@ -386,7 +418,12 @@ static inline void _SMP_lock_Initialize(
const char *name
)
{
- _SMP_ticket_lock_Initialize( &lock->ticket_lock, name );
+ _SMP_ticket_lock_Initialize( &lock->Ticket_lock );
+#if defined( RTEMS_PROFILING )
+ _SMP_lock_Stats_initialize( &lock->Stats, name );
+#else
+ (void) name;
+#endif
}
/**
@@ -394,7 +431,7 @@ static inline void _SMP_lock_Initialize(
*
* Concurrent destruction leads to unpredictable results.
*
- * @param[in,out] lock The SMP lock control.
+ * @param[in] lock The SMP lock control.
*/
#if defined( RTEMS_SMP_LOCK_DO_NOT_INLINE )
void _SMP_lock_Destroy( SMP_lock_Control *lock );
@@ -404,7 +441,8 @@ static inline void _SMP_lock_Destroy_body( SMP_lock_Control *lock )
static inline void _SMP_lock_Destroy( SMP_lock_Control *lock )
#endif
{
- _SMP_ticket_lock_Destroy( &lock->ticket_lock );
+ _SMP_ticket_lock_Destroy( &lock->Ticket_lock );
+ _SMP_lock_Stats_destroy( &lock->Stats );
}
/**
@@ -414,8 +452,8 @@ static inline void _SMP_lock_Destroy( SMP_lock_Control *lock )
* current thread of execution is not interrupted indefinite once it obtained
* the SMP lock.
*
- * @param[in,out] lock The SMP lock control.
- * @param[in,out] context The local SMP lock context for an acquire and release
+ * @param[in] lock The SMP lock control.
+ * @param[in] context The local SMP lock context for an acquire and release
* pair.
*/
#if defined( RTEMS_SMP_LOCK_DO_NOT_INLINE )
@@ -433,14 +471,18 @@ static inline void _SMP_lock_Acquire(
)
{
(void) context;
- _SMP_ticket_lock_Acquire( &lock->ticket_lock, &context->Stats_context );
+ _SMP_ticket_lock_Acquire(
+ &lock->Ticket_lock,
+ &lock->Stats,
+ &context->Stats_context
+ );
}
/**
* @brief Releases an SMP lock.
*
- * @param[in,out] lock The SMP lock control.
- * @param[in,out] context The local SMP lock context for an acquire and release
+ * @param[in] lock The SMP lock control.
+ * @param[in] context The local SMP lock context for an acquire and release
* pair.
*/
#if defined( RTEMS_SMP_LOCK_DO_NOT_INLINE )
@@ -458,14 +500,18 @@ static inline void _SMP_lock_Release(
)
{
(void) context;
- _SMP_ticket_lock_Release( &lock->ticket_lock, &context->Stats_context );
+ _SMP_ticket_lock_Release(
+ &lock->Ticket_lock,
+ &lock->Stats,
+ &context->Stats_context
+ );
}
/**
* @brief Disables interrupts and acquires the SMP lock.
*
- * @param[in,out] lock The SMP lock control.
- * @param[in,out] context The local SMP lock context for an acquire and release
+ * @param[in] lock The SMP lock control.
+ * @param[in] context The local SMP lock context for an acquire and release
* pair.
*/
#if defined( RTEMS_SMP_LOCK_DO_NOT_INLINE )
@@ -489,8 +535,8 @@ static inline void _SMP_lock_ISR_disable_and_acquire(
/**
* @brief Releases the SMP lock and enables interrupts.
*
- * @param[in,out] lock The SMP lock control.
- * @param[in,out] context The local SMP lock context for an acquire and release
+ * @param[in] lock The SMP lock control.
+ * @param[in] context The local SMP lock context for an acquire and release
* pair.
*/
#if defined( RTEMS_SMP_LOCK_DO_NOT_INLINE )
@@ -512,6 +558,7 @@ static inline void _SMP_lock_Release_and_ISR_enable(
}
#if defined( RTEMS_PROFILING )
+
typedef struct {
SMP_lock_Control Lock;
Chain_Control Stats_chain;
@@ -596,11 +643,9 @@ static inline void _SMP_lock_Stats_iteration_stop(
_Chain_Extract_unprotected( &iteration_context->Node );
_SMP_lock_Release_and_ISR_enable( &control->Lock, &lock_context );
}
-#endif
static inline void _SMP_lock_Stats_destroy( SMP_lock_Stats *stats )
{
-#if defined( RTEMS_PROFILING )
if ( !_Chain_Is_node_off_chain( &stats->Node ) ) {
SMP_lock_Stats_control *control = &_SMP_lock_Stats_control;
SMP_lock_Context lock_context;
@@ -629,9 +674,6 @@ static inline void _SMP_lock_Stats_destroy( SMP_lock_Stats *stats )
_SMP_lock_Release_and_ISR_enable( &control->Lock, &lock_context );
}
-#else
- (void) stats;
-#endif
}
static inline void _SMP_lock_Stats_release_update(
@@ -639,7 +681,6 @@ static inline void _SMP_lock_Stats_release_update(
const SMP_lock_Stats_context *stats_context
)
{
-#if defined( RTEMS_PROFILING )
CPU_Counter_ticks first = stats_context->acquire_instant;
CPU_Counter_ticks second = _CPU_Counter_read();
CPU_Counter_ticks delta = _CPU_Counter_difference( second, first );
@@ -658,12 +699,10 @@ static inline void _SMP_lock_Stats_release_update(
_SMP_lock_Release_and_ISR_enable( &control->Lock, &lock_context );
}
}
-#else
- (void) stats;
- (void) stats_context;
-#endif
}
+#endif /* RTEMS_PROFILING */
+
/**@}*/
#ifdef __cplusplus
diff --git a/cpukit/score/src/profilingsmplock.c b/cpukit/score/src/profilingsmplock.c
index be60ba9..a77e1a1 100644
--- a/cpukit/score/src/profilingsmplock.c
+++ b/cpukit/score/src/profilingsmplock.c
@@ -21,19 +21,19 @@
#if defined( RTEMS_PROFILING )
SMP_lock_Stats_control _SMP_lock_Stats_control = {
.Lock = {
- .ticket_lock = {
+ .Ticket_lock = {
.next_ticket = ATOMIC_INITIALIZER_UINT( 0U ),
- .now_serving = ATOMIC_INITIALIZER_UINT( 0U ),
- .Stats = {
- .Node = CHAIN_NODE_INITIALIZER_ONE_NODE_CHAIN(
- &_SMP_lock_Stats_control.Stats_chain
- ),
- .name = "SMP lock stats"
- }
+ .now_serving = ATOMIC_INITIALIZER_UINT( 0U )
+ },
+ .Stats = {
+ .Node = CHAIN_NODE_INITIALIZER_ONE_NODE_CHAIN(
+ &_SMP_lock_Stats_control.Stats_chain
+ ),
+ .name = "SMP Lock Stats"
}
},
.Stats_chain = CHAIN_INITIALIZER_ONE_NODE(
- &_SMP_lock_Stats_control.Lock.ticket_lock.Stats.Node
+ &_SMP_lock_Stats_control.Lock.Stats.Node
),
.Iterator_chain = CHAIN_INITIALIZER_EMPTY(
_SMP_lock_Stats_control.Iterator_chain
diff --git a/cpukit/score/src/smp.c b/cpukit/score/src/smp.c
index 8ffeb1d..a64287e 100644
--- a/cpukit/score/src/smp.c
+++ b/cpukit/score/src/smp.c
@@ -80,7 +80,8 @@ void _SMP_Handler_initialize( void )
for ( cpu_index = 0 ; cpu_index < cpu_max; ++cpu_index ) {
Per_CPU_Control *cpu = _Per_CPU_Get_by_index( cpu_index );
- _SMP_ticket_lock_Initialize( &cpu->Lock, "per-CPU" );
+ _SMP_ticket_lock_Initialize( &cpu->Lock );
+ _SMP_lock_Stats_initialize( &cpu->Lock_stats, "Per-CPU" );
}
/*
diff --git a/testsuites/sptests/sp37/init.c b/testsuites/sptests/sp37/init.c
index 1dd434f..2fbe117 100644
--- a/testsuites/sptests/sp37/init.c
+++ b/testsuites/sptests/sp37/init.c
@@ -161,8 +161,8 @@ static void test_isr_level( void )
#if defined(RTEMS_SMP) && defined(RTEMS_PROFILING)
static const size_t lock_size =
- offsetof( ISR_lock_Control, Lock.ticket_lock.Stats.name )
- + sizeof( ((ISR_lock_Control *) 0)->Lock.ticket_lock.Stats.name );
+ offsetof( ISR_lock_Control, Lock.Stats.name )
+ + sizeof( ((ISR_lock_Control *) 0)->Lock.Stats.name );
#else
static const size_t lock_size = sizeof( ISR_lock_Control );
#endif
More information about the vc
mailing list