From: richardbarry Date: Sun, 22 Oct 2006 20:28:16 +0000 (+0000) Subject: Changes from V4.1.2 X-Git-Tag: V4.1.3~8 X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=470c8fe7d30ad721a9c811ddfd8616901d7084b4;p=freertos Changes from V4.1.2 + Tasks that block with a timeout of portMAX_DELAY are now blocked indefinitely. Previously portMAX_DELAY was just the longest block time possible. git-svn-id: https://svn.code.sf.net/p/freertos/code/trunk@49 1d2547de-c912-0410-9cb9-b8ca96c0e9e2 --- diff --git a/Source/tasks.c b/Source/tasks.c index 3d6f3175f..9b722bf75 100644 --- a/Source/tasks.c +++ b/Source/tasks.c @@ -181,6 +181,12 @@ Changes from V4.0.5 + Added utility functions and xOverflowCount variable to facilitate the queue.c changes. + +Changes from V4.1.2 + + + Tasks that block with a timeout of portMAX_DELAY are now blocked + indefinitely. Previously portMAX_DELAY was just the longest block time + possible. */ #include @@ -1418,26 +1424,36 @@ portTickType xTimeToWake; is the first to be woken by the event. */ vListInsert( ( xList * ) pxEventList, ( xListItem * ) &( pxCurrentTCB->xEventListItem ) ); - /* Calculate the time at which the task should be woken if the event does - not occur. This may overflow but this doesn't matter. */ - xTimeToWake = xTickCount + xTicksToWait; - /* We must remove ourselves from the ready list before adding ourselves to the blocked list as the same list item is used for both lists. We have exclusive access to the ready lists as the scheduler is locked. */ vListRemove( ( xListItem * ) &( pxCurrentTCB->xGenericListItem ) ); - listSET_LIST_ITEM_VALUE( &( pxCurrentTCB->xGenericListItem ), xTimeToWake ); - - if( xTimeToWake < xTickCount ) + if( xTicksToWait == portMAX_DELAY ) { - /* Wake time has overflowed. Place this item in the overflow list. */ - vListInsert( ( xList * ) pxOverflowDelayedTaskList, ( xListItem * ) &( pxCurrentTCB->xGenericListItem ) ); + /* Add ourselves to the suspended task list instead of a delayed task + list to ensure we are not woken by a timing event. We will block + indefinitely. */ + vListInsertEnd( ( xList * ) &xSuspendedTaskList, ( xListItem * ) &( pxCurrentTCB->xGenericListItem ) ); } else { - /* The wake time has not overflowed, so we can use the current block list. */ - vListInsert( ( xList * ) pxDelayedTaskList, ( xListItem * ) &( pxCurrentTCB->xGenericListItem ) ); + /* Calculate the time at which the task should be woken if the event does + not occur. This may overflow but this doesn't matter. */ + xTimeToWake = xTickCount + xTicksToWait; + + listSET_LIST_ITEM_VALUE( &( pxCurrentTCB->xGenericListItem ), xTimeToWake ); + + if( xTimeToWake < xTickCount ) + { + /* Wake time has overflowed. Place this item in the overflow list. */ + vListInsert( ( xList * ) pxOverflowDelayedTaskList, ( xListItem * ) &( pxCurrentTCB->xGenericListItem ) ); + } + else + { + /* The wake time has not overflowed, so we can use the current block list. */ + vListInsert( ( xList * ) pxDelayedTaskList, ( xListItem * ) &( pxCurrentTCB->xGenericListItem ) ); + } } } /*-----------------------------------------------------------*/