]> git.sur5r.net Git - freertos/blobdiff - FreeRTOS/Demo/Common/Minimal/TimerDemo.c
Ensure both one-shot and auto-reload are written consistently with a hyphen in comments.
[freertos] / FreeRTOS / Demo / Common / Minimal / TimerDemo.c
index 2e96a773974f0b51954b471f2ad89a7d4a653ae9..29478819a141747086bcda190d780a01faffdc02 100644 (file)
@@ -1,67 +1,29 @@
 /*\r
-    FreeRTOS V8.1.1 - Copyright (C) 2014 Real Time Engineers Ltd.\r
-    All rights reserved\r
-\r
-    VISIT http://www.FreeRTOS.org TO ENSURE YOU ARE USING THE LATEST VERSION.\r
-\r
-    ***************************************************************************\r
-     *                                                                       *\r
-     *    FreeRTOS provides completely free yet professionally developed,    *\r
-     *    robust, strictly quality controlled, supported, and cross          *\r
-     *    platform software that has become a de facto standard.             *\r
-     *                                                                       *\r
-     *    Help yourself get started quickly and support the FreeRTOS         *\r
-     *    project by purchasing a FreeRTOS tutorial book, reference          *\r
-     *    manual, or both from: http://www.FreeRTOS.org/Documentation        *\r
-     *                                                                       *\r
-     *    Thank you!                                                         *\r
-     *                                                                       *\r
-    ***************************************************************************\r
-\r
-    This file is part of the FreeRTOS distribution.\r
-\r
-    FreeRTOS is free software; you can redistribute it and/or modify it under\r
-    the terms of the GNU General Public License (version 2) as published by the\r
-    Free Software Foundation >>!AND MODIFIED BY!<< the FreeRTOS exception.\r
-\r
-    >>!   NOTE: The modification to the GPL is included to allow you to     !<<\r
-    >>!   distribute a combined work that includes FreeRTOS without being   !<<\r
-    >>!   obliged to provide the source code for proprietary components     !<<\r
-    >>!   outside of the FreeRTOS kernel.                                   !<<\r
-\r
-    FreeRTOS is distributed in the hope that it will be useful, but WITHOUT ANY\r
-    WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS\r
-    FOR A PARTICULAR PURPOSE.  Full license text is available from the following\r
-    link: http://www.freertos.org/a00114.html\r
-\r
-    1 tab == 4 spaces!\r
-\r
-    ***************************************************************************\r
-     *                                                                       *\r
-     *    Having a problem?  Start by reading the FAQ "My application does   *\r
-     *    not run, what could be wrong?"                                     *\r
-     *                                                                       *\r
-     *    http://www.FreeRTOS.org/FAQHelp.html                               *\r
-     *                                                                       *\r
-    ***************************************************************************\r
-\r
-    http://www.FreeRTOS.org - Documentation, books, training, latest versions,\r
-    license and Real Time Engineers Ltd. contact details.\r
-\r
-    http://www.FreeRTOS.org/plus - A selection of FreeRTOS ecosystem products,\r
-    including FreeRTOS+Trace - an indispensable productivity tool, a DOS\r
-    compatible FAT file system, and our tiny thread aware UDP/IP stack.\r
-\r
-    http://www.OpenRTOS.com - Real Time Engineers ltd license FreeRTOS to High\r
-    Integrity Systems to sell under the OpenRTOS brand.  Low cost OpenRTOS\r
-    licenses offer ticketed support, indemnification and middleware.\r
-\r
-    http://www.SafeRTOS.com - High Integrity Systems also provide a safety\r
-    engineered and independently SIL3 certified version for use in safety and\r
-    mission critical applications that require provable dependability.\r
-\r
-    1 tab == 4 spaces!\r
-*/\r
+ * FreeRTOS Kernel V10.2.1\r
+ * Copyright (C) 2019 Amazon.com, Inc. or its affiliates.  All Rights Reserved.\r
+ *\r
+ * Permission is hereby granted, free of charge, to any person obtaining a copy of\r
+ * this software and associated documentation files (the "Software"), to deal in\r
+ * the Software without restriction, including without limitation the rights to\r
+ * use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of\r
+ * the Software, and to permit persons to whom the Software is furnished to do so,\r
+ * subject to the following conditions:\r
+ *\r
+ * The above copyright notice and this permission notice shall be included in all\r
+ * copies or substantial portions of the Software.\r
+ *\r
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR\r
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS\r
+ * FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR\r
+ * COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER\r
+ * IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN\r
+ * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.\r
+ *\r
+ * http://www.FreeRTOS.org\r
+ * http://aws.amazon.com/freertos\r
+ *\r
+ * 1 tab == 4 spaces!\r
+ */\r
 \r
 \r
 /*\r
 \r
 #define tmrdemoDONT_BLOCK                              ( ( TickType_t ) 0 )\r
 #define tmrdemoONE_SHOT_TIMER_PERIOD   ( xBasePeriod * ( TickType_t ) 3 )\r
-#define trmdemoNUM_TIMER_RESETS                        ( ( uint8_t ) 10 )\r
+#define tmrdemoNUM_TIMER_RESETS                        ( ( uint8_t ) 10 )\r
+\r
+#ifndef tmrTIMER_TEST_TASK_STACK_SIZE\r
+       #define tmrTIMER_TEST_TASK_STACK_SIZE configMINIMAL_STACK_SIZE\r
+#endif\r
 \r
 /*-----------------------------------------------------------*/\r
 \r
@@ -104,7 +70,7 @@ static void prvISRAutoReloadTimerCallback( TimerHandle_t pxExpiredTimer );
 static void prvISROneShotTimerCallback( TimerHandle_t pxExpiredTimer );\r
 \r
 /* The test functions used by the timer test task.  These manipulate the auto\r
-reload and one shot timers in various ways, then delay, then inspect the timers\r
+reload and one-shot timers in various ways, then delay, then inspect the timers\r
 to ensure they have behaved as expected. */\r
 static void prvTest1_CreateTimersWithoutSchedulerRunning( void );\r
 static void prvTest2_CheckTaskAndTimersInitialState( void );\r
@@ -124,14 +90,14 @@ static volatile BaseType_t xTestStatus = pdPASS;
 detect a stalled task - a test that is no longer running. */\r
 static volatile uint32_t ulLoopCounter = 0;\r
 \r
-/* A set of auto reload timers - each of which use the same callback function.\r
+/* A set of auto-reload timers - each of which use the same callback function.\r
 The callback function uses the timer ID to index into, and then increment, a\r
-counter in the ucAutoReloadTimerCounters[] array.  The auto reload timers\r
+counter in the ucAutoReloadTimerCounters[] array.  The auto-reload timers\r
 referenced from xAutoReloadTimers[] are used by the prvTimerTestTask task. */\r
 static TimerHandle_t xAutoReloadTimers[ configTIMER_QUEUE_LENGTH + 1 ] = { 0 };\r
 static uint8_t ucAutoReloadTimerCounters[ configTIMER_QUEUE_LENGTH + 1 ] = { 0 };\r
 \r
-/* The one shot timer is configured to use a callback function that increments\r
+/* The one-shot timer is configured to use a callback function that increments\r
 ucOneShotTimerCounter each time it gets called. */\r
 static TimerHandle_t xOneShotTimer = NULL;\r
 static uint8_t ucOneShotTimerCounter = ( uint8_t ) 0;\r
@@ -142,7 +108,7 @@ ucISRReloadTimerCounter each time its callback function is executed. */
 static TimerHandle_t xISRAutoReloadTimer = NULL;\r
 static uint8_t ucISRAutoReloadTimerCounter = ( uint8_t ) 0;\r
 \r
-/* The ISR one shot timer is controlled from the tick hook to exercise the timer\r
+/* The ISR one-shot timer is controlled from the tick hook to exercise the timer\r
 API functions that can be used from an ISR.  It is configured to increment\r
 ucISRReloadTimerCounter each time its callback function is executed. */\r
 static TimerHandle_t xISROneShotTimer = NULL;\r
@@ -175,7 +141,7 @@ void vStartTimerDemoTask( TickType_t xBasePeriodIn )
        task, which will then preempt this task). */\r
        if( xTestStatus != pdFAIL )\r
        {\r
-               xTaskCreate( prvTimerTestTask, "Tmr Tst", configMINIMAL_STACK_SIZE, NULL, configTIMER_TASK_PRIORITY - 1, NULL );\r
+               xTaskCreate( prvTimerTestTask, "Tmr Tst", tmrTIMER_TEST_TASK_STACK_SIZE, NULL, configTIMER_TASK_PRIORITY - 1, NULL );\r
        }\r
 }\r
 /*-----------------------------------------------------------*/\r
@@ -184,11 +150,12 @@ static void prvTimerTestTask( void *pvParameters )
 {\r
        ( void ) pvParameters;\r
 \r
-       /* Create a one-shot timer for use later on in this test. */\r
+       /* Create a one-shot timer for use later on in this test.  For test purposes it\r
+       is created as an auto-reload timer then converted to a one-shot timer. */\r
        xOneShotTimer = xTimerCreate(   "Oneshot Timer",                                /* Text name to facilitate debugging.  The kernel does not use this itself. */\r
                                                                        tmrdemoONE_SHOT_TIMER_PERIOD,   /* The period for the timer. */\r
-                                                                       pdFALSE,                                                /* Don't auto-reload - hence a one shot timer. */\r
-                                                                       ( void * ) 0,                                   /* The timer identifier.  In this case this is not used as the timer has its own callback. */\r
+                                                                       pdFALSE,                                                /* Autorealod is false, so created as a one-shot timer. */\r
+                                                                       ( void * ) 0,                                   /* The timer identifier.  Initialise to 0, then increment each time it is called. */\r
                                                                        prvOneShotTimerCallback );              /* The callback to be called when the timer expires. */\r
 \r
        if( xOneShotTimer == NULL )\r
@@ -197,6 +164,20 @@ static void prvTimerTestTask( void *pvParameters )
                configASSERT( xTestStatus );\r
        }\r
 \r
+       /* Purely for test coverage purposes - change and query the reload mode to\r
+       auto-reload then back to one-shot. */\r
+\r
+       /* Change timer to auto-reload. */\r
+       vTimerSetReloadMode( xOneShotTimer, pdTRUE );\r
+\r
+       /* Timer should now be auto-reload. */\r
+       configASSERT( uxTimerGetReloadMode( xOneShotTimer ) == pdTRUE );\r
+\r
+       /* Change timer to one-shot, which is what is needed for this test. */\r
+       vTimerSetReloadMode( xOneShotTimer, pdFALSE );\r
+\r
+       /* Check change to one-shot was successful. */\r
+       configASSERT( uxTimerGetReloadMode( xOneShotTimer ) == pdFALSE );\r
 \r
        /* Ensure all the timers are in their expected initial state.  This\r
        depends on the timer service task having a higher priority than this task. */\r
@@ -204,14 +185,14 @@ static void prvTimerTestTask( void *pvParameters )
 \r
        for( ;; )\r
        {\r
-               /* Check the auto reload timers expire at the expected/correct rates. */\r
+               /* Check the auto-reload timers expire at the expected/correct rates. */\r
                prvTest3_CheckAutoReloadExpireRates();\r
 \r
-               /* Check the auto reload timers can be stopped correctly, and correctly\r
+               /* Check the auto-reload timers can be stopped correctly, and correctly\r
                report their state. */\r
                prvTest4_CheckAutoReloadTimersCanBeStopped();\r
 \r
-               /* Check the one shot timer only calls its callback once after it has been\r
+               /* Check the one-shot timer only calls its callback once after it has been\r
                started, and that it reports its state correctly. */\r
                prvTest5_CheckBasicOneShotTimerBehaviour();\r
 \r
@@ -256,7 +237,6 @@ static TickType_t xIterationsWithoutCounterIncrement = ( TickType_t ) 0, xLastCy
                {\r
                        /* The tests appear to be no longer running (stalled). */\r
                        xTestStatus = pdFAIL;\r
-                       configASSERT( xTestStatus );\r
                }\r
        }\r
        else\r
@@ -277,22 +257,20 @@ static TickType_t xIterationsWithoutCounterIncrement = ( TickType_t ) 0, xLastCy
 \r
 static void prvTest1_CreateTimersWithoutSchedulerRunning( void )\r
 {\r
-UBaseType_t xTimer;\r
+TickType_t xTimer;\r
 \r
        for( xTimer = 0; xTimer < configTIMER_QUEUE_LENGTH; xTimer++ )\r
        {\r
-               /* As the timer queue is not yet full, it should be possible to both create\r
-               and start a timer.  These timers are being started before the scheduler has\r
-               been started, so their block times should get set to zero within the timer\r
-               API itself. */\r
+               /* As the timer queue is not yet full, it should be possible to both\r
+               create and start a timer.  These timers are being started before the\r
+               scheduler has been started, so their block times should get set to zero\r
+               within the timer API itself. */\r
                xAutoReloadTimers[ xTimer ] = xTimerCreate( "FR Timer",                                                 /* Text name to facilitate debugging.  The kernel does not use this itself. */\r
                                                                                                        ( ( xTimer + ( TickType_t ) 1 ) * xBasePeriod ),/* The period for the timer.  The plus 1 ensures a period of zero is not specified. */\r
                                                                                                        pdTRUE,                                                         /* Auto-reload is set to true. */\r
-                                                                                                       ( void * ) xTimer,                                      /* An identifier for the timer as all the auto reload timers use the same callback. */\r
+                                                                                                       ( void * ) xTimer,                                      /* An identifier for the timer as all the auto-reload timers use the same callback. */\r
                                                                                                        prvAutoReloadTimerCallback );           /* The callback to be called when the timer expires. */\r
 \r
-               configASSERT( strcmp( pcTimerGetTimerName( xAutoReloadTimers[ xTimer ] ), "FR Timer" ) == 0 );\r
-\r
                if( xAutoReloadTimers[ xTimer ] == NULL )\r
                {\r
                        xTestStatus = pdFAIL;\r
@@ -300,6 +278,8 @@ UBaseType_t xTimer;
                }\r
                else\r
                {\r
+                       configASSERT( strcmp( pcTimerGetName( xAutoReloadTimers[ xTimer ] ), "FR Timer" ) == 0 );\r
+\r
                        /* The scheduler has not yet started, so the block period of\r
                        portMAX_DELAY should just get set to zero in xTimerStart().  Also,\r
                        the timer queue is not yet full so xTimerStart() should return\r
@@ -318,7 +298,7 @@ UBaseType_t xTimer;
        xAutoReloadTimers[ configTIMER_QUEUE_LENGTH ] = xTimerCreate( "FR Timer",                                       /* Text name to facilitate debugging.  The kernel does not use this itself. */\r
                                                                                                        ( configTIMER_QUEUE_LENGTH * xBasePeriod ),     /* The period for the timer. */\r
                                                                                                        pdTRUE,                                                                         /* Auto-reload is set to true. */\r
-                                                                                                       ( void * ) xTimer,                                                      /* An identifier for the timer as all the auto reload timers use the same callback. */\r
+                                                                                                       ( void * ) xTimer,                                                      /* An identifier for the timer as all the auto-reload timers use the same callback. */\r
                                                                                                        prvAutoReloadTimerCallback );                           /* The callback executed when the timer expires. */\r
 \r
        if( xAutoReloadTimers[ configTIMER_QUEUE_LENGTH ] == NULL )\r
@@ -341,13 +321,13 @@ UBaseType_t xTimer;
        API functions that can be called from an ISR. */\r
        xISRAutoReloadTimer = xTimerCreate( "ISR AR",                                                   /* The text name given to the timer. */\r
                                                                                0xffff,                                                         /* The timer is not given a period yet - this will be done from the tick hook, but a period of 0 is invalid. */\r
-                                                                               pdTRUE,                                                         /* This is an auto reload timer. */\r
+                                                                               pdTRUE,                                                         /* This is an auto-reload timer. */\r
                                                                                ( void * ) NULL,                                        /* The identifier is not required. */\r
                                                                                prvISRAutoReloadTimerCallback );        /* The callback that is executed when the timer expires. */\r
 \r
        xISROneShotTimer = xTimerCreate(        "ISR OS",                                                       /* The text name given to the timer. */\r
                                                                                0xffff,                                                         /* The timer is not given a period yet - this will be done from the tick hook, but a period of 0 is invalid. */\r
-                                                                               pdFALSE,                                                        /* This is a one shot timer. */\r
+                                                                               pdFALSE,                                                        /* This is a one-shot timer. */\r
                                                                                ( void * ) NULL,                                        /* The identifier is not required. */\r
                                                                                prvISROneShotTimerCallback );           /* The callback that is executed when the timer expires. */\r
 \r
@@ -366,8 +346,8 @@ uint8_t ucTimer;
        /* Ensure all the timers are in their expected initial state.  This     depends\r
        on the timer service task having a higher priority than this task.\r
 \r
-       auto reload timers 0 to ( configTIMER_QUEUE_LENGTH - 1 ) should now be active,\r
-       and auto reload timer configTIMER_QUEUE_LENGTH should not yet be active (it\r
+       auto-reload timers 0 to ( configTIMER_QUEUE_LENGTH - 1 ) should now be active,\r
+       and auto-reload timer configTIMER_QUEUE_LENGTH should not yet be active (it\r
        could not be started prior to the scheduler being started when it was\r
        created). */\r
        for( ucTimer = 0; ucTimer < ( uint8_t ) configTIMER_QUEUE_LENGTH; ucTimer++ )\r
@@ -391,16 +371,20 @@ static void       prvTest3_CheckAutoReloadExpireRates( void )
 {\r
 uint8_t ucMaxAllowableValue, ucMinAllowableValue, ucTimer;\r
 TickType_t xBlockPeriod, xTimerPeriod, xExpectedNumber;\r
+UBaseType_t uxOriginalPriority;\r
 \r
-       /* Check the auto reload timers expire at the expected rates. */\r
-\r
+       /* Check the auto-reload timers expire at the expected rates.  Do this at a\r
+       high priority for maximum accuracy.  This is ok as most of the time is spent\r
+       in the Blocked state. */\r
+       uxOriginalPriority = uxTaskPriorityGet( NULL );\r
+       vTaskPrioritySet( NULL, ( configMAX_PRIORITIES - 1 ) );\r
 \r
        /* Delaying for configTIMER_QUEUE_LENGTH * xBasePeriod ticks should allow\r
-       all the auto reload timers to expire at least once. */\r
+       all the auto-reload timers to expire at least once. */\r
        xBlockPeriod = ( ( TickType_t ) configTIMER_QUEUE_LENGTH ) * xBasePeriod;\r
        vTaskDelay( xBlockPeriod );\r
 \r
-       /* Check that all the auto reload timers have called their callback\r
+       /* Check that all the auto-reload timers have called their callback\r
        function the expected number of times. */\r
        for( ucTimer = 0; ucTimer < ( uint8_t ) configTIMER_QUEUE_LENGTH; ucTimer++ )\r
        {\r
@@ -421,6 +405,9 @@ TickType_t xBlockPeriod, xTimerPeriod, xExpectedNumber;
                }\r
        }\r
 \r
+       /* Return to the original priority. */\r
+       vTaskPrioritySet( NULL, uxOriginalPriority );\r
+\r
        if( xTestStatus == pdPASS )\r
        {\r
                /* No errors have been reported so increment the loop counter so the\r
@@ -434,7 +421,7 @@ static void prvTest4_CheckAutoReloadTimersCanBeStopped( void )
 {\r
 uint8_t ucTimer;\r
 \r
-       /* Check the auto reload timers can be stopped correctly, and correctly\r
+       /* Check the auto-reload timers can be stopped correctly, and correctly\r
        report their state. */\r
 \r
        /* Stop all the active timers. */\r
@@ -500,10 +487,10 @@ uint8_t ucTimer;
 \r
 static void prvTest5_CheckBasicOneShotTimerBehaviour( void )\r
 {\r
-       /* Check the one shot timer only calls its callback once after it has been\r
+       /* Check the one-shot timer only calls its callback once after it has been\r
        started, and that it reports its state correctly. */\r
 \r
-       /* The one shot timer should not be active yet. */\r
+       /* The one-shot timer should not be active yet. */\r
        if( xTimerIsTimerActive( xOneShotTimer ) != pdFALSE )\r
        {\r
                xTestStatus = pdFAIL;\r
@@ -516,7 +503,7 @@ static void prvTest5_CheckBasicOneShotTimerBehaviour( void )
                configASSERT( xTestStatus );\r
        }\r
 \r
-       /* Start the one shot timer and check that it reports its state correctly. */\r
+       /* Start the one-shot timer and check that it reports its state correctly. */\r
        xTimerStart( xOneShotTimer, tmrdemoDONT_BLOCK );\r
        if( xTimerIsTimerActive( xOneShotTimer ) == pdFALSE )\r
        {\r
@@ -524,7 +511,7 @@ static void prvTest5_CheckBasicOneShotTimerBehaviour( void )
                configASSERT( xTestStatus );\r
        }\r
 \r
-       /* Delay for three times as long as the one shot timer period, then check\r
+       /* Delay for three times as long as the one-shot timer period, then check\r
        to ensure it has only called its callback once, and is now not in the\r
        active state. */\r
        vTaskDelay( tmrdemoONE_SHOT_TIMER_PERIOD * ( TickType_t ) 3 );\r
@@ -542,7 +529,7 @@ static void prvTest5_CheckBasicOneShotTimerBehaviour( void )
        }\r
        else\r
        {\r
-               /* Reset the one shot timer callback count. */\r
+               /* Reset the one-shot timer callback count. */\r
                ucOneShotTimerCounter = ( uint8_t ) 0;\r
        }\r
 \r
@@ -561,7 +548,7 @@ uint8_t ucTimer;
 \r
        /* Check timer reset behaviour. */\r
 \r
-       /* Restart the one shot timer and check it reports its status correctly. */\r
+       /* Restart the one-shot timer and check it reports its status correctly. */\r
        xTimerStart( xOneShotTimer, tmrdemoDONT_BLOCK );\r
        if( xTimerIsTimerActive( xOneShotTimer ) == pdFALSE )\r
        {\r
@@ -569,7 +556,7 @@ uint8_t ucTimer;
                configASSERT( xTestStatus );\r
        }\r
 \r
-       /* Restart one of the auto reload timers and check that it reports its\r
+       /* Restart one of the auto-reload timers and check that it reports its\r
        status correctly. */\r
        xTimerStart( xAutoReloadTimers[ configTIMER_QUEUE_LENGTH - 1 ], tmrdemoDONT_BLOCK );\r
        if( xTimerIsTimerActive( xAutoReloadTimers[ configTIMER_QUEUE_LENGTH - 1 ] ) == pdFALSE )\r
@@ -578,9 +565,9 @@ uint8_t ucTimer;
                configASSERT( xTestStatus );\r
        }\r
 \r
-       for( ucTimer = 0; ucTimer < trmdemoNUM_TIMER_RESETS; ucTimer++ )\r
+       for( ucTimer = 0; ucTimer < tmrdemoNUM_TIMER_RESETS; ucTimer++ )\r
        {\r
-               /* Delay for half as long as the one shot timer period, then reset it.\r
+               /* Delay for half as long as the one-shot timer period, then reset it.\r
                It should never expire while this is done, so its callback count should\r
                never increment. */\r
                vTaskDelay( tmrdemoONE_SHOT_TIMER_PERIOD / 2 );\r
@@ -640,7 +627,7 @@ uint8_t ucTimer;
                configASSERT( xTestStatus );\r
        }\r
 \r
-       /* The one shot timer should no longer be active, while the auto reload\r
+       /* The one-shot timer should no longer be active, while the auto-reload\r
        timer should still be active. */\r
        if( xTimerIsTimerActive( xAutoReloadTimers[ configTIMER_QUEUE_LENGTH - 1 ] ) == pdFALSE )\r
        {\r
@@ -654,7 +641,7 @@ uint8_t ucTimer;
                configASSERT( xTestStatus );\r
        }\r
 \r
-       /* Stop the auto reload timer again. */\r
+       /* Stop the auto-reload timer again. */\r
        xTimerStop( xAutoReloadTimers[ configTIMER_QUEUE_LENGTH - 1 ], tmrdemoDONT_BLOCK );\r
 \r
        if( xTimerIsTimerActive( xAutoReloadTimers[ configTIMER_QUEUE_LENGTH - 1 ] ) != pdFALSE )\r
@@ -739,11 +726,16 @@ static TickType_t uxTick = ( TickType_t ) -1;
        will expire when the kernel's tick count is (100 + xBasePeriod).  For this\r
        reason xMargin is used as an allowable margin for premature timer expiries\r
        as well as late timer expiries. */\r
-       const TickType_t xMargin = 6;\r
+       #ifdef _WINDOWS_\r
+               /* Windows is not real real time. */\r
+               const TickType_t xMargin = 20;\r
+       #else\r
+               const TickType_t xMargin = 6;\r
+       #endif /* _WINDOWS_ */\r
 #else\r
        #ifdef _WINDOWS_\r
                /* Windows is not real real time. */\r
-               const TickType_t xMargin = 8;\r
+               const TickType_t xMargin = 20;\r
        #else\r
                const TickType_t xMargin = 4;\r
        #endif /* _WINDOWS_ */\r
@@ -793,8 +785,8 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( xBasePeriod + xMargin ) )\r
        {\r
-               /* Both timers should now have expired once.  The auto reload timer will\r
-               still be active, but the one shot timer should now have stopped. */\r
+               /* Both timers should now have expired once.  The auto-reload timer will\r
+               still be active, but the one-shot timer should now have stopped. */\r
                if( ( ucISRAutoReloadTimerCounter != 1 ) || ( ucISROneShotTimerCounter != 1 ) )\r
                {\r
                        xTestStatus = pdFAIL;\r
@@ -803,7 +795,7 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( ( 2 * xBasePeriod ) - xMargin ) )\r
        {\r
-               /* The auto reload timer will still be active, but the one shot timer\r
+               /* The auto-reload timer will still be active, but the one-shot timer\r
                should now have stopped - however, at this time neither of the timers\r
                should have expired again since the last test. */\r
                if( ( ucISRAutoReloadTimerCounter != 1 ) || ( ucISROneShotTimerCounter != 1 ) )\r
@@ -814,9 +806,9 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( ( 2 * xBasePeriod ) + xMargin ) )\r
        {\r
-               /* The auto reload timer will still be active, but the one shot timer\r
-               should now have stopped.  At this time the auto reload timer should have\r
-               expired again, but the one shot timer count should not have changed. */\r
+               /* The auto-reload timer will still be active, but the one-shot timer\r
+               should now have stopped.  At this time the auto-reload timer should have\r
+               expired again, but the one-shot timer count should not have changed. */\r
                if( ucISRAutoReloadTimerCounter != 2 )\r
                {\r
                        xTestStatus = pdFAIL;\r
@@ -831,7 +823,7 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( ( 2 * xBasePeriod ) + ( xBasePeriod >> ( TickType_t ) 2U ) ) )\r
        {\r
-               /* The auto reload timer will still be active, but the one shot timer\r
+               /* The auto-reload timer will still be active, but the one-shot timer\r
                should now have stopped.  Again though, at this time, neither timer call\r
                back should have been called since the last test. */\r
                if( ucISRAutoReloadTimerCounter != 2 )\r
@@ -848,13 +840,13 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( 3 * xBasePeriod ) )\r
        {\r
-               /* Start the one shot timer again. */\r
+               /* Start the one-shot timer again. */\r
                xTimerStartFromISR( xISROneShotTimer, NULL );\r
        }\r
        else if( uxTick == ( ( 3 * xBasePeriod ) + xMargin ) )\r
        {\r
-               /* The auto reload timer and one shot timer will be active.  At\r
-               this time the auto reload timer should have     expired again, but the one\r
+               /* The auto-reload timer and one-shot timer will be active.  At\r
+               this time the auto-reload timer should have     expired again, but the one\r
                shot timer count should not have changed yet. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
                {\r
@@ -868,13 +860,13 @@ static TickType_t uxTick = ( TickType_t ) -1;
                        configASSERT( xTestStatus );\r
                }\r
 \r
-               /* Now stop the auto reload timer.  The one shot timer was started\r
+               /* Now stop the auto-reload timer.  The one-shot timer was started\r
                a few ticks ago. */\r
                xTimerStopFromISR( xISRAutoReloadTimer, NULL );\r
        }\r
        else if( uxTick == ( 4 * ( xBasePeriod - xMargin ) ) )\r
        {\r
-               /* The auto reload timer is now stopped, and the one shot timer is\r
+               /* The auto-reload timer is now stopped, and the one-shot timer is\r
                active, but at this time neither timer should have expired since the\r
                last test. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
@@ -891,8 +883,8 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( ( 4 * xBasePeriod ) + xMargin ) )\r
        {\r
-               /* The auto reload timer is now stopped, and the one shot timer is\r
-               active.  The one shot timer should have expired again, but the auto\r
+               /* The auto-reload timer is now stopped, and the one-shot timer is\r
+               active.  The one-shot timer should have expired again, but the auto\r
                reload timer should not have executed its callback. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
                {\r
@@ -908,7 +900,7 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( 8 * xBasePeriod ) )\r
        {\r
-               /* The auto reload timer is now stopped, and the one shot timer has\r
+               /* The auto-reload timer is now stopped, and the one-shot timer has\r
                already expired and then stopped itself.  Both callback counters should\r
                not have incremented since the last test. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
@@ -923,14 +915,14 @@ static TickType_t uxTick = ( TickType_t ) -1;
                        configASSERT( xTestStatus );\r
                }\r
 \r
-               /* Now reset the one shot timer. */\r
+               /* Now reset the one-shot timer. */\r
                xTimerResetFromISR( xISROneShotTimer, NULL );\r
        }\r
        else if( uxTick == ( ( 9 * xBasePeriod ) - xMargin ) )\r
        {\r
-               /* Only the one shot timer should be running, but it should not have\r
+               /* Only the one-shot timer should be running, but it should not have\r
                expired since the last test.  Check the callback counters have not\r
-               incremented, then reset the one shot timer again. */\r
+               incremented, then reset the one-shot timer again. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
                {\r
                        xTestStatus = pdFAIL;\r
@@ -947,9 +939,9 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( ( 10 * xBasePeriod ) - ( 2 * xMargin ) ) )\r
        {\r
-               /* Only the one shot timer should be running, but it should not have\r
+               /* Only the one-shot timer should be running, but it should not have\r
                expired since the last test.  Check the callback counters have not\r
-               incremented, then reset the one shot timer again. */\r
+               incremented, then reset the one-shot timer again. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
                {\r
                        xTestStatus = pdFAIL;\r
@@ -966,9 +958,9 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( ( 11 * xBasePeriod ) - ( 3 * xMargin ) ) )\r
        {\r
-               /* Only the one shot timer should be running, but it should not have\r
+               /* Only the one-shot timer should be running, but it should not have\r
                expired since the last test.  Check the callback counters have not\r
-               incremented, then reset the one shot timer once again. */\r
+               incremented, then reset the one-shot timer once again. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
                {\r
                        xTestStatus = pdFAIL;\r
@@ -985,10 +977,10 @@ static TickType_t uxTick = ( TickType_t ) -1;
        }\r
        else if( uxTick == ( ( 12 * xBasePeriod ) - ( 2 * xMargin ) ) )\r
        {\r
-               /* Only the one shot timer should have been running and this time it\r
+               /* Only the one-shot timer should have been running and this time it\r
                should have     expired.  Check its callback count has been incremented.\r
-               The auto reload timer is still not running so should still have the same\r
-               count value.  This time the one shot timer is not reset so should not\r
+               The auto-reload timer is still not running so should still have the same\r
+               count value.  This time the one-shot timer is not reset so should not\r
                restart from its expiry period again. */\r
                if( ucISRAutoReloadTimerCounter != 3 )\r
                {\r
@@ -1028,12 +1020,12 @@ static TickType_t uxTick = ( TickType_t ) -1;
 \r
 static void prvAutoReloadTimerCallback( TimerHandle_t pxExpiredTimer )\r
 {\r
-uint32_t ulTimerID;\r
+size_t uxTimerID;\r
 \r
-       ulTimerID = ( uint32_t ) pvTimerGetTimerID( pxExpiredTimer );\r
-       if( ulTimerID <= ( configTIMER_QUEUE_LENGTH + 1 ) )\r
+       uxTimerID = ( size_t ) pvTimerGetTimerID( pxExpiredTimer );\r
+       if( uxTimerID <= ( configTIMER_QUEUE_LENGTH + 1 ) )\r
        {\r
-               ( ucAutoReloadTimerCounters[ ulTimerID ] )++;\r
+               ( ucAutoReloadTimerCounters[ uxTimerID ] )++;\r
        }\r
        else\r
        {\r
@@ -1046,9 +1038,22 @@ uint32_t ulTimerID;
 \r
 static void prvOneShotTimerCallback( TimerHandle_t pxExpiredTimer )\r
 {\r
-       /* The parameter is not used in this case as only one timer uses this\r
-       callback function. */\r
-       ( void ) pxExpiredTimer;\r
+/* A count is kept of the number of times this callback function is executed.\r
+The count is stored as the timer's ID.  This is only done to test the\r
+vTimerSetTimerID() function. */\r
+static size_t uxCallCount = 0;\r
+size_t uxLastCallCount;\r
+\r
+       /* Obtain the timer's ID, which should be a count of the number of times\r
+       this callback function has been executed. */\r
+       uxLastCallCount = ( size_t ) pvTimerGetTimerID( pxExpiredTimer );\r
+       configASSERT( uxLastCallCount == uxCallCount );\r
+\r
+       /* Increment the call count, then save it back as the timer's ID.  This is\r
+       only done to test the vTimerSetTimerID() API function. */\r
+       uxLastCallCount++;\r
+       vTimerSetTimerID( pxExpiredTimer, ( void * ) uxLastCallCount );\r
+       uxCallCount++;\r
 \r
        ucOneShotTimerCounter++;\r
 }\r