Skip to content

sys/xtimer: remove redundant callback schedule for overflow#9489

Closed
Hyungsin wants to merge 1 commit intoRIOT-OS:masterfrom
Hyungsin:forupstream_xtimer_remove_redundancy
Closed

sys/xtimer: remove redundant callback schedule for overflow#9489
Hyungsin wants to merge 1 commit intoRIOT-OS:masterfrom
Hyungsin:forupstream_xtimer_remove_redundancy

Conversation

@Hyungsin
Copy link

@Hyungsin Hyungsin commented Jul 3, 2018

Contribution description

xtimer schedules a callback on next overflow (_lltimer_set(0xFFFFFFFF)) when there is no timer in the current timer period.

However, if there is no xtimer EVEN in the future timer period (i.e., literally zero on-going xtimer), this next overflow callback does not have to be scheduled.

This PR removes such redundant overflow callback schedules, which reduces energy/time resource consumption by scheduling overflow callbacks only when necessary.

Issues/PRs references

@Hyungsin Hyungsin force-pushed the forupstream_xtimer_remove_redundancy branch 7 times, most recently from 0eb8de5 to 08c60ea Compare July 3, 2018 19:43
@Hyungsin Hyungsin force-pushed the forupstream_xtimer_remove_redundancy branch from 08c60ea to 88711cc Compare July 3, 2018 19:46
@Hyungsin
Copy link
Author

Hyungsin commented Jul 9, 2018

closed in favor of #9530

@Hyungsin Hyungsin closed this Jul 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant