fix bugs

agent/cmd/main.go

@@ -20,7 +20,7 @@ func main() {
 	app := cli.NewApp()
 	app.Name = "sky_php_agent"
 	app.Usage = "the skywalking trace sending agent"
-	app.Version = "3.2.6"
+	app.Version = "3.2.7"
 	app.Flags = []cli.Flag{
 		&cli.StringSliceFlag{Name: "grpc", Usage: "SkyWalking collector grpc address", Value: cli.NewStringSlice("127.0.0.1:11800")},
 		&cli.StringFlag{Name: "socket", Usage: "Pipeline for communicating with PHP", Value: "/var/run/sky-agent.sock"},

agent/service/agent.go

@@ -5,8 +5,6 @@ import (
 	"agent/agent/pb/agent"
 	"agent/agent/pb/agent2"
 	"agent/agent/pb/register2"
-	"container/list"
-	"fmt"
 	cli "github.com/urfave/cli/v2"
 	"google.golang.org/grpc"
 	"math/rand"
@@ -38,19 +36,20 @@ type Agent struct {
 	socket            string
 	socketListener    net.Listener
 	register          chan *register
-	registerCache     sync.Map
-	registerCacheLock sync.Mutex
+	registerCache     map[int]registerCache
+	registerCacheLock sync.RWMutex
 	trace             chan string
-	queue             *list.List
+	queue             []string
+	queueLock         sync.Mutex
 }
 
 func NewAgent(cli *cli.Context) *Agent {
 	var agent = &Agent{
-		flag:     cli,
-		socket:   cli.String("socket"),
-		register: make(chan *register),
-		trace:    make(chan string),
-		queue:    list.New(),
+		flag:          cli,
+		socket:        cli.String("socket"),
+		register:      make(chan *register),
+		trace:         make(chan string),
+		registerCache: make(map[int]registerCache),
 	}
 
 	go agent.sub()
@@ -60,22 +59,25 @@ func NewAgent(cli *cli.Context) *Agent {
 
 func (t *Agent) Run() {
 	log.Info("hello skywalking")
-	t.connGRPC()
-	t.listenSocket()
 
-	log.Info("🍺 skywalking php agent started successfully, enjoy yourself")
 	defer func() {
 		var err error
-		err = t.socketListener.Close()
-		if err != nil {
-			log.Errorln(err)
+		if t.socketListener != nil {
+			err = t.socketListener.Close()
+			if err != nil {
+				log.Errorln(err)
+			}
 		}
 
-		err = t.grpcConn.Close()
-		if err != nil {
-			log.Errorln(err)
+		if t.grpcConn != nil {
+			err = t.grpcConn.Close()
+			if err != nil {
+				log.Errorln(err)
+			}
 		}
 	}()
+	t.connGRPC()
+	t.listenSocket()
 }
 
 func (t *Agent) connGRPC() {
@@ -99,6 +101,7 @@ func (t *Agent) connGRPC() {
 	t.grpcClient.segmentClientV6 = agent2.NewTraceSegmentReportServiceClient(t.grpcConn)
 	t.grpcClient.pingClient5 = agent.NewInstanceDiscoveryServiceClient(t.grpcConn)
 	t.grpcClient.pintClient6 = register2.NewServiceInstancePingClient(t.grpcConn)
+	log.Info("🍺 skywalking php agent started successfully, enjoy yourself")
 }
 
 func (t *Agent) listenSocket() {
@@ -144,17 +147,21 @@ func (t *Agent) sub() {
 	for {
 		select {
 		case <-traceSendTicker.C:
-			len := t.queue.Len()
+			len := len(t.queue)
 			if len > 0 {
 				var segments []*upstreamSegment
-				for i := 0; i < len; i++ {
-					// front top 100
-					e := t.queue.Front()
-					st := format(fmt.Sprintf("%v", e.Value))
+
+				t.queueLock.Lock()
+				list := t.queue[:]
+				t.queue = []string{}
+				t.queueLock.Unlock()
+
+				for _, trace := range list {
+					info, st := format(trace)
 					if st != nil {
+						t.recoverRegister(info)
 						segments = append(segments, st)
 					}
-					t.queue.Remove(e)
 				}
 				go t.send(segments)
 			}
@@ -163,8 +170,9 @@ func (t *Agent) sub() {
 		case register := <-t.register:
 			go t.doRegister(register)
 		case trace := <-t.trace:
-			t.queue.PushBack(trace)
-			go t.recoverRegister(trace)
+			t.queueLock.Lock()
+			t.queue = append(t.queue, trace)
+			t.queueLock.Unlock()
 		}
 	}
 }

agent/service/heartbeat.go

@@ -9,10 +9,10 @@ import (
 
 func (t *Agent) heartbeat() {
 
-	t.registerCache.Range(func(key, value interface{}) bool {
+	t.registerCacheLock.Lock()
+	defer t.registerCacheLock.Unlock()
+	for _, bind := range t.registerCache {
 		log.Infoln("heartbeat")
-		bind := value.(registerCache)
-
 		if bind.Version == 5 {
 			ctx, cancel := context.WithTimeout(context.Background(), time.Second*3)
 			defer cancel()
@@ -43,6 +43,5 @@ func (t *Agent) heartbeat() {
 				log.Infof("heartbeat appId %d appInsId %d", bind.AppId, bind.InstanceId)
 			}
 		}
-		return true
-	})
+	}
 }

agent/service/register.go

@@ -42,18 +42,21 @@ func ip4s() []string {
 	return ips
 }
 
-func (t *Agent) recoverRegister(r string) {
-	var info trace
-	err := json.Unmarshal([]byte(r), &info)
-	if err == nil {
-		if _, ok := t.registerCache.Load(info.Pid); !ok {
-			t.registerCache.Store(info.Pid, registerCache{
-				Version:    info.Version,
-				AppId:      info.ApplicationId,
-				InstanceId: info.ApplicationInstance,
-				Uuid:       info.Uuid,
-			})
+func (t *Agent) recoverRegister(info trace) {
+
+	t.registerCacheLock.RLock()
+	_, ok := t.registerCache[info.Pid]
+	t.registerCacheLock.RUnlock()
+
+	if !ok {
+		t.registerCacheLock.Lock()
+		t.registerCache[info.Pid] = registerCache{
+			Version:    info.Version,
+			AppId:      info.ApplicationId,
+			InstanceId: info.ApplicationInstance,
+			Uuid:       info.Uuid,
 		}
+		t.registerCacheLock.Unlock()
 	}
 }
 
@@ -68,8 +71,10 @@ func (t *Agent) doRegister(r *register) {
 	}
 
 	pid := info.Pid
-	if value, ok := t.registerCache.Load(pid); ok {
-		bind := value.(registerCache)
+	t.registerCacheLock.RLock()
+	bind, ok := t.registerCache[pid]
+	t.registerCacheLock.RUnlock()
+	if ok {
 		log.Infof("register => pid %d appid %d insId %d", pid, bind.AppId, bind.InstanceId)
 		r.c.Write([]byte(strconv.FormatInt(int64(bind.AppId), 10) + "," + strconv.FormatInt(int64(bind.InstanceId), 10) + "," + bind.Uuid))
 		return
@@ -79,9 +84,8 @@ func (t *Agent) doRegister(r *register) {
 
 	t.registerCacheLock.Lock()
 	defer t.registerCacheLock.Unlock()
-
 	// if map not found pid.. start register
-	if _, ok := t.registerCache.Load(pid); !ok {
+	if !ok {
 		log.Infof("start register pid %d used SkyWalking v%d", pid, info.Version)
 		var regAppStatus = false
 		var appId int32 = 0
@@ -254,12 +258,12 @@ func (t *Agent) doRegister(r *register) {
 			}
 
 			if appInsId != 0 {
-				t.registerCache.Store(pid, registerCache{
+				t.registerCache[pid] = registerCache{
 					Version:    info.Version,
 					AppId:      appId,
 					InstanceId: appInsId,
 					Uuid:       agentUUID,
-				})
+				}
 				log.Infof("register pid %d appid %d insId %d", pid, appId, appInsId)
 			}
 		} else {

agent/service/trace.go

@@ -123,13 +123,13 @@ func (t *Agent) send(segments []*upstreamSegment) {
 	log.Info("sending success...")
 }
 
-func format(j string) *upstreamSegment {
+func format(j string) (trace, *upstreamSegment) {
 	info := trace{}
 	err := json.Unmarshal([]byte(j), &info)
 
 	if err != nil {
 		log.Error("trace json decode:", err)
-		return nil
+		return info, nil
 	}
 	if info.Version == 5 {
 		var globalTrace []*agent.UniqueId
@@ -190,14 +190,14 @@ func format(j string) *upstreamSegment {
 		//log.Info(seg)
 		if err != nil {
 			log.Error("trace json encode:", err)
-			return nil
+			return info, nil
 		}
 
 		segment := &agent.UpstreamSegment{
 			GlobalTraceIds: globalTrace,
 			Segment:        seg,
 		}
-		return &upstreamSegment{
+		return info, &upstreamSegment{
 			Version: info.Version,
 			segment: segment,
 		}
@@ -261,7 +261,7 @@ func format(j string) *upstreamSegment {
 		//log.Info(seg)
 		if err != nil {
 			log.Error("trace proto encode:", err)
-			return nil
+			return info, nil
 		}
 
 		segment := &agent.UpstreamSegment{
@@ -269,12 +269,12 @@ func format(j string) *upstreamSegment {
 			Segment:        seg,
 		}
 
-		return &upstreamSegment{
+		return info, &upstreamSegment{
 			Version: info.Version,
 			segment: segment,
 		}
 	}
-	return nil
+	return info, nil
 }
 
 func buildRefs(span *agent.SpanObject, refs []ref) {

php_skywalking.h

@@ -25,7 +25,7 @@ extern zend_module_entry skywalking_module_entry;
 #define phpext_skywalking_ptr &skywalking_module_entry
 
 #define SKY_DEBUG 0
-#define PHP_SKYWALKING_VERSION "3.2.6" /* Replace with version number for your extension */
+#define PHP_SKYWALKING_VERSION "3.2.7" /* Replace with version number for your extension */
 
 #ifdef PHP_WIN32
 #	define PHP_SKYWALKING_API __declspec(dllexport)