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 /fs/btrfs/inode.c | |
| parent | a1b83ac52d23c85581b87836c346fb5cb90f6bfd (diff) | |
| download | talos-obmc-linux-50f1319cb5f7690e4d9de18d1a75ea89296d0e53.tar.gz talos-obmc-linux-50f1319cb5f7690e4d9de18d1a75ea89296d0e53.zip | |
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>
Diffstat (limited to 'fs/btrfs/inode.c')
| -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, WARN_ON(1); return ret; } + ret = 0; while (1) { cond_resched(); @@ -2181,8 +2182,6 @@ static noinline int record_one_backref(u64 inum, u64 offset, u64 root_id, old->len || extent_offset + num_bytes <= old->extent_offset + old->offset) continue; - - ret = 0; break; } |
