提交 3cd6dbe6 编写于 作者: J Jeff Mahoney 提交者: Linus Torvalds

reiserfs: cleanup path functions

This patch cleans up some redundancies in the reiserfs tree path code.

decrement_bcount() is essentially the same function as brelse(), so we use
that instead.

decrement_counters_in_path() is exactly the same function as pathrelse(), so
we kill that and use pathrelse() instead.

There's also a bit of cleanup that makes the code a bit more readable.
Signed-off-by: NJeff Mahoney <jeffm@suse.com>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
上级 fba4ebb5
...@@ -753,20 +753,21 @@ static void free_buffers_in_tb(struct tree_balance *p_s_tb) ...@@ -753,20 +753,21 @@ static void free_buffers_in_tb(struct tree_balance *p_s_tb)
{ {
int n_counter; int n_counter;
decrement_counters_in_path(p_s_tb->tb_path); pathrelse(p_s_tb->tb_path);
for (n_counter = 0; n_counter < MAX_HEIGHT; n_counter++) { for (n_counter = 0; n_counter < MAX_HEIGHT; n_counter++) {
decrement_bcount(p_s_tb->L[n_counter]); brelse(p_s_tb->L[n_counter]);
brelse(p_s_tb->R[n_counter]);
brelse(p_s_tb->FL[n_counter]);
brelse(p_s_tb->FR[n_counter]);
brelse(p_s_tb->CFL[n_counter]);
brelse(p_s_tb->CFR[n_counter]);
p_s_tb->L[n_counter] = NULL; p_s_tb->L[n_counter] = NULL;
decrement_bcount(p_s_tb->R[n_counter]);
p_s_tb->R[n_counter] = NULL; p_s_tb->R[n_counter] = NULL;
decrement_bcount(p_s_tb->FL[n_counter]);
p_s_tb->FL[n_counter] = NULL; p_s_tb->FL[n_counter] = NULL;
decrement_bcount(p_s_tb->FR[n_counter]);
p_s_tb->FR[n_counter] = NULL; p_s_tb->FR[n_counter] = NULL;
decrement_bcount(p_s_tb->CFL[n_counter]);
p_s_tb->CFL[n_counter] = NULL; p_s_tb->CFL[n_counter] = NULL;
decrement_bcount(p_s_tb->CFR[n_counter]);
p_s_tb->CFR[n_counter] = NULL; p_s_tb->CFR[n_counter] = NULL;
} }
} }
...@@ -1022,7 +1023,7 @@ static int get_far_parent(struct tree_balance *p_s_tb, ...@@ -1022,7 +1023,7 @@ static int get_far_parent(struct tree_balance *p_s_tb,
if (buffer_locked(*pp_s_com_father)) { if (buffer_locked(*pp_s_com_father)) {
__wait_on_buffer(*pp_s_com_father); __wait_on_buffer(*pp_s_com_father);
if (FILESYSTEM_CHANGED_TB(p_s_tb)) { if (FILESYSTEM_CHANGED_TB(p_s_tb)) {
decrement_bcount(*pp_s_com_father); brelse(*pp_s_com_father);
return REPEAT_SEARCH; return REPEAT_SEARCH;
} }
} }
...@@ -1050,8 +1051,8 @@ static int get_far_parent(struct tree_balance *p_s_tb, ...@@ -1050,8 +1051,8 @@ static int get_far_parent(struct tree_balance *p_s_tb,
return IO_ERROR; return IO_ERROR;
if (FILESYSTEM_CHANGED_TB(p_s_tb)) { if (FILESYSTEM_CHANGED_TB(p_s_tb)) {
decrement_counters_in_path(&s_path_to_neighbor_father); pathrelse(&s_path_to_neighbor_father);
decrement_bcount(*pp_s_com_father); brelse(*pp_s_com_father);
return REPEAT_SEARCH; return REPEAT_SEARCH;
} }
...@@ -1063,7 +1064,7 @@ static int get_far_parent(struct tree_balance *p_s_tb, ...@@ -1063,7 +1064,7 @@ static int get_far_parent(struct tree_balance *p_s_tb,
FIRST_PATH_ELEMENT_OFFSET, "PAP-8192: path length is too small"); FIRST_PATH_ELEMENT_OFFSET, "PAP-8192: path length is too small");
s_path_to_neighbor_father.path_length--; s_path_to_neighbor_father.path_length--;
decrement_counters_in_path(&s_path_to_neighbor_father); pathrelse(&s_path_to_neighbor_father);
return CARRY_ON; return CARRY_ON;
} }
...@@ -1086,10 +1087,10 @@ static int get_parents(struct tree_balance *p_s_tb, int n_h) ...@@ -1086,10 +1087,10 @@ static int get_parents(struct tree_balance *p_s_tb, int n_h)
if (n_path_offset <= FIRST_PATH_ELEMENT_OFFSET) { if (n_path_offset <= FIRST_PATH_ELEMENT_OFFSET) {
/* The root can not have parents. /* The root can not have parents.
Release nodes which previously were obtained as parents of the current node neighbors. */ Release nodes which previously were obtained as parents of the current node neighbors. */
decrement_bcount(p_s_tb->FL[n_h]); brelse(p_s_tb->FL[n_h]);
decrement_bcount(p_s_tb->CFL[n_h]); brelse(p_s_tb->CFL[n_h]);
decrement_bcount(p_s_tb->FR[n_h]); brelse(p_s_tb->FR[n_h]);
decrement_bcount(p_s_tb->CFR[n_h]); brelse(p_s_tb->CFR[n_h]);
p_s_tb->FL[n_h] = p_s_tb->CFL[n_h] = p_s_tb->FR[n_h] = p_s_tb->FL[n_h] = p_s_tb->CFL[n_h] = p_s_tb->FR[n_h] =
p_s_tb->CFR[n_h] = NULL; p_s_tb->CFR[n_h] = NULL;
return CARRY_ON; return CARRY_ON;
...@@ -1115,9 +1116,9 @@ static int get_parents(struct tree_balance *p_s_tb, int n_h) ...@@ -1115,9 +1116,9 @@ static int get_parents(struct tree_balance *p_s_tb, int n_h)
return n_ret_value; return n_ret_value;
} }
decrement_bcount(p_s_tb->FL[n_h]); brelse(p_s_tb->FL[n_h]);
p_s_tb->FL[n_h] = p_s_curf; /* New initialization of FL[n_h]. */ p_s_tb->FL[n_h] = p_s_curf; /* New initialization of FL[n_h]. */
decrement_bcount(p_s_tb->CFL[n_h]); brelse(p_s_tb->CFL[n_h]);
p_s_tb->CFL[n_h] = p_s_curcf; /* New initialization of CFL[n_h]. */ p_s_tb->CFL[n_h] = p_s_curcf; /* New initialization of CFL[n_h]. */
RFALSE((p_s_curf && !B_IS_IN_TREE(p_s_curf)) || RFALSE((p_s_curf && !B_IS_IN_TREE(p_s_curf)) ||
...@@ -1145,10 +1146,10 @@ static int get_parents(struct tree_balance *p_s_tb, int n_h) ...@@ -1145,10 +1146,10 @@ static int get_parents(struct tree_balance *p_s_tb, int n_h)
p_s_tb->rkey[n_h] = n_position; p_s_tb->rkey[n_h] = n_position;
} }
decrement_bcount(p_s_tb->FR[n_h]); brelse(p_s_tb->FR[n_h]);
p_s_tb->FR[n_h] = p_s_curf; /* New initialization of FR[n_path_offset]. */ p_s_tb->FR[n_h] = p_s_curf; /* New initialization of FR[n_path_offset]. */
decrement_bcount(p_s_tb->CFR[n_h]); brelse(p_s_tb->CFR[n_h]);
p_s_tb->CFR[n_h] = p_s_curcf; /* New initialization of CFR[n_path_offset]. */ p_s_tb->CFR[n_h] = p_s_curcf; /* New initialization of CFR[n_path_offset]. */
RFALSE((p_s_curf && !B_IS_IN_TREE(p_s_curf)) || RFALSE((p_s_curf && !B_IS_IN_TREE(p_s_curf)) ||
...@@ -1964,7 +1965,7 @@ static int get_neighbors(struct tree_balance *p_s_tb, int n_h) ...@@ -1964,7 +1965,7 @@ static int get_neighbors(struct tree_balance *p_s_tb, int n_h)
if (!p_s_bh) if (!p_s_bh)
return IO_ERROR; return IO_ERROR;
if (FILESYSTEM_CHANGED_TB(p_s_tb)) { if (FILESYSTEM_CHANGED_TB(p_s_tb)) {
decrement_bcount(p_s_bh); brelse(p_s_bh);
PROC_INFO_INC(p_s_sb, get_neighbors_restart[n_h]); PROC_INFO_INC(p_s_sb, get_neighbors_restart[n_h]);
return REPEAT_SEARCH; return REPEAT_SEARCH;
} }
...@@ -1980,7 +1981,7 @@ static int get_neighbors(struct tree_balance *p_s_tb, int n_h) ...@@ -1980,7 +1981,7 @@ static int get_neighbors(struct tree_balance *p_s_tb, int n_h)
dc_size(B_N_CHILD(p_s_tb->FL[0], n_child_position)), dc_size(B_N_CHILD(p_s_tb->FL[0], n_child_position)),
"PAP-8290: invalid child size of left neighbor"); "PAP-8290: invalid child size of left neighbor");
decrement_bcount(p_s_tb->L[n_h]); brelse(p_s_tb->L[n_h]);
p_s_tb->L[n_h] = p_s_bh; p_s_tb->L[n_h] = p_s_bh;
} }
...@@ -2001,11 +2002,11 @@ static int get_neighbors(struct tree_balance *p_s_tb, int n_h) ...@@ -2001,11 +2002,11 @@ static int get_neighbors(struct tree_balance *p_s_tb, int n_h)
if (!p_s_bh) if (!p_s_bh)
return IO_ERROR; return IO_ERROR;
if (FILESYSTEM_CHANGED_TB(p_s_tb)) { if (FILESYSTEM_CHANGED_TB(p_s_tb)) {
decrement_bcount(p_s_bh); brelse(p_s_bh);
PROC_INFO_INC(p_s_sb, get_neighbors_restart[n_h]); PROC_INFO_INC(p_s_sb, get_neighbors_restart[n_h]);
return REPEAT_SEARCH; return REPEAT_SEARCH;
} }
decrement_bcount(p_s_tb->R[n_h]); brelse(p_s_tb->R[n_h]);
p_s_tb->R[n_h] = p_s_bh; p_s_tb->R[n_h] = p_s_bh;
RFALSE(!n_h RFALSE(!n_h
...@@ -2511,16 +2512,17 @@ int fix_nodes(int n_op_mode, struct tree_balance *p_s_tb, struct item_head *p_s_ ...@@ -2511,16 +2512,17 @@ int fix_nodes(int n_op_mode, struct tree_balance *p_s_tb, struct item_head *p_s_
} }
brelse(p_s_tb->L[i]); brelse(p_s_tb->L[i]);
p_s_tb->L[i] = NULL;
brelse(p_s_tb->R[i]); brelse(p_s_tb->R[i]);
p_s_tb->R[i] = NULL;
brelse(p_s_tb->FL[i]); brelse(p_s_tb->FL[i]);
p_s_tb->FL[i] = NULL;
brelse(p_s_tb->FR[i]); brelse(p_s_tb->FR[i]);
p_s_tb->FR[i] = NULL;
brelse(p_s_tb->CFL[i]); brelse(p_s_tb->CFL[i]);
p_s_tb->CFL[i] = NULL;
brelse(p_s_tb->CFR[i]); brelse(p_s_tb->CFR[i]);
p_s_tb->L[i] = NULL;
p_s_tb->R[i] = NULL;
p_s_tb->FL[i] = NULL;
p_s_tb->FR[i] = NULL;
p_s_tb->CFL[i] = NULL;
p_s_tb->CFR[i] = NULL; p_s_tb->CFR[i] = NULL;
} }
......
...@@ -23,7 +23,6 @@ ...@@ -23,7 +23,6 @@
* get_rkey * get_rkey
* key_in_buffer * key_in_buffer
* decrement_bcount * decrement_bcount
* decrement_counters_in_path
* reiserfs_check_path * reiserfs_check_path
* pathrelse_and_restore * pathrelse_and_restore
* pathrelse * pathrelse
...@@ -359,36 +358,6 @@ static inline int key_in_buffer(struct treepath *p_s_chk_path, /* Path which sho ...@@ -359,36 +358,6 @@ static inline int key_in_buffer(struct treepath *p_s_chk_path, /* Path which sho
return 1; return 1;
} }
inline void decrement_bcount(struct buffer_head *p_s_bh)
{
if (p_s_bh) {
if (atomic_read(&(p_s_bh->b_count))) {
put_bh(p_s_bh);
return;
}
reiserfs_panic(NULL, "PAP-5070",
"trying to free free buffer %b", p_s_bh);
}
}
/* Decrement b_count field of the all buffers in the path. */
void decrement_counters_in_path(struct treepath *p_s_search_path)
{
int n_path_offset = p_s_search_path->path_length;
RFALSE(n_path_offset < ILLEGAL_PATH_ELEMENT_OFFSET ||
n_path_offset > EXTENDED_MAX_HEIGHT - 1,
"PAP-5080: invalid path offset of %d", n_path_offset);
while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) {
struct buffer_head *bh;
bh = PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--);
decrement_bcount(bh);
}
p_s_search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET;
}
int reiserfs_check_path(struct treepath *p) int reiserfs_check_path(struct treepath *p)
{ {
RFALSE(p->path_length != ILLEGAL_PATH_ELEMENT_OFFSET, RFALSE(p->path_length != ILLEGAL_PATH_ELEMENT_OFFSET,
...@@ -396,12 +365,11 @@ int reiserfs_check_path(struct treepath *p) ...@@ -396,12 +365,11 @@ int reiserfs_check_path(struct treepath *p)
return 0; return 0;
} }
/* Release all buffers in the path. Restore dirty bits clean /* Drop the reference to each buffer in a path and restore
** when preparing the buffer for the log * dirty bits clean when preparing the buffer for the log.
** * This version should only be called from fix_nodes() */
** only called from fix_nodes() void pathrelse_and_restore(struct super_block *sb,
*/ struct treepath *p_s_search_path)
void pathrelse_and_restore(struct super_block *s, struct treepath *p_s_search_path)
{ {
int n_path_offset = p_s_search_path->path_length; int n_path_offset = p_s_search_path->path_length;
...@@ -409,16 +377,15 @@ void pathrelse_and_restore(struct super_block *s, struct treepath *p_s_search_pa ...@@ -409,16 +377,15 @@ void pathrelse_and_restore(struct super_block *s, struct treepath *p_s_search_pa
"clm-4000: invalid path offset"); "clm-4000: invalid path offset");
while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) { while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) {
reiserfs_restore_prepared_buffer(s, struct buffer_head *bh;
PATH_OFFSET_PBUFFER bh = PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--);
(p_s_search_path, reiserfs_restore_prepared_buffer(sb, bh);
n_path_offset)); brelse(bh);
brelse(PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--));
} }
p_s_search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET; p_s_search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET;
} }
/* Release all buffers in the path. */ /* Drop the reference to each buffer in a path */
void pathrelse(struct treepath *p_s_search_path) void pathrelse(struct treepath *p_s_search_path)
{ {
int n_path_offset = p_s_search_path->path_length; int n_path_offset = p_s_search_path->path_length;
...@@ -631,7 +598,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key, /* ...@@ -631,7 +598,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key, /*
we must be careful to release all nodes in a path before we either we must be careful to release all nodes in a path before we either
discard the path struct or re-use the path struct, as we do here. */ discard the path struct or re-use the path struct, as we do here. */
decrement_counters_in_path(p_s_search_path); pathrelse(p_s_search_path);
right_neighbor_of_leaf_node = 0; right_neighbor_of_leaf_node = 0;
...@@ -691,7 +658,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key, /* ...@@ -691,7 +658,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key, /*
PROC_INFO_INC(p_s_sb, search_by_key_restarted); PROC_INFO_INC(p_s_sb, search_by_key_restarted);
PROC_INFO_INC(p_s_sb, PROC_INFO_INC(p_s_sb,
sbk_restarted[expected_level - 1]); sbk_restarted[expected_level - 1]);
decrement_counters_in_path(p_s_search_path); pathrelse(p_s_search_path);
/* Get the root block number so that we can repeat the search /* Get the root block number so that we can repeat the search
starting from the root. */ starting from the root. */
...@@ -1868,7 +1835,7 @@ int reiserfs_do_truncate(struct reiserfs_transaction_handle *th, struct inode *p ...@@ -1868,7 +1835,7 @@ int reiserfs_do_truncate(struct reiserfs_transaction_handle *th, struct inode *p
if (journal_transaction_should_end(th, 0) || if (journal_transaction_should_end(th, 0) ||
reiserfs_transaction_free_space(th) <= JOURNAL_FOR_FREE_BLOCK_AND_UPDATE_SD) { reiserfs_transaction_free_space(th) <= JOURNAL_FOR_FREE_BLOCK_AND_UPDATE_SD) {
int orig_len_alloc = th->t_blocks_allocated; int orig_len_alloc = th->t_blocks_allocated;
decrement_counters_in_path(&s_search_path); pathrelse(&s_search_path);
if (update_timestamps) { if (update_timestamps) {
p_s_inode->i_mtime = p_s_inode->i_ctime = p_s_inode->i_mtime = p_s_inode->i_ctime =
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册