fix: some fixes

perf: change to dirt make
fix: remove empty slice
chore: use syscall in x/sys/unix

Author: titanxxh

.github/workflows/pre_check.yaml

@@ -4,7 +4,7 @@ on: [ pull_request ]
 
 jobs:
   compliant:
-    runs-on: [ self-hosted, X64 ]
+    runs-on: [ self-hosted, X64, Linux ]
     steps:
       - uses: actions/checkout@v3
 
@@ -16,29 +16,18 @@ jobs:
       - name: Check Spell
         uses: crate-ci/typos@v1.13.14
 
-  staticcheck:
-    runs-on: [ self-hosted, X64 ]
+  # 用一个新的、更强大的 linter 作业替换 staticcheck
+  linter:
+    runs-on: [ self-hosted, X64, Linux ]
     steps:
       - uses: actions/checkout@v3
       - name: Set up Go
         uses: actions/setup-go@v3
         with:
-          go-version: 1.19
-
-      - uses: actions/cache@v3
-        with:
-          path: ~/go/pkg/mod
-          key: reviewdog-${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
-          restore-keys: |
-            reviewdog-${{ runner.os }}-go-
-      - uses: reviewdog/action-staticcheck@v1
+          go-version: 1.20
+      - name: Run golangci-lint
+        # 使用官方的 golangci-lint-action
+        uses: golangci/golangci-lint-action@v6
         with:
-          github_token: ${{ secrets.github_token }}
-          # Change reviewdog reporter if you need [github-pr-check,github-check,github-pr-review].
-          reporter: github-pr-review
-          # Report all results.
-          filter_mode: added
-          # Exit with 1 when it find at least one finding.
-          fail_on_error: true
-          # Set staticcheck flags
-          staticcheck_flags: -checks=inherit,-SA1029,-SA2002,-SA4006,-SA2002,-SA9003,-S1024 -exclude=example
+          # 指定要使用的 golangci-lint 版本
+          version: v2.5.0

.github/workflows/tests.yaml

@@ -10,34 +10,37 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v3
         with:
-          go-version: '1.17'
+          go-version: '1.20'
       - name: Unit Test
         run: go test -gcflags=-l -covermode=atomic -coverprofile=coverage.txt
       - name: Codecov
         run: bash <(curl -s https://codecov.io/bash)
 
   benchmark-test:
+    needs: unit-scenario-test
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
       - name: Set up Go
         uses: actions/setup-go@v3
         with:
-          go-version: '1.17'
+          go-version: '1.20'
       - name: Benchmark
         run: go test -gcflags='all=-N -l' -bench=. -benchmem -run=none
 
   compatibility-test:
+    needs: benchmark-test
     strategy:
+      max-parallel: 1
       matrix:
-        go: [1.13, 1.14, 1.15, 1.16, 1.18, 1.19, "1.20" ]
+        go: [ 1.21, 1.22, 1.23, 1.24, 1.25 ]
         os: [ X64 ]
-    runs-on: ${{ matrix.os }}
+    runs-on: [ Linux, '${{ matrix.os }}' ]
     steps:
       - uses: actions/checkout@v3
       - name: Set up Go
         uses: actions/setup-go@v3
         with:
           go-version: ${{ matrix.go }}
       - name: Unit Test
-        run: go test -gcflags=-l -covermode=atomic
+        run: go test -gcflags=-l -covermode=atomic

.gitignore

@@ -1 +1,2 @@
 .idea
+coverage.txt

bench_test.go

@@ -18,13 +18,13 @@ package shmipc
 
 import (
 	"fmt"
+	syscall "golang.org/x/sys/unix"
 	"io"
 	"math/rand"
 	"net"
 	"net/http"
 	_ "net/http/pprof"
 	"strconv"
-	"syscall"
 	"testing"
 	"time"
 )

block_io.go

@@ -18,8 +18,8 @@ package shmipc
 
 import (
 	"fmt"
+	syscall "golang.org/x/sys/unix"
 	"io"
-	"syscall"
 )
 
 func blockReadFull(connFd int, data []byte) error {

buffer.go

@@ -21,6 +21,8 @@ import (
 	"sync"
 	"sync/atomic"
 	"unsafe"
+
+	"github.com/bytedance/gopkg/lang/dirtmake"
 )
 
 var (
@@ -333,7 +335,7 @@ func (l *linkedBuffer) ReadBytes(size int) (result []byte, err error) {
 	}
 	// slow path
 	l.len -= size
-	result = make([]byte, 0, size)
+	result = dirtmake.Bytes(0, size)
 
 	for size > 0 {
 		readData, _ := l.sliceList.front().read(size)

buffer_manager.go

@@ -19,12 +19,12 @@ package shmipc
 import (
 	"errors"
 	"fmt"
+	syscall "golang.org/x/sys/unix"
 	"os"
 	"path/filepath"
 	"sort"
 	"sync"
 	"sync/atomic"
-	"syscall"
 	"time"
 	"unsafe"
 )

epoll_linux.go

@@ -1,3 +1,4 @@
+//go:build !arm64
 // +build !arm64
 
 /*
@@ -19,11 +20,12 @@
 package shmipc
 
 import (
-	"syscall"
+	syscall "golang.org/x/sys/unix"
+
 	"unsafe"
 )
 
-const epollModeET = -syscall.EPOLLET
+const epollModeET = syscall.EPOLLET
 
 type epollEvent struct {
 	events uint32

epoll_linux_arm64.go

@@ -17,7 +17,7 @@
 package shmipc
 
 import (
-	"syscall"
+	syscall "golang.org/x/sys/unix"
 	"unsafe"
 )
 

event_dispatcher_linux.go

@@ -1,3 +1,4 @@
+//go:build !race
 // +build !race
 
 /*
@@ -20,11 +21,11 @@ package shmipc
 
 import (
 	"fmt"
+	syscall "golang.org/x/sys/unix"
 	"os"
 	"runtime"
 	"sync"
 	"sync/atomic"
-	"syscall"
 	"unsafe"
 )
 
@@ -74,7 +75,7 @@ func (c *connEventHandler) onRemoteClose() {
 	c.deferredClose()
 }
 
-//avoiding write concurrently, blocking until return
+// avoiding write concurrently, blocking until return
 func (c *connEventHandler) writev(data ...[]byte) error {
 	if len(data) == 0 {
 		return nil
@@ -135,7 +136,7 @@ func (c *connEventHandler) doWritev(data ...[]byte) (int, error) {
 	return writtenSliceNum, nil
 }
 
-//avoiding write concurrently, blocking until return
+// avoiding write concurrently, blocking until return
 func (c *connEventHandler) write(data []byte) error {
 	written, size := 0, len(data)
 	for written < size {