common: allow size alignment that is not a power of two

Do not assume the alignment is a power of two in the is_n_align_sized()
predicate. When used in the context of erasure code it is common
for chunks to not be powers of two.

Signed-off-by: Loic Dachary <loic@dachary.org>

diff --git a/src/include/buffer.h b/src/include/buffer.h
index 2e19a90e59f24eaeb96fcead2dff81733b1e2bb4..45e0f1dba95b97d52867dc3c68a143b4bfbd75e8 100644 (file)
--- a/src/include/buffer.h
+++ b/src/include/buffer.h
@@ -184,7 +184,7 @@ public:
     bool is_page_aligned() const { return is_aligned(CEPH_PAGE_SIZE); }
     bool is_n_align_sized(unsigned align) const
     {
-      return (length() & (align-1)) == 0;
+      return (length() % align) == 0;
     }
     bool is_n_page_sized() const { return is_n_align_sized(CEPH_PAGE_SIZE); }