From: Lynn Boger Date: Wed, 12 Dec 2018 21:18:19 +0000 (-0500) Subject: os/signal: increase wait time for signal delivery time in testcase X-Git-Tag: go1.12beta1~74 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=5dc144c607cb355e9f4a01bfa0b8f054899156a2;p=gostls13.git os/signal: increase wait time for signal delivery time in testcase This increases the time to wait from 1 to 2 seconds in the TestAtomicStop testcase. When running with gccgo on ppc64 & ppc64le on a loaded systems these testcases can intermittently fail with the current value. Updates #29046 Change-Id: If420274dd65926d933a3024903b5c757c300bd60 Reviewed-on: https://go-review.googlesource.com/c/153826 Run-TryBot: Lynn Boger TryBot-Result: Gobot Gobot Reviewed-by: Ian Lance Taylor --- diff --git a/src/os/signal/signal_test.go b/src/os/signal/signal_test.go index ecb05fd16c..6ea59f4697 100644 --- a/src/os/signal/signal_test.go +++ b/src/os/signal/signal_test.go @@ -432,12 +432,12 @@ func atomicStopTestProgram() { // At this point we should either die from SIGINT or // get a notification on cs. If neither happens, we - // dropped the signal. Give it a second to deliver, - // which is far far longer than it should require. + // dropped the signal. It is given 2 seconds to + // deliver, as needed for gccgo on some loaded test systems. select { case <-cs: - case <-time.After(1 * time.Second): + case <-time.After(2 * time.Second): if !printed { fmt.Print("lost signal on tries:") printed = true