summaryrefslogtreecommitdiff
path: root/fs/ocfs2
diff options
context:
space:
mode:
authorTao Ma <tao.ma@oracle.com>2010-04-06 12:46:46 +0400
committerJoel Becker <joel.becker@oracle.com>2010-05-06 05:18:09 +0400
commit3e4218df3176657be72ad2fa199779be6c11fe4f (patch)
tree99512a1b274e7e99af1b7fe9d388efce79399c20 /fs/ocfs2
parentb065556a7d1a9205403db77a318a5c5aa530e701 (diff)
downloadlinux-3e4218df3176657be72ad2fa199779be6c11fe4f.tar.xz
ocfs2/trivial: Code cleanup for allocation reservation.
Two tiny cleanup for allocation reservation. 1. Remove some extra codes in ocfs2_local_alloc_find_clear_bits. 2. Remove an unuseful variables in ocfs2_find_resv_lhs. Signed-off-by: Tao Ma <tao.ma@oracle.com> Acked-by: Mark Fasheh <mfasheh@suse.com> Signed-off-by: Joel Becker <joel.becker@oracle.com>
Diffstat (limited to 'fs/ocfs2')
-rw-r--r--fs/ocfs2/localalloc.c7
-rw-r--r--fs/ocfs2/reservations.c2
2 files changed, 2 insertions, 7 deletions
diff --git a/fs/ocfs2/localalloc.c b/fs/ocfs2/localalloc.c
index 00022aac2e8c..63c41e206792 100644
--- a/fs/ocfs2/localalloc.c
+++ b/fs/ocfs2/localalloc.c
@@ -879,13 +879,10 @@ static int ocfs2_local_alloc_find_clear_bits(struct ocfs2_super *osb,
mlog(0, "Exiting loop, bitoff = %d, numfound = %d\n", bitoff,
numfound);
- if (numfound == *numbits) {
+ if (numfound == *numbits)
bitoff = startoff - numfound;
- *numbits = numfound;
- } else {
- numfound = 0;
+ else
bitoff = -1;
- }
bail:
if (local_resv)
diff --git a/fs/ocfs2/reservations.c b/fs/ocfs2/reservations.c
index cb813ef98846..40650021fc24 100644
--- a/fs/ocfs2/reservations.c
+++ b/fs/ocfs2/reservations.c
@@ -371,7 +371,6 @@ ocfs2_find_resv_lhs(struct ocfs2_reservation_map *resmap, unsigned int goal)
struct ocfs2_alloc_reservation *resv = NULL;
struct ocfs2_alloc_reservation *prev_resv = NULL;
struct rb_node *node = resmap->m_reservations.rb_node;
- struct rb_node *prev = NULL;
assert_spin_locked(&resv_lock);
@@ -392,7 +391,6 @@ ocfs2_find_resv_lhs(struct ocfs2_reservation_map *resmap, unsigned int goal)
}
prev_resv = resv;
- prev = node;
node = rb_next(node);
}