13022591351 commented on PR #3759:
URL: https://github.com/apache/nuttx-apps/pull/3759#issuecomment-5568833940
Thanks for the review, and sorry for the delayed update.
I force-pushed the branch after addressing all three comments:
- Removed the `__KERNEL__` path from the apps test.
- Removed the top-level `CONFIG_DISABLE_PTHREAD` implementation split.
Semaphore waits and basic workers are now shared, while conditional
compilation is limited to tests that actually require pthread-backed
custom queues.
- Squashed the no-pthread USRWORK coverage into the main test commit.
The branch is now a single commit rebased onto the current nuttx-apps master.
The dependent NuttX work queue API PR, apache/nuttx#19982, has also been
merged.
I rebuilt and tested the updated code on PX4 FMUv6C in Flat, Protected, and
Protected-with-pthreads-disabled configurations. All work queue tests passed,
including repeated runs without cumulative heap growth.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
