Merge pull request #917 from jakobmoellerdev/s390x-arch

fix: reallow building for s390x

.github/workflows/main.yaml

@@ -18,6 +18,8 @@ jobs:
       - run: make setup
       - run: make check-uncommitted
       - run: make test
+      - run: make build-topolvm GOARCH=s390x
+        name: "Build TopoLVM for s390x architecture"
       - run: make groupname-test
 
   build-images:

README.md

@@ -15,13 +15,14 @@ Our supported platform are:
 
 - Kubernetes: 1.29, 1.28, 1.27
 - Node OS: Linux with LVM2 (\*1), specifically tested on Ubuntu
-- CPU Architecture: x86 (\*2), arm64 (\*3), ppc64le (\*3)
+- CPU Architecture: x86 (\*2), arm64 (\*3), ppc64le (\*3), s390x (\*4)
 - Filesystems: ext4, xfs, btrfs(experimental)
 - lvm version 2.02.163 or later (adds JSON output support)
 
 \*1 The host's Linux Kernel must be v4.9 or later which supports `rmapbt` and `reflink`, if you use xfs filesystem with an official docker image.  
 \*2 Tier1 support. The official docker images are provided and all functionalities are tested by CI.  
 \*3 Tier2 support. The official docker images are provided, but no tests run by CI.  
+\*4 Tier3 support. No docker images are provided, and no tests run by CI, but binaries are built.
 
 Docker images are available on [ghcr.io](https://github.com/orgs/topolvm/packages).  
 

internal/driver/node.go

@@ -434,11 +434,12 @@ func (s *nodeServerNoLocked) NodeGetVolumeStats(ctx context.Context, req *csi.No
 
 	var usage []*csi.VolumeUsage
 	if sfs.Blocks > 0 {
+		//nolint:unconvert // explicit conversion of Frsize for s390x.
 		usage = append(usage, &csi.VolumeUsage{
 			Unit:      csi.VolumeUsage_BYTES,
-			Total:     int64(sfs.Blocks) * sfs.Frsize,
-			Used:      int64(sfs.Blocks-sfs.Bfree) * sfs.Frsize,
-			Available: int64(sfs.Bavail) * sfs.Frsize,
+			Total:     int64(sfs.Blocks) * int64(sfs.Frsize),
+			Used:      int64(sfs.Blocks-sfs.Bfree) * int64(sfs.Frsize),
+			Available: int64(sfs.Bavail) * int64(sfs.Frsize),
 		})
 	}
 	if sfs.Files > 0 {