RTEMS | SMP: Minor system start improvements (!227)

[Chris Johns (@chris)]

Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/227 was reviewed by Chris Johns

--
  
Chris Johns started a new discussion on cpukit/score/src/percpustatewait.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/227#note_112393

 > +#include <rtems/score/smpimpl.h>
 > +
 > +bool _Per_CPU_Wait_for_non_initial_state( uint32_t cpu_index )

Why return anything when it is always `true` given the commit changes functionality by removing the timeout?

--
  
Chris Johns started a new discussion on cpukit/score/src/smp.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/227#note_112394

 > +    if ( cpu != cpu_self ) {
 >        if ( _Scheduler_Should_start_processor( assignment ) ) {
 >          started = _CPU_SMP_Start_processor( cpu_index );

This is an interface that accepts a `bool` result of the CPU starting but this change removes that logic from the arm BSPs. Do any of the SMP supportted arch detect a CPU failing to start?


-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/227
You're receiving this email because of your account on gitlab.rtems.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20240920/a6eed6a3/attachment-0001.htm>