Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

timers: float delays can cause 100% cpu usage. #8067

Closed
wants to merge 3 commits into from
Closed

timers: float delays can cause 100% cpu usage. #8067

wants to merge 3 commits into from

Commits on Aug 4, 2014

  1. timers: float delays can cause 100% cpu usage. 

    The changes in befbbad switch all comparisons over to unsigned
integer comparisons, but in certain cases list.msecs will be fractional.
If this is the case, the timer will never be removed from the list.
This fixes the issue by casting msecs to an integer by rounding up
or down.

Fixes 8065.
    @chrisdickinson
    chrisdickinson committed Aug 4, 2014
    Configuration menu
    Copy the full SHA
    785a03a View commit details
    Browse the repository at this point in the history

  2. move coercion further up

    @chrisdickinson
    chrisdickinson committed Aug 4, 2014
    Configuration menu
    Copy the full SHA
    9ca0343 View commit details
    Browse the repository at this point in the history

  3. add (probably insufficient) test

    @chrisdickinson
    chrisdickinson committed Aug 4, 2014
    Configuration menu
    Copy the full SHA
    fa479bd View commit details
    Browse the repository at this point in the history