Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FreeRTOS-Plus-TCP port fixes #822

Merged
merged 10 commits into from
Dec 10, 2024

Conversation

bjsowa
Copy link
Contributor

@bjsowa bjsowa commented Dec 8, 2024

  1. Fixed mutex drop by using vSemaphoreDelete instead of z_free.
  2. Added static allocation support to mutexes.
  3. Fixed _z_task_detach by pointing it to _z_task_cancel.
  4. Fixed freeing task resources. The current implementation does not make much sense.
  5. Fixed issues with task cancelling and joining.
  6. Used gettimeofday for z_time functions. It doesn't really make sense to use tick count for this. If compiled with newlib for a target that does not support this function, user can just implement _gettimeofday if needed.

Copy link

github-actions bot commented Dec 8, 2024

PR missing one of the required labels: {'documentation', 'dependencies', 'breaking-change', 'enhancement', 'internal', 'new feature', 'bug'}

@gabrik gabrik added the bug Something isn't working label Dec 9, 2024
@bjsowa bjsowa marked this pull request as draft December 9, 2024 09:54
@bjsowa
Copy link
Contributor Author

bjsowa commented Dec 9, 2024

I'm still testing some things, will mark this as ready for review once I'm done.

@bjsowa bjsowa force-pushed the fix/freertos-system-refactor branch from 44649a2 to 09dc516 Compare December 10, 2024 00:57
@bjsowa bjsowa marked this pull request as ready for review December 10, 2024 01:12
@milyin milyin merged commit 916ec33 into eclipse-zenoh:main Dec 10, 2024
64 checks passed
@bjsowa bjsowa deleted the fix/freertos-system-refactor branch December 10, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants