RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

score: Fix _Thread_queue_Path_release()

Browse Source 
It is possible that the owner of the terminal link of a thread queue
path waits on a thread queue.  However, this thread queue has no owner,
e.g. a thread queue of a message queue.
This commit is contained in:
Sebastian Huber
2016-08-04 10:16:14 +02:00
parent 6117f29cd4
commit ca783bbe4c
1 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
cpukit/score/src/threadqenqueue.c
View File

@@ -177,19 +177,24 @@ static void _Thread_queue_Path_release( Thread_queue_Path *path )
if ( head != node ) { if ( head != node ) {
Thread_queue_Link *link; Thread_queue_Link *link;
/* The terminal link has an owner which does not wait on a thread queue */ /*
* The terminal link may have an owner which does not wait on a thread
* queue.
*/
link = THREAD_QUEUE_LINK_OF_PATH_NODE( node ); link = THREAD_QUEUE_LINK_OF_PATH_NODE( node );
_Assert( link->Queue_context.Wait.queue == NULL );
_Thread_Wait_release_default_critical( if ( link->Queue_context.Wait.queue == NULL ) {
link->owner, _Thread_Wait_release_default_critical(
&link->Queue_context.Lock_context link->owner,
); &link->Queue_context.Lock_context
);
node = _Chain_Previous( node ); node = _Chain_Previous( node );
#if defined(RTEMS_DEBUG) #if defined(RTEMS_DEBUG)
_Chain_Set_off_chain( &link->Path_node ); _Chain_Set_off_chain( &link->Path_node );
#endif #endif
}
while ( head != node ) { while ( head != node ) {
/* The other links have an owner which waits on a thread queue */ /* The other links have an owner which waits on a thread queue */