[PATCH 1/3] score: Use consistent type for SMP messages

Sebastian Huber sebastian.huber at embedded-brains.de
Mon Jul 7 06:50:47 UTC 2014


I had a closer look at the change, and it makes no sense.  If you look at

void _SMP_Send_message( uint32_t cpu_index, unsigned long message )
{
   Per_CPU_Control *cpu = _Per_CPU_Get_by_index( cpu_index );

   _Atomic_Fetch_or_ulong( &cpu->message, message, ATOMIC_ORDER_RELAXED );

   _CPU_SMP_Send_interrupt( cpu_index );
}

you see that the message parameter is used by _Atomic_Fetch_or_ulong().  So the 
type must be unsigned long and not uint32_t.

On 2014-07-03 11:37, Daniel Cederman wrote:
> ---
>   cpukit/score/include/rtems/score/smpimpl.h |    2 +-
>   cpukit/score/src/smp.c                     |    2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cpukit/score/include/rtems/score/smpimpl.h b/cpukit/score/include/rtems/score/smpimpl.h
> index e2fee39..ba16c8f 100644
> --- a/cpukit/score/include/rtems/score/smpimpl.h
> +++ b/cpukit/score/include/rtems/score/smpimpl.h
> @@ -159,7 +159,7 @@ static inline void _SMP_Inter_processor_interrupt_handler( void )
>    *  @param[in] cpu_index The target processor of the message.
>    *  @param[in] message The message.
>    */
> -void _SMP_Send_message( uint32_t cpu_index, unsigned long message );
> +void _SMP_Send_message( uint32_t cpu_index, uint32_t message );
>
>   /**
>    *  @brief Request of others CPUs.
> diff --git a/cpukit/score/src/smp.c b/cpukit/score/src/smp.c
> index f0554fe..59448a0 100644
> --- a/cpukit/score/src/smp.c
> +++ b/cpukit/score/src/smp.c
> @@ -153,7 +153,7 @@ void _SMP_Request_shutdown( void )
>     _Giant_Drop( self_cpu );
>   }
>
> -void _SMP_Send_message( uint32_t cpu_index, unsigned long message )
> +void _SMP_Send_message( uint32_t cpu_index, uint32_t message )
>   {
>     Per_CPU_Control *cpu = _Per_CPU_Get_by_index( cpu_index );
>
>


-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.


More information about the devel mailing list