fix: incorrect semaphore initialization (#411)

Consensys · Jun 15, 2023 · a3b568a · a3b568a
1 parent 4392bab
commit a3b568a
Show file tree

Hide file tree

Showing 11 changed files with 40 additions and 20 deletions.
diff --git a/ecc/bls12-377/multiexp.go b/ecc/bls12-377/multiexp.go
diff --git a/ecc/bls12-378/multiexp.go b/ecc/bls12-378/multiexp.go
diff --git a/ecc/bls12-381/multiexp.go b/ecc/bls12-381/multiexp.go
diff --git a/ecc/bls24-315/multiexp.go b/ecc/bls24-315/multiexp.go
diff --git a/ecc/bls24-317/multiexp.go b/ecc/bls24-317/multiexp.go
diff --git a/ecc/bn254/multiexp.go b/ecc/bn254/multiexp.go
diff --git a/ecc/bw6-633/multiexp.go b/ecc/bw6-633/multiexp.go
diff --git a/ecc/bw6-756/multiexp.go b/ecc/bw6-756/multiexp.go
diff --git a/ecc/bw6-761/multiexp.go b/ecc/bw6-761/multiexp.go
diff --git a/ecc/secp256k1/multiexp.go b/ecc/secp256k1/multiexp.go
diff --git a/internal/generator/ecc/template/multiexp.go.tmpl b/internal/generator/ecc/template/multiexp.go.tmpl
@@ -382,7 +382,8 @@ func _innerMsm{{ $.UPointName }}(p *{{ $.TJacobian }}, c uint64, points []{{ $.T
 	// (only if nbTasks < nbCPU)
 	var sem chan struct{}
 	if config.NbTasks < runtime.NumCPU() {
-		sem = make(chan struct{}, config.NbTasks * 2) // *2 because if chunk is overweight we split it in two
+		// we add nbChunks because if chunk is overweight we split it in two
+		sem = make(chan struct{}, config.NbTasks + int(nbChunks)) 
 		for i:=0; i < config.NbTasks; i++ {
 			sem <- struct{}{}
 		}