Merge pull request #272 from libp2p/clean-up-dial-sync

simplify the DialSync code

Author: marten-seemann

p2p/net/swarm/dial_sync.go

@@ -22,35 +22,26 @@ func newDialSync(worker dialWorkerFunc) *DialSync {
 // DialSync is a dial synchronization helper that ensures that at most one dial
 // to any given peer is active at any given time.
 type DialSync struct {
+	mutex      sync.Mutex
 	dials      map[peer.ID]*activeDial
-	dialsLk    sync.Mutex
 	dialWorker dialWorkerFunc
 }
 
 type activeDial struct {
-	id     peer.ID
 	refCnt int
 
 	ctx    context.Context
 	cancel func()
 
 	reqch chan dialRequest
-
-	ds *DialSync
 }
 
-func (ad *activeDial) decref() {
-	ad.ds.dialsLk.Lock()
-	ad.refCnt--
-	if ad.refCnt == 0 {
-		ad.cancel()
-		close(ad.reqch)
-		delete(ad.ds.dials, ad.id)
-	}
-	ad.ds.dialsLk.Unlock()
+func (ad *activeDial) close() {
+	ad.cancel()
+	close(ad.reqch)
 }
 
-func (ad *activeDial) dial(ctx context.Context, p peer.ID) (*Conn, error) {
+func (ad *activeDial) dial(ctx context.Context) (*Conn, error) {
 	dialCtx := ad.ctx
 
 	if forceDirect, reason := network.GetForceDirectDial(ctx); forceDirect {
@@ -76,30 +67,26 @@ func (ad *activeDial) dial(ctx context.Context, p peer.ID) (*Conn, error) {
 }
 
 func (ds *DialSync) getActiveDial(p peer.ID) (*activeDial, error) {
-	ds.dialsLk.Lock()
-	defer ds.dialsLk.Unlock()
+	ds.mutex.Lock()
+	defer ds.mutex.Unlock()
 
 	actd, ok := ds.dials[p]
 	if !ok {
 		// This code intentionally uses the background context. Otherwise, if the first call
 		// to Dial is canceled, subsequent dial calls will also be canceled.
 		// XXX: this also breaks direct connection logic. We will need to pipe the
 		// information through some other way.
-		adctx, cancel := context.WithCancel(context.Background())
+		ctx, cancel := context.WithCancel(context.Background())
 		actd = &activeDial{
-			id:     p,
-			ctx:    adctx,
+			ctx:    ctx,
 			cancel: cancel,
 			reqch:  make(chan dialRequest),
-			ds:     ds,
 		}
 		go ds.dialWorker(p, actd.reqch)
 		ds.dials[p] = actd
 	}
-
-	// increase ref count before dropping dialsLk
+	// increase ref count before dropping mutex
 	actd.refCnt++
-
 	return actd, nil
 }
 
@@ -111,6 +98,14 @@ func (ds *DialSync) Dial(ctx context.Context, p peer.ID) (*Conn, error) {
 		return nil, err
 	}
 
-	defer ad.decref()
-	return ad.dial(ctx, p)
+	defer func() {
+		ds.mutex.Lock()
+		defer ds.mutex.Unlock()
+		ad.refCnt--
+		if ad.refCnt == 0 {
+			ad.close()
+			delete(ds.dials, p)
+		}
+	}()
+	return ad.dial(ctx)
 }