From 2bc8e6fc9a18d46fe388d2c66753e7fdb9a372b8 Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Sat, 2 Aug 2014 15:22:37 -0400 Subject: [PATCH] runner: Take the number of workers as a flag. Default to the number of CPUs. Avoids the tests launching 64 valgrinds in parallel on machines without gobs of memory. Change-Id: I9eeb365b48aa7407e303d161f90ce69a591a884c Reviewed-on: https://boringssl-review.googlesource.com/1375 Reviewed-by: Adam Langley --- ssl/test/runner/runner.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ssl/test/runner/runner.go b/ssl/test/runner/runner.go index 07b1a127..7958a77b 100644 --- a/ssl/test/runner/runner.go +++ b/ssl/test/runner/runner.go @@ -9,6 +9,7 @@ import ( "net" "os" "os/exec" + "runtime" "strings" "sync" "syscall" @@ -856,6 +857,7 @@ func statusPrinter(doneChan chan struct{}, statusChan chan statusMsg, total int) func main() { var flagTest *string = flag.String("test", "", "The name of a test to run, or empty to run all tests") + var flagNumWorkers *int = flag.Int("num-workers", runtime.NumCPU(), "The number of workers to run in parallel.") flag.Parse() @@ -866,7 +868,7 @@ func main() { var wg sync.WaitGroup - const numWorkers = 64 + numWorkers := *flagNumWorkers statusChan := make(chan statusMsg, numWorkers) testChan := make(chan *testCase, numWorkers)