]> git.saurik.com Git - redis.git/commit
Force expire all timer events when system clock skew is detected.
authorJokea <tfengjun@gmail.com>
Thu, 30 Aug 2012 07:08:19 +0000 (15:08 +0800)
committerantirez <antirez@gmail.com>
Thu, 4 Oct 2012 17:30:48 +0000 (19:30 +0200)
commite480c279bc28832ca78a33f7f52c0b57ae5a16f1
treef0f1e5953c98dfca6dca7d3b8dc82b45a64cf9db
parent0c19880ce61bb4516346288a33797ec650e13842
Force expire all timer events when system clock skew is detected.

When system time changes back, the timer will not worker properly
hence some core functionality of redis will stop working(e.g. replication,
bgsave, etc). See issue #633 for details.

The patch saves the previous time and when a system clock skew is detected,
it will force expire all timers.

Modiifed by @antirez: the previous time was moved into the eventLoop
structure to make sure the library is still thread safe as long as you
use different event loops into different threads (otherwise you need
some synchronization). More comments added about the reasoning at the
base of the patch, that's worth reporting here:

/* If the system clock is moved to the future, and then set back to the
 * right value, time events may be delayed in a random way. Often this
 * means that scheduled operations will not be performed soon enough.
 *
 * Here we try to detect system clock skews, and force all the time
 * events to be processed ASAP when this happens: the idea is that
 * processing events earlier is less dangerous than delaying them
 * indefinitely, and practice suggests it is. */
src/ae.c
src/ae.h