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.

Sign up for GitHub

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

Jump to bottom

Race condition in hwtimer_wait() #25

Closed
OlegHahm opened this issue Mar 13, 2013 · 1 comment
Closed

Race condition in hwtimer_wait() #25

OlegHahm opened this issue Mar 13, 2013 · 1 comment
Assignees
@OlegHahm
Labels
Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Milestone
Release 2013.08

Comments

@OlegHahm
Copy link
Member

The function hwtimer_wait() internally calls hwtimer_set() to schedule a thread_wakeup() and then calls thread_sleep(), If an IRQ occurs before thread_sleep() gets called, this thread might never wake up again. One solution could be the usage of mutexes instead (as in former µkleos/FeuerWare code).
@OlegHahm OlegHahm mentioned this issue Mar 13, 2013
Closed
@ghost ghost assigned OlegHahm Aug 4, 2013
OlegHahm referenced this issue in OlegHahm/RIOT Aug 8, 2013
@OlegHahm
fix for #25 by using mutexes for hwtimer_wait
3f95d72
miri64 added a commit that referenced this issue Aug 11, 2013
@miri64
Merge pull request #105 from OlegHahm/hwtimer_wait
78db093 
fix for #25 by using mutexes for hwtimer_wait
@OlegHahm
Copy link
Member Author

Should be fixed by #105.

@thomaseichinger thomaseichinger mentioned this issue Aug 23, 2013
Merged
OlegHahm referenced this issue in OlegHahm/RIOT Nov 6, 2013
@OlegHahm
Merge pull request #25 from thomaseichinger/ADC
fee5ad9 
GPIO implementation was moved to RIOT/cpu/mc1322x
@OlegHahm OlegHahm mentioned this issue Jan 23, 2014
Closed
riot-ci pushed a commit to riot-ci/RIOT that referenced this issue May 24, 2019
@miri64
minmea: bump version to current master
6273b30 
$ git log --oneline dbf51f03c5058746b4c4b3d6fdcd3d0357c5da36..HEAD
ae4dd94 Merge pull request RIOT-OS#38 from noahp/noahp/check-url
b1371aa 🔗 update Check link
65bf740 Merge pull request RIOT-OS#35 from gdpinchina/master
61c6ef7 fix test for gga
4514f6d fix test for gga
113221d fix test for gga
35b00bc fix test in gga
9fcd245 remove indent
59c3e0a fix gga char alignment problem in test.c
79b964a modefy initialization for dgps_age
c4a5a01 modify dgps_age type
c01cbcf modify dgps_age type
cd27e72 Merge pull request RIOT-OS#29 from swilson/master
4d97109 Fix compiler warnings.
0456bbb Define timegm
04f1141 Add TI-RTOS compat header
8e4fc4e README.md: formatting fixes
0b02771 minmea.h: add Windows compatibility layer
ded4faf Merge pull request RIOT-OS#25 from asund/master
f50449e Fixup whitespace
80af9b4 * Arrange year interpretation in order of likeliness * Add unit tests
ef8bca7 clarity of century use in struct tm
486da2b Small fixes to minmea_gettime * Handle four digit years provided by ZDA in minmea_gettime * Assume GPS epoch to allow dates prior 2000 to be parsed
fjmolinas pushed a commit to fjmolinas/RIOT that referenced this issue Jun 5, 2019
@miri64 @fjmolinas
minmea: bump version to current master
7c13996 
$ git log --oneline dbf51f03c5058746b4c4b3d6fdcd3d0357c5da36..HEAD
ae4dd94 Merge pull request RIOT-OS#38 from noahp/noahp/check-url
b1371aa 🔗 update Check link
65bf740 Merge pull request RIOT-OS#35 from gdpinchina/master
61c6ef7 fix test for gga
4514f6d fix test for gga
113221d fix test for gga
35b00bc fix test in gga
9fcd245 remove indent
59c3e0a fix gga char alignment problem in test.c
79b964a modefy initialization for dgps_age
c4a5a01 modify dgps_age type
c01cbcf modify dgps_age type
cd27e72 Merge pull request RIOT-OS#29 from swilson/master
4d97109 Fix compiler warnings.
0456bbb Define timegm
04f1141 Add TI-RTOS compat header
8e4fc4e README.md: formatting fixes
0b02771 minmea.h: add Windows compatibility layer
ded4faf Merge pull request RIOT-OS#25 from asund/master
f50449e Fixup whitespace
80af9b4 * Arrange year interpretation in order of likeliness * Add unit tests
ef8bca7 clarity of century use in struct tm
486da2b Small fixes to minmea_gettime * Handle four digit years provided by ZDA in minmea_gettime * Assume GPS epoch to allow dates prior 2000 to be parsed
geromueller pushed a commit to geromueller/RIOT that referenced this issue Jun 19, 2019
@miri64 @geromueller
minmea: bump version to current master
afc1a59 
$ git log --oneline dbf51f03c5058746b4c4b3d6fdcd3d0357c5da36..HEAD
ae4dd94 Merge pull request RIOT-OS#38 from noahp/noahp/check-url
b1371aa 🔗 update Check link
65bf740 Merge pull request RIOT-OS#35 from gdpinchina/master
61c6ef7 fix test for gga
4514f6d fix test for gga
113221d fix test for gga
35b00bc fix test in gga
9fcd245 remove indent
59c3e0a fix gga char alignment problem in test.c
79b964a modefy initialization for dgps_age
c4a5a01 modify dgps_age type
c01cbcf modify dgps_age type
cd27e72 Merge pull request RIOT-OS#29 from swilson/master
4d97109 Fix compiler warnings.
0456bbb Define timegm
04f1141 Add TI-RTOS compat header
8e4fc4e README.md: formatting fixes
0b02771 minmea.h: add Windows compatibility layer
ded4faf Merge pull request RIOT-OS#25 from asund/master
f50449e Fixup whitespace
80af9b4 * Arrange year interpretation in order of likeliness * Add unit tests
ef8bca7 clarity of century use in struct tm
486da2b Small fixes to minmea_gettime * Handle four digit years provided by ZDA in minmea_gettime * Assume GPS epoch to allow dates prior 2000 to be parsed
eduazocar pushed a commit to eduazocar/RIOT that referenced this issue Jul 1, 2022
@luisan00
Merge pull request RIOT-OS#25 from Mesh4all/feature/ci_tests
fc324eb 
Add static test
chrysn pushed a commit to chrysn-pull-requests/RIOT that referenced this issue Sep 16, 2022
@OlegHahm
Merge pull request RIOT-OS#25 from cgundogan/pr/openwsn/remove_vtimer
f1d1223 
openwsn: remove vtimer, use xtimer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OlegHahm OlegHahm

Labels
Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
Release 2013.08
Development

No branches or pull requests
1 participant
@OlegHahm