diff options
| author | Liu Bo <liubo2009@cn.fujitsu.com> | 2012-07-10 07:28:38 -0400 |
|---|---|---|
| committer | Chris Mason <chris.mason@fusionio.com> | 2012-07-23 16:28:00 -0400 |
| commit | 51a8cf9d2d97017d334f33f1b39067bd2f03bc49 (patch) | |
| tree | a80be16062c2f5ff10d766852cf9d79a852aa0ef | |
| parent | c0901581ad077004145c9ee80e843fba71c100b8 (diff) | |
Btrfs: fix btrfs_is_free_space_inode to recognize btree inode
For btree inode, its root is also 'tree root', so btree inode can be
misunderstood as a free space inode.
We should add one more check for btree inode.
Signed-off-by: Liu Bo <liubo2009@cn.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
| -rw-r--r-- | fs/btrfs/btrfs_inode.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/btrfs/btrfs_inode.h b/fs/btrfs/btrfs_inode.h index 12394a90d60f..b168238bcb13 100644 --- a/fs/btrfs/btrfs_inode.h +++ b/fs/btrfs/btrfs_inode.h | |||
| @@ -194,8 +194,10 @@ static inline void btrfs_i_size_write(struct inode *inode, u64 size) | |||
| 194 | static inline bool btrfs_is_free_space_inode(struct btrfs_root *root, | 194 | static inline bool btrfs_is_free_space_inode(struct btrfs_root *root, |
| 195 | struct inode *inode) | 195 | struct inode *inode) |
| 196 | { | 196 | { |
| 197 | if (root == root->fs_info->tree_root || | 197 | if (root == root->fs_info->tree_root && |
| 198 | BTRFS_I(inode)->location.objectid == BTRFS_FREE_INO_OBJECTID) | 198 | btrfs_ino(inode) != BTRFS_BTREE_INODE_OBJECTID) |
| 199 | return true; | ||
| 200 | if (BTRFS_I(inode)->location.objectid == BTRFS_FREE_INO_OBJECTID) | ||
| 199 | return true; | 201 | return true; |
| 200 | return false; | 202 | return false; |
| 201 | } | 203 | } |
