Quality RTOS & Embedded Software

 Real time embedded FreeRTOS RSS feed 
Quick Start Supported MCUs PDF Books Trace Tools Ecosystem


Minor bug in 4.3.1 xQueueSend()/XQueueRecieve

Posted by Dan on July 27, 2007
In 4.3.1:

Small issue in xQueueSend() and xQueueReceive() in queue.c.

Near the end, where it has the code:

if( xTicksToWait > 0 )
if( xTaskCheckForTimeOut( &xTimeOut, &xTicksToWait ) == pdFALSE )

I recommend:

#if ( INCLUDE_vTaskSuspend == 1 )
if(( xTicksToWait > 0 ) && ( xTicksToWait != portMAX_DELAY ))
if( xTicksToWait > 0 )
if( xTaskCheckForTimeOut( &xTimeOut, &xTicksToWait ) == pdFALSE )

xTicksToWait is set portMAX_DELAY for indefinite waits, but without
the above change, it will be decremented. This slightly effects
later code, which processes threads differently depending on if
their wait time is portMAX_DELAY or not.

Dan Searles

RE: Minor bug in 4.3.1 xQueueSend()/XQueueRecieve

Posted by Richard on July 27, 2007
Hmm. I think you have a good point. This part of the code came about from the SafeRTOS validation process - but in SafeRTOS there is no method of waiting indefinitely so the problem does not exist. The error was made when integrating this into the FreeRTOS.org code.

I think the fix would be better in xTaskCheckForTimeOut() to prevent it being duplicated in xQueueSend() and xQueueReceive(). Also these two functions are alread the most complex in the system, so best not to add code to them if possible.

Thank you for your contribution.


[ Back to the top ]    [ About FreeRTOS ]    [ Privacy ]    [ Sitemap ]    [ ]

Copyright (C) Amazon Web Services, Inc. or its affiliates. All rights reserved.

Latest News

FreeRTOS v10.2.0 is available for immediate download. MIT licensed, and including RISC-V and ARMv8-M (Cortex-M33) demos.

NXP tweet showing LPC5500 (ARMv8-M Cortex-M33) running FreeRTOS.

View a recording of the "OTA Update Security and Reliability" webinar, presented by TI and AWS.


FreeRTOS and other embedded software careers at AWS.

FreeRTOS Partners

ARM Connected RTOS partner for all ARM microcontroller cores

Cadence Tensilica Cortes

Espressif ESP32

IAR Partner

Microchip Premier RTOS Partner

RTOS partner of NXP for all NXP ARM microcontrollers





STMicro RTOS partner supporting ARM7, ARM Cortex-M3, ARM Cortex-M4 and ARM Cortex-M0

Texas Instruments MCU Developer Network RTOS partner for ARM and MSP430 microcontrollers

OpenRTOS and SafeRTOS

Xilinx Microblaze and Zynq partner