Skip to content

Conversation

@jefftenney
Copy link
Contributor

Description

This PR fixes #557 (which had to be reverted in #566) by putting a warning in TimerDemo.h about the backlog tests. There are two distinct parts to this PR:

Note that the warning in TimerDemo.h is important with or without the additional backlog test.

Test Steps

Run the timer demo in isolation (not concurrently with other demos). Call vTimerDemoIncludeBacklogTests( pdTRUE ) to include the new test provided in this PR. (The test detects an issue resolved by FreeRTOS/FreeRTOS-Kernel#305 and identified below.)

Related Issue

https://forums.freertos.org/t/bug-xtimeristimeractive-is-returning-wrong-value-v10-3-0/9469

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jefftenney jefftenney requested a review from a team as a code owner May 7, 2021 23:05
@jefftenney
Copy link
Contributor Author

@RichardBarry, @dan4thewin Just trying to clean up my mess before somebody steps in it. No rush.

@jefftenney jefftenney force-pushed the revert-566-revert-557-add-test-late-timer-start branch from 5f6f1c9 to ba09a98 Compare August 17, 2021 21:46
@jefftenney jefftenney force-pushed the revert-566-revert-557-add-test-late-timer-start branch from ba09a98 to 438d3a6 Compare August 17, 2021 22:13
@n9wxu n9wxu merged commit 2e77211 into FreeRTOS:main Aug 18, 2022
@jefftenney jefftenney deleted the revert-566-revert-557-add-test-late-timer-start branch August 18, 2022 21:18
moninom1 added a commit to moninom1/FreeRTOS that referenced this pull request Apr 17, 2023
* Add IPv6 data path and endpoint changes to FreeRTOS_IP.c
* Add FreeRTOS_IPv4.h and FreeRTOS_IPv6.h files.
* Fix compilation issues and run Uncrustify
Zangetsu112 pushed a commit to Zangetsu112/FreeRTOS-evpp that referenced this pull request Aug 18, 2025
* Add test for timer start delayed past expiration

This reverts commit 9cf8b2e, which
itself reverted 60dbcf0.

* Add warning about vTimerDemoIncludeBacklogTests()

* uncrustify for CI check

* Remove unintentional changes to spacing in comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants