From: Patrick Donnelly <pdonnell@ibm.com>
Date: Sat, 30 May 2026 16:16:19 +0000 (-0400)
Subject: SimpleRADOSStriper: fix incorrect variable usage in allocation growth calculation
X-Git-Tag: testing/wip-pdonnell-testing-20260605.140416-main~1^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=a24fccd1242a13e433666d7cab0351262b09dbea;p=ceph-ci.git

SimpleRADOSStriper: fix incorrect variable usage in allocation growth calculation

In set_metadata(), the calculation for new_allocated was incorrectly
applying the base-2 rounding mask to the old size variable instead of
the requested new_size. This commit fixes the logic to use the correct
target size when determining the new boundaries.

Fixes: https://tracker.ceph.com/issues/77009
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
---

diff --git a/src/SimpleRADOSStriper.cc b/src/SimpleRADOSStriper.cc
index c0000a5d017..cc51b1537e6 100644
--- a/src/SimpleRADOSStriper.cc
+++ b/src/SimpleRADOSStriper.cc
@@ -397,7 +397,7 @@ int SimpleRADOSStriper::set_metadata(uint64_t new_size, bool update_size)
   auto op = librados::ObjectWriteOperation();
   if (new_size > allocated) {
     uint64_t mask = (1<<object_size)-1;
-    new_allocated = min_growth + ((size + mask) & ~mask); /* round up base 2 */
+    new_allocated = min_growth + ((new_size + mask) & ~mask); /* round up base 2 */
     op.setxattr(XATTR_ALLOCATED, uint2bl(new_allocated));
     do_op = true;
     if (logger) logger->inc(P_UPDATE_ALLOCATED);