btrfs: cleanup for open-coded alignment
authorQu Wenruo <quwenruo@cn.fujitsu.com>
Tue, 26 Feb 2013 08:10:22 +0000 (08:10 +0000)
committerJosef Bacik <jbacik@fusionio.com>
Tue, 26 Feb 2013 16:04:13 +0000 (11:04 -0500)
commitfda2832febb1928da0625b2c5d15559b29d7e740
treedac83eff57b657dfc04ba72fa7bcce4b4674b810
parent8c4ce81e911ab6c84e4f75e18d4ceb3fa555c35b
btrfs: cleanup for open-coded alignment

Though most of the btrfs codes are using ALIGN macro for page alignment,
there are still some codes using open-coded alignment like the
following:
------
        u64 mask = ((u64)root->stripesize - 1);
        u64 ret = (val + mask) & ~mask;
------
Or even hidden one:
------
        num_bytes = (end - start + blocksize) & ~(blocksize - 1);
------

Sometimes these open-coded alignment is not so easy to understand for
newbie like me.

This commit changes the open-coded alignment to the ALIGN macro for a
better readability.

Also there is a previous patch from David Sterba with similar changes,
but the patch is for 3.2 kernel and seems not merged.
http://www.spinics.net/lists/linux-btrfs/msg12747.html

Cc: David Sterba <dave@jikos.cz>
Signed-off-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
fs/btrfs/extent-tree.c
fs/btrfs/extent_io.c
fs/btrfs/file.c
fs/btrfs/inode.c
fs/btrfs/tree-log.c
fs/btrfs/volumes.c