From 32ebe81cf3e222b41b03b774ce59bee2e7ec923f Mon Sep 17 00:00:00 2001 From: Dmitry Vyukov Date: Mon, 20 May 2019 18:53:12 +0200 Subject: pkg/repro: fix no output timeout We duplicated the no output timeout in the repro package, and it got out of sync. It's not 3 mins now, but 5 mins. Remove the duplication and fix this. --- pkg/repro/repro.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'pkg/repro') diff --git a/pkg/repro/repro.go b/pkg/repro/repro.go index 61793ac76..b3b8d739c 100644 --- a/pkg/repro/repro.go +++ b/pkg/repro/repro.go @@ -254,9 +254,9 @@ func (ctx *context) extractProg(entries []*prog.LogEntry) (*Result, error) { } // The shortest duration is 10 seconds to detect simple crashes (i.e. no races and no hangs). - // The longest duration is 5 minutes to catch races and hangs. Note that this value must be larger - // than hang/no output detection duration in vm.MonitorExecution, which is currently set to 3 mins. - timeouts := []time.Duration{10 * time.Second, 1 * time.Minute, 5 * time.Minute} + // The longest duration is 6 minutes to catch races and hangs. Note that this value must be larger + // than hang/no output detection duration in vm.MonitorExecution, which is currently set to 5 mins. + timeouts := []time.Duration{10 * time.Second, 1 * time.Minute, vm.NoOutputTimeout + time.Minute} for _, timeout := range timeouts { // Execute each program separately to detect simple crashes caused by a single program. -- cgit mrf-deployment