diff options
| author | Michael Pratt <mpratt@google.com> | 2020-12-01 17:24:33 -0500 |
|---|---|---|
| committer | Michael Pratt <mpratt@google.com> | 2020-12-03 21:21:23 +0000 |
| commit | b635e4b808bf45ebd66e9f687e18b9af6bd634c1 (patch) | |
| tree | a4568a92d6cb9ab0afd7ec8f28433a636d260529 /src/runtime/time.go | |
| parent | 4eb7ceba067cf4f3851f2eaf63c9929386594adf (diff) | |
| download | go-b635e4b808bf45ebd66e9f687e18b9af6bd634c1.tar.gz go-b635e4b808bf45ebd66e9f687e18b9af6bd634c1.zip | |
time, runtime: don't set timer when = 0
timer when == 0, in the context of timer0When and timerModifiedEarliest,
is a sentinel value meaning there are no timers on the heap.
TestCheckRuntimeTimerOverflow reaching into the runtime to set a timer
to when = 0 when it is otherwise not possible breaks this invariant.
After golang.org/cl/258303, we will no longer detect and run this timer,
thus blocking any other timers lower on the heap from running. This
manifests as random timers failing to fire in other tests.
The need to set this overflowed timer to when = 0 is gone with the old
timer proc implementation, so we can simply remove it.
Fixes #42424
Change-Id: Iea32100136ad8ec1bedfa77b1e7d9ed868812838
Reviewed-on: https://go-review.googlesource.com/c/go/+/274632
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Michael Pratt <mpratt@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Trust: Michael Pratt <mpratt@google.com>
Diffstat (limited to 'src/runtime/time.go')
| -rw-r--r-- | src/runtime/time.go | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/runtime/time.go b/src/runtime/time.go index 75b66f8492..83d93c5686 100644 --- a/src/runtime/time.go +++ b/src/runtime/time.go @@ -23,6 +23,8 @@ type timer struct { // Timer wakes up at when, and then at when+period, ... (period > 0 only) // each time calling f(arg, now) in the timer goroutine, so f must be // a well-behaved function and not block. + // + // when must be positive on an active timer. when int64 period int64 f func(interface{}, uintptr) |