-
Notifications
You must be signed in to change notification settings - Fork 7
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
Eliminate race conditions #23
Comments
We should first add a mutex SPI to be able to run either with Raritan framework mutexes of C++11 mutexes. |
One race condition here: |
In ezsp-dongle.cpp: |
Fixes done on branch https://github.com/Legrandgroup/libezsp/tree/thread-safety are part of solving this. |
Implemented in PR #76 |
When using threads for timer or libserialcpp SPI (that is internally using threads also), there may be additional race conditions that are not usually hit if running a mainloop (eg: on Raritan framework).
Serial read/write coud can be run simultaneously and race conditions must be handled.
Timer elapsing are implemented using threads when not running on the Raritan mainloop and the actions for the code executed in timers should be protected using mutexes.
The text was updated successfully, but these errors were encountered: