test

Yiling-J · Oct 1, 2024 · 3cf7507 · 3cf7507
1 parent f974188
commit 3cf7507
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/.github/workflows/go.yaml b/.github/workflows/go.yaml
@@ -15,7 +15,7 @@ jobs:
     name: test
     strategy:
       matrix:
-        go: ["1.19.x", "1.22.x", "1.23.x"]
+        go: ["1.22.x"]
     runs-on: macos-latest
     steps:
       - name: Setup Go
@@ -28,5 +28,5 @@ jobs:
       - name: Test
         run: go test ./... -run=TestNode_ -count=1
 
-      - name: Bench
-        run: go test -run='^$' -bench BenchmarkNode_ ./... -timeout=0
+      # - name: Bench
+      #   run: go test -run='^$' -bench BenchmarkNode_ ./... -timeout=0
diff --git a/seqlock/seq_test.go b/seqlock/seq_test.go
@@ -15,6 +15,7 @@ package node
 
 import (
 	"runtime"
+	"sync"
 	"sync/atomic"
 	"testing"
 	"time"
@@ -26,19 +27,19 @@ type Node[K comparable, V any] struct {
 	lock  atomic.Uint32
 }
 
+var q sync.Mutex
+
 func (n *Node[K, V]) Value() V {
-	var l atomic.Uint32
 	for {
 		seq := n.lock.Load()
-		l.Load()
 		if seq&1 != 0 {
 			runtime.Gosched()
 			continue
 		}
 
 		value := n.value
 
-		l.Add(0)
+		q.TryLock()
 		if seq == n.lock.Load() {
 			return value
 		}