Skip to contentSkip to conversation

btrfs: use IS_ALIGNED() for assertion in btrfs_lookup_csums_range() for simplicity

PATCH New Satoru Takeuchi 1 participant 1 message Jul 25, 2014 Btrfs file system

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 btrfs: use IS_ALIGNED() for assertion in btrfs_lookup_csums_range() for simplicity

From: Satoru Takeuchi <[email protected]>

btrfs_lookup_csums_range() uses ALIGN() to check if "start" and "end + 1" are aligned to "root->sectorsize". It's better to replace these with IS_ALIGNED() for simplicity.

---
 fs/btrfs/file-item.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
fs/btrfs/file-item.c +2 -2
index f46cfe4..4b9e40f 100644
@@ -329,8 +329,8 @@ int btrfs_lookup_csums_range(struct btrfs_root *root, u64 start, u64 end,
 	u64 csum_end;
 	u16 csum_size = btrfs_super_csum_size(root->fs_info->super_copy);
 
-	ASSERT(start == ALIGN(start, root->sectorsize) &&
-	       (end + 1) == ALIGN(end + 1, root->sectorsize));
+	ASSERT(IS_ALIGNED(start, root->sectorsize) &&
+	       IS_ALIGNED(end + 1, root->sectorsize));
 
 	path = btrfs_alloc_path();
 	if (!path)
Signature · 1 line
1.9.3