diff options
author | Josef Bacik <jbacik@fusionio.com> | 2013-07-22 12:50:37 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2013-09-01 08:04:23 -0400 |
commit | 50f1319cb5f7690e4d9de18d1a75ea89296d0e53 (patch) | |
tree | 7159026e037c35707d7814d1a442b10dbe9c5dfc | |
parent | a1b83ac52d23c85581b87836c346fb5cb90f6bfd (diff) |
Btrfs: reset ret in record_one_backref
I was getting warnings when running find ./ -type f -exec btrfs fi defrag -f {}
\; from record_one_backref because ret was set. Turns out it was because it was
set to 1 because the search slot didn't come out exact and we never reset it.
So reset it to 0 right after the search so we don't leak this and get
uneccessary warnings. Thanks,
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
-rw-r--r-- | fs/btrfs/inode.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 021694c08181..d3280b2b7592 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c | |||
@@ -2132,6 +2132,7 @@ static noinline int record_one_backref(u64 inum, u64 offset, u64 root_id, | |||
2132 | WARN_ON(1); | 2132 | WARN_ON(1); |
2133 | return ret; | 2133 | return ret; |
2134 | } | 2134 | } |
2135 | ret = 0; | ||
2135 | 2136 | ||
2136 | while (1) { | 2137 | while (1) { |
2137 | cond_resched(); | 2138 | cond_resched(); |
@@ -2181,8 +2182,6 @@ static noinline int record_one_backref(u64 inum, u64 offset, u64 root_id, | |||
2181 | old->len || extent_offset + num_bytes <= | 2182 | old->len || extent_offset + num_bytes <= |
2182 | old->extent_offset + old->offset) | 2183 | old->extent_offset + old->offset) |
2183 | continue; | 2184 | continue; |
2184 | |||
2185 | ret = 0; | ||
2186 | break; | 2185 | break; |
2187 | } | 2186 | } |
2188 | 2187 | ||