commit e817a2a25d8e1f87490d050c6d1ed293f0c81fd3
author Jan Schär <jan@monogon.tech> Thu Mar 06 17:46:25 2025 +0100
committer Jan Schär <jan@monogon.tech> Tue Mar 18 14:02:05 2025 +0000
tree 2abb5abe8adc173ef8c5582cde4538b01b0d0c90
parent b00f7f9a97eae55ae6df80bbdea46815498898fa

osbase/fsquota: replace float calculation

Floats have limited precision and are not needed here.

Change-Id: I51053821af226d312e94d51c4d21224e1ca53908
Reviewed-on: https://review.monogon.dev/c/monogon/+/4009
Tested-by: Jenkins CI
Reviewed-by: Lorenz Brun <lorenz@monogon.tech>

osbase/fsquota/fsquota.go

diff --git a/osbase/fsquota/fsquota.go b/osbase/fsquota/fsquota.go
index 4919841..e4f3e92 100644
--- a/osbase/fsquota/fsquota.go
+++ b/osbase/fsquota/fsquota.go
@@ -11,7 +11,6 @@
 import (
 	"errors"
 	"fmt"
-	"math"
 	"os"
 
 	"golang.org/x/sys/unix"
@@ -84,7 +83,10 @@
 	}
 
 	// Always round up to the nearest block size
-	bytesLimitBlocks := uint64(math.Ceil(float64(maxBytes) / float64(1024)))
+	bytesLimitBlocks := maxBytes / 1024
+	if bytesLimitBlocks*1024 < maxBytes {
+		bytesLimitBlocks += 1
+	}
 
 	return quotactl.SetQuota(dir, quotactl.QuotaTypeProject, lastID, &quotactl.Quota{
 		BHardLimit: bytesLimitBlocks,