volume.tier.upload progress starts negative #2992

fix https://github.com/chrislusf/seaweedfs/issues/2992

weed/storage/backend/s3_backend/s3_upload.go

@@ -2,12 +2,11 @@ package s3_backend
 
 import (
 	"fmt"
-	"os"
-	"sync/atomic"
-
 	"github.com/aws/aws-sdk-go/aws"
 	"github.com/aws/aws-sdk-go/service/s3/s3iface"
 	"github.com/aws/aws-sdk-go/service/s3/s3manager"
+	"os"
+	"sync"
 
 	"github.com/chrislusf/seaweedfs/weed/glog"
 )
@@ -42,7 +41,7 @@ func uploadToS3(sess s3iface.S3API, filename string, destBucket string, destKey
 	fileReader := &s3UploadProgressedReader{
 		fp:      f,
 		size:    fileSize,
-		read: -fileSize,
+		signMap: map[int64]struct{}{},
 		fn:      fn,
 	}
 
@@ -65,10 +64,13 @@ func uploadToS3(sess s3iface.S3API, filename string, destBucket string, destKey
 }
 
 // adapted from https://github.com/aws/aws-sdk-go/pull/1868
+// https://github.com/aws/aws-sdk-go/blob/main/example/service/s3/putObjectWithProcess/putObjWithProcess.go
 type s3UploadProgressedReader struct {
 	fp      *os.File
 	size    int64
 	read    int64
+	signMap map[int64]struct{}
+	mux     sync.Mutex
 	fn      func(progressed int64, percentage float32) error
 }
 
@@ -82,8 +84,14 @@ func (r *s3UploadProgressedReader) ReadAt(p []byte, off int64) (int, error) {
 		return n, err
 	}
 
-	// Got the length have read( or means has uploaded), and you can construct your message
+	r.mux.Lock()
-	atomic.AddInt64(&r.read, int64(n))
+	// Ignore the first signature call
+	if _, ok := r.signMap[off]; ok {
+		r.read += int64(n)
+	} else {
+		r.signMap[off] = struct{}{}
+	}
+	r.mux.Unlock()
 
 	if r.fn != nil {
 		read := r.read