fix: fix backing image size calculation

ref: longhorn/longhorn 10342 Signed-off-by: Jack Lin <[email protected]>
longhorn · Feb 5, 2025 · b1bf0f6 · b1bf0f6
1 parent 6e966aa
commit b1bf0f6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pkg/spdk/backing_image.go b/pkg/spdk/backing_image.go
@@ -131,7 +131,7 @@ func (bi *BackingImage) Create(spdkClient *spdkclient.Client, superiorPortAlloca
 		if err != nil {
 			bi.log.WithError(err).Error("Failed to create backing image")
 		}
-		// update the backing image afte preparing
+		// update the backing image after preparing
 		bi.UpdateCh <- nil
 	}()
 

diff --git a/pkg/spdk/server.go b/pkg/spdk/server.go
@@ -286,7 +286,7 @@ func (s *Server) verify() (err error) {
 				logrus.WithError(err).Warnf("failed to extract backing image name and disk UUID from lvol name %v", lvolName)
 				continue
 			}
-			actualSize := bdevLvol.DriverSpecific.Lvol.NumAllocatedClusters * uint64(defaultClusterSize)
+			size := bdevLvol.NumBlocks * uint64(bdevLvol.BlockSize)
 			alias := bdevLvol.Aliases[0]
 			expectedChecksum, err := GetSnapXattr(spdkClient, alias, types.BackingImageSnapshotAttrChecksum)
 			if err != nil {
@@ -298,7 +298,7 @@ func (s *Server) verify() (err error) {
 				logrus.WithError(err).Warnf("failed to retrieve backing image UUID attribute for snapshot %v", alias)
 				continue
 			}
-			backingImage := NewBackingImage(s.ctx, backingImageName, backingImageUUID, lvsUUID, actualSize, expectedChecksum, s.updateChs[types.InstanceTypeBackingImage])
+			backingImage := NewBackingImage(s.ctx, backingImageName, backingImageUUID, lvsUUID, size, expectedChecksum, s.updateChs[types.InstanceTypeBackingImage])
 			backingImage.Alias = alias
 			// For uncahced backing image, we set the state to pending first, so we can distinguish it from the cached but starting backing image
 			backingImage.State = types.BackingImageStatePending