RTEMS | cpukit: fix off-chain state checks and pointer clearing (!1067)

[Chandan U (@Chandanuvm)]

Chandan U commented on a discussion on cpukit/score/src/threadqflush.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1067#note_143862

 >        Scheduler_Node *scheduler_node;
 >  
 >        scheduler_node = _Thread_Scheduler_get_home_node( first );
 > +      _Chain_Initialize_node( &scheduler_node->Wait.Priority.Node.Node.Chain );

Thank you for the suggestions, 

To maintain the separation properly and keep the RB tree logic same and not alter it just for clearing chain pointers  i felt 2nd alternative is better.

So i considered the 2nd alternative and made changes as mentioned and also i have added the safety assertion check in threadqflush.c before append is called for the scheduler node.

-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1067#note_143862
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/20260228/9e3a6df9/attachment.htm>