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

Optimize scheduler #19

Closed
OlegHahm opened this issue Mar 1, 2013 · 4 comments · Fixed by #1306
Closed

Optimize scheduler #19

OlegHahm opened this issue Mar 1, 2013 · 4 comments · Fixed by #1306
Labels
State: archived State: The PR has been archived for possible future re-adaptation Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Comments

@OlegHahm
Copy link
Member

OlegHahm commented Mar 1, 2013

The scheduler - particular the function sched_run() - comprises some possibly unnecessary checks for null pointer and so on.

Additionally, some functionality is only need with certain modules (like NSS) activated and could be excluded (using precompiler instructions) if theses modules are not active.
OlegHahm referenced this issue in OlegHahm/RIOT Nov 6, 2013
@OlegHahm
Merge pull request #19 from OlegHahm/msb430_uart
dc5e7d7 
Msb430 uart
@Kijewski
Copy link
Contributor

Is this issue still valid?

@OlegHahm
Copy link
Member Author

I guess it is.

@miri64 miri64 mentioned this issue Mar 4, 2014
Merged
@OlegHahm OlegHahm added this to the FIX ME FIRST milestone Jun 3, 2014
@Kijewski Kijewski mentioned this issue Jun 10, 2014
Merged
@Kijewski
Copy link
Contributor

Please see #1306.

@OlegHahm OlegHahm added the memo label Jun 18, 2014
@OlegHahm
Copy link
Member Author

I think there are still some unnecessary checks, but let's leave this as a memo.

@miri64 miri64 mentioned this issue Jan 7, 2016
Merged
murdock-ci pushed a commit to murdock-ci/RIOT that referenced this issue Mar 28, 2017
@OlegHahm
Merge pull request RIOT-OS#19 from miri64/cppcheck/fix/pr6703-2
b168d39 
More fixes to RIOT-OS#6703
riot-ci pushed a commit to riot-ci/RIOT that referenced this issue Dec 5, 2018
@cladmi
Merge pull request RIOT-OS#19 from fedepell/rb-flasher-deps
7fe6a38 
riotboot: account for flasher building dependencies before trying to use it
CW-75 pushed a commit to CW-75/RIOT that referenced this issue Jul 1, 2022
@luisan00
Digital temperature sensor DS18B20 (RIOT-OS#19)
51a3086 
tests/driver_ds18: add test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
State: archived State: The PR has been archived for possible future re-adaptation Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

core: remove unnecessary tests in sched.c
2 participants
@Kijewski @OlegHahm