diff options
author | Jeff Mahoney <jeffm@suse.com> | 2009-03-30 14:02:49 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-03-30 15:16:40 -0400 |
commit | d68caa9530a8ba54f97002e02bf6a0ad2462b8c0 (patch) | |
tree | 39a2b877483270253f95f3678a4559e9bd5524e8 | |
parent | a063ae17925cafabe55ebe1957ca0e8c480bd132 (diff) |
reiserfs: rename p_._ variables
This patch is a simple s/p_._//g to the reiserfs code. This is the
fifth in a series of patches to rip out some of the awful variable
naming in reiserfs.
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/reiserfs/file.c | 6 | ||||
-rw-r--r-- | fs/reiserfs/fix_node.c | 169 | ||||
-rw-r--r-- | fs/reiserfs/stree.c | 472 | ||||
-rw-r--r-- | fs/reiserfs/tail_conversion.c | 28 | ||||
-rw-r--r-- | include/linux/reiserfs_fs.h | 46 |
5 files changed, 365 insertions, 356 deletions
diff --git a/fs/reiserfs/file.c b/fs/reiserfs/file.c index a73579f66214..cde16429ff00 100644 --- a/fs/reiserfs/file.c +++ b/fs/reiserfs/file.c | |||
@@ -134,10 +134,10 @@ static void reiserfs_vfs_truncate_file(struct inode *inode) | |||
134 | * be removed... | 134 | * be removed... |
135 | */ | 135 | */ |
136 | 136 | ||
137 | static int reiserfs_sync_file(struct file *p_s_filp, | 137 | static int reiserfs_sync_file(struct file *filp, |
138 | struct dentry *p_s_dentry, int datasync) | 138 | struct dentry *dentry, int datasync) |
139 | { | 139 | { |
140 | struct inode *inode = p_s_dentry->d_inode; | 140 | struct inode *inode = dentry->d_inode; |
141 | int n_err; | 141 | int n_err; |
142 | int barrier_done; | 142 | int barrier_done; |
143 | 143 | ||
diff --git a/fs/reiserfs/fix_node.c b/fs/reiserfs/fix_node.c index 5236a8829e31..d97a55574ba9 100644 --- a/fs/reiserfs/fix_node.c +++ b/fs/reiserfs/fix_node.c | |||
@@ -780,9 +780,9 @@ static void free_buffers_in_tb(struct tree_balance *tb) | |||
780 | /* The function is NOT SCHEDULE-SAFE! */ | 780 | /* The function is NOT SCHEDULE-SAFE! */ |
781 | static int get_empty_nodes(struct tree_balance *tb, int n_h) | 781 | static int get_empty_nodes(struct tree_balance *tb, int n_h) |
782 | { | 782 | { |
783 | struct buffer_head *p_s_new_bh, | 783 | struct buffer_head *new_bh, |
784 | *p_s_Sh = PATH_H_PBUFFER(tb->tb_path, n_h); | 784 | *Sh = PATH_H_PBUFFER(tb->tb_path, n_h); |
785 | b_blocknr_t *p_n_blocknr, a_n_blocknrs[MAX_AMOUNT_NEEDED] = { 0, }; | 785 | b_blocknr_t *blocknr, a_n_blocknrs[MAX_AMOUNT_NEEDED] = { 0, }; |
786 | int n_counter, n_number_of_freeblk, n_amount_needed, /* number of needed empty blocks */ | 786 | int n_counter, n_number_of_freeblk, n_amount_needed, /* number of needed empty blocks */ |
787 | n_retval = CARRY_ON; | 787 | n_retval = CARRY_ON; |
788 | struct super_block *sb = tb->tb_sb; | 788 | struct super_block *sb = tb->tb_sb; |
@@ -810,8 +810,8 @@ static int get_empty_nodes(struct tree_balance *tb, int n_h) | |||
810 | 1) : 0; | 810 | 1) : 0; |
811 | 811 | ||
812 | /* Allocate missing empty blocks. */ | 812 | /* Allocate missing empty blocks. */ |
813 | /* if p_s_Sh == 0 then we are getting a new root */ | 813 | /* if Sh == 0 then we are getting a new root */ |
814 | n_amount_needed = (p_s_Sh) ? (tb->blknum[n_h] - 1) : 1; | 814 | n_amount_needed = (Sh) ? (tb->blknum[n_h] - 1) : 1; |
815 | /* Amount_needed = the amount that we need more than the amount that we have. */ | 815 | /* Amount_needed = the amount that we need more than the amount that we have. */ |
816 | if (n_amount_needed > n_number_of_freeblk) | 816 | if (n_amount_needed > n_number_of_freeblk) |
817 | n_amount_needed -= n_number_of_freeblk; | 817 | n_amount_needed -= n_number_of_freeblk; |
@@ -824,25 +824,25 @@ static int get_empty_nodes(struct tree_balance *tb, int n_h) | |||
824 | return NO_DISK_SPACE; | 824 | return NO_DISK_SPACE; |
825 | 825 | ||
826 | /* for each blocknumber we just got, get a buffer and stick it on FEB */ | 826 | /* for each blocknumber we just got, get a buffer and stick it on FEB */ |
827 | for (p_n_blocknr = a_n_blocknrs, n_counter = 0; | 827 | for (blocknr = a_n_blocknrs, n_counter = 0; |
828 | n_counter < n_amount_needed; p_n_blocknr++, n_counter++) { | 828 | n_counter < n_amount_needed; blocknr++, n_counter++) { |
829 | 829 | ||
830 | RFALSE(!*p_n_blocknr, | 830 | RFALSE(!*blocknr, |
831 | "PAP-8135: reiserfs_new_blocknrs failed when got new blocks"); | 831 | "PAP-8135: reiserfs_new_blocknrs failed when got new blocks"); |
832 | 832 | ||
833 | p_s_new_bh = sb_getblk(sb, *p_n_blocknr); | 833 | new_bh = sb_getblk(sb, *blocknr); |
834 | RFALSE(buffer_dirty(p_s_new_bh) || | 834 | RFALSE(buffer_dirty(new_bh) || |
835 | buffer_journaled(p_s_new_bh) || | 835 | buffer_journaled(new_bh) || |
836 | buffer_journal_dirty(p_s_new_bh), | 836 | buffer_journal_dirty(new_bh), |
837 | "PAP-8140: journlaled or dirty buffer %b for the new block", | 837 | "PAP-8140: journlaled or dirty buffer %b for the new block", |
838 | p_s_new_bh); | 838 | new_bh); |
839 | 839 | ||
840 | /* Put empty buffers into the array. */ | 840 | /* Put empty buffers into the array. */ |
841 | RFALSE(tb->FEB[tb->cur_blknum], | 841 | RFALSE(tb->FEB[tb->cur_blknum], |
842 | "PAP-8141: busy slot for new buffer"); | 842 | "PAP-8141: busy slot for new buffer"); |
843 | 843 | ||
844 | set_buffer_journal_new(p_s_new_bh); | 844 | set_buffer_journal_new(new_bh); |
845 | tb->FEB[tb->cur_blknum++] = p_s_new_bh; | 845 | tb->FEB[tb->cur_blknum++] = new_bh; |
846 | } | 846 | } |
847 | 847 | ||
848 | if (n_retval == CARRY_ON && FILESYSTEM_CHANGED_TB(tb)) | 848 | if (n_retval == CARRY_ON && FILESYSTEM_CHANGED_TB(tb)) |
@@ -898,7 +898,7 @@ static int get_rfree(struct tree_balance *tb, int h) | |||
898 | /* Check whether left neighbor is in memory. */ | 898 | /* Check whether left neighbor is in memory. */ |
899 | static int is_left_neighbor_in_cache(struct tree_balance *tb, int n_h) | 899 | static int is_left_neighbor_in_cache(struct tree_balance *tb, int n_h) |
900 | { | 900 | { |
901 | struct buffer_head *p_s_father, *left; | 901 | struct buffer_head *father, *left; |
902 | struct super_block *sb = tb->tb_sb; | 902 | struct super_block *sb = tb->tb_sb; |
903 | b_blocknr_t n_left_neighbor_blocknr; | 903 | b_blocknr_t n_left_neighbor_blocknr; |
904 | int n_left_neighbor_position; | 904 | int n_left_neighbor_position; |
@@ -908,18 +908,18 @@ static int is_left_neighbor_in_cache(struct tree_balance *tb, int n_h) | |||
908 | return 0; | 908 | return 0; |
909 | 909 | ||
910 | /* Calculate father of the node to be balanced. */ | 910 | /* Calculate father of the node to be balanced. */ |
911 | p_s_father = PATH_H_PBUFFER(tb->tb_path, n_h + 1); | 911 | father = PATH_H_PBUFFER(tb->tb_path, n_h + 1); |
912 | 912 | ||
913 | RFALSE(!p_s_father || | 913 | RFALSE(!father || |
914 | !B_IS_IN_TREE(p_s_father) || | 914 | !B_IS_IN_TREE(father) || |
915 | !B_IS_IN_TREE(tb->FL[n_h]) || | 915 | !B_IS_IN_TREE(tb->FL[n_h]) || |
916 | !buffer_uptodate(p_s_father) || | 916 | !buffer_uptodate(father) || |
917 | !buffer_uptodate(tb->FL[n_h]), | 917 | !buffer_uptodate(tb->FL[n_h]), |
918 | "vs-8165: F[h] (%b) or FL[h] (%b) is invalid", | 918 | "vs-8165: F[h] (%b) or FL[h] (%b) is invalid", |
919 | p_s_father, tb->FL[n_h]); | 919 | father, tb->FL[n_h]); |
920 | 920 | ||
921 | /* Get position of the pointer to the left neighbor into the left father. */ | 921 | /* Get position of the pointer to the left neighbor into the left father. */ |
922 | n_left_neighbor_position = (p_s_father == tb->FL[n_h]) ? | 922 | n_left_neighbor_position = (father == tb->FL[n_h]) ? |
923 | tb->lkey[n_h] : B_NR_ITEMS(tb->FL[n_h]); | 923 | tb->lkey[n_h] : B_NR_ITEMS(tb->FL[n_h]); |
924 | /* Get left neighbor block number. */ | 924 | /* Get left neighbor block number. */ |
925 | n_left_neighbor_blocknr = | 925 | n_left_neighbor_blocknr = |
@@ -940,10 +940,10 @@ static int is_left_neighbor_in_cache(struct tree_balance *tb, int n_h) | |||
940 | #define LEFT_PARENTS 'l' | 940 | #define LEFT_PARENTS 'l' |
941 | #define RIGHT_PARENTS 'r' | 941 | #define RIGHT_PARENTS 'r' |
942 | 942 | ||
943 | static void decrement_key(struct cpu_key *p_s_key) | 943 | static void decrement_key(struct cpu_key *key) |
944 | { | 944 | { |
945 | // call item specific function for this key | 945 | // call item specific function for this key |
946 | item_ops[cpu_key_k_type(p_s_key)]->decrement_key(p_s_key); | 946 | item_ops[cpu_key_k_type(key)]->decrement_key(key); |
947 | } | 947 | } |
948 | 948 | ||
949 | /* Calculate far left/right parent of the left/right neighbor of the current node, that | 949 | /* Calculate far left/right parent of the left/right neighbor of the current node, that |
@@ -956,17 +956,17 @@ static void decrement_key(struct cpu_key *p_s_key) | |||
956 | */ | 956 | */ |
957 | static int get_far_parent(struct tree_balance *tb, | 957 | static int get_far_parent(struct tree_balance *tb, |
958 | int n_h, | 958 | int n_h, |
959 | struct buffer_head **pp_s_father, | 959 | struct buffer_head **pfather, |
960 | struct buffer_head **pp_s_com_father, char c_lr_par) | 960 | struct buffer_head **pcom_father, char c_lr_par) |
961 | { | 961 | { |
962 | struct buffer_head *p_s_parent; | 962 | struct buffer_head *parent; |
963 | INITIALIZE_PATH(s_path_to_neighbor_father); | 963 | INITIALIZE_PATH(s_path_to_neighbor_father); |
964 | struct treepath *p_s_path = tb->tb_path; | 964 | struct treepath *path = tb->tb_path; |
965 | struct cpu_key s_lr_father_key; | 965 | struct cpu_key s_lr_father_key; |
966 | int n_counter, | 966 | int n_counter, |
967 | n_position = INT_MAX, | 967 | n_position = INT_MAX, |
968 | n_first_last_position = 0, | 968 | n_first_last_position = 0, |
969 | n_path_offset = PATH_H_PATH_OFFSET(p_s_path, n_h); | 969 | n_path_offset = PATH_H_PATH_OFFSET(path, n_h); |
970 | 970 | ||
971 | /* Starting from F[n_h] go upwards in the tree, and look for the common | 971 | /* Starting from F[n_h] go upwards in the tree, and look for the common |
972 | ancestor of F[n_h], and its neighbor l/r, that should be obtained. */ | 972 | ancestor of F[n_h], and its neighbor l/r, that should be obtained. */ |
@@ -979,25 +979,25 @@ static int get_far_parent(struct tree_balance *tb, | |||
979 | for (; n_counter > FIRST_PATH_ELEMENT_OFFSET; n_counter--) { | 979 | for (; n_counter > FIRST_PATH_ELEMENT_OFFSET; n_counter--) { |
980 | /* Check whether parent of the current buffer in the path is really parent in the tree. */ | 980 | /* Check whether parent of the current buffer in the path is really parent in the tree. */ |
981 | if (!B_IS_IN_TREE | 981 | if (!B_IS_IN_TREE |
982 | (p_s_parent = PATH_OFFSET_PBUFFER(p_s_path, n_counter - 1))) | 982 | (parent = PATH_OFFSET_PBUFFER(path, n_counter - 1))) |
983 | return REPEAT_SEARCH; | 983 | return REPEAT_SEARCH; |
984 | /* Check whether position in the parent is correct. */ | 984 | /* Check whether position in the parent is correct. */ |
985 | if ((n_position = | 985 | if ((n_position = |
986 | PATH_OFFSET_POSITION(p_s_path, | 986 | PATH_OFFSET_POSITION(path, |
987 | n_counter - 1)) > | 987 | n_counter - 1)) > |
988 | B_NR_ITEMS(p_s_parent)) | 988 | B_NR_ITEMS(parent)) |
989 | return REPEAT_SEARCH; | 989 | return REPEAT_SEARCH; |
990 | /* Check whether parent at the path really points to the child. */ | 990 | /* Check whether parent at the path really points to the child. */ |
991 | if (B_N_CHILD_NUM(p_s_parent, n_position) != | 991 | if (B_N_CHILD_NUM(parent, n_position) != |
992 | PATH_OFFSET_PBUFFER(p_s_path, n_counter)->b_blocknr) | 992 | PATH_OFFSET_PBUFFER(path, n_counter)->b_blocknr) |
993 | return REPEAT_SEARCH; | 993 | return REPEAT_SEARCH; |
994 | /* Return delimiting key if position in the parent is not equal to first/last one. */ | 994 | /* Return delimiting key if position in the parent is not equal to first/last one. */ |
995 | if (c_lr_par == RIGHT_PARENTS) | 995 | if (c_lr_par == RIGHT_PARENTS) |
996 | n_first_last_position = B_NR_ITEMS(p_s_parent); | 996 | n_first_last_position = B_NR_ITEMS(parent); |
997 | if (n_position != n_first_last_position) { | 997 | if (n_position != n_first_last_position) { |
998 | *pp_s_com_father = p_s_parent; | 998 | *pcom_father = parent; |
999 | get_bh(*pp_s_com_father); | 999 | get_bh(*pcom_father); |
1000 | /*(*pp_s_com_father = p_s_parent)->b_count++; */ | 1000 | /*(*pcom_father = parent)->b_count++; */ |
1001 | break; | 1001 | break; |
1002 | } | 1002 | } |
1003 | } | 1003 | } |
@@ -1009,22 +1009,22 @@ static int get_far_parent(struct tree_balance *tb, | |||
1009 | (tb->tb_path, | 1009 | (tb->tb_path, |
1010 | FIRST_PATH_ELEMENT_OFFSET)->b_blocknr == | 1010 | FIRST_PATH_ELEMENT_OFFSET)->b_blocknr == |
1011 | SB_ROOT_BLOCK(tb->tb_sb)) { | 1011 | SB_ROOT_BLOCK(tb->tb_sb)) { |
1012 | *pp_s_father = *pp_s_com_father = NULL; | 1012 | *pfather = *pcom_father = NULL; |
1013 | return CARRY_ON; | 1013 | return CARRY_ON; |
1014 | } | 1014 | } |
1015 | return REPEAT_SEARCH; | 1015 | return REPEAT_SEARCH; |
1016 | } | 1016 | } |
1017 | 1017 | ||
1018 | RFALSE(B_LEVEL(*pp_s_com_father) <= DISK_LEAF_NODE_LEVEL, | 1018 | RFALSE(B_LEVEL(*pcom_father) <= DISK_LEAF_NODE_LEVEL, |
1019 | "PAP-8185: (%b %z) level too small", | 1019 | "PAP-8185: (%b %z) level too small", |
1020 | *pp_s_com_father, *pp_s_com_father); | 1020 | *pcom_father, *pcom_father); |
1021 | 1021 | ||
1022 | /* Check whether the common parent is locked. */ | 1022 | /* Check whether the common parent is locked. */ |
1023 | 1023 | ||
1024 | if (buffer_locked(*pp_s_com_father)) { | 1024 | if (buffer_locked(*pcom_father)) { |
1025 | __wait_on_buffer(*pp_s_com_father); | 1025 | __wait_on_buffer(*pcom_father); |
1026 | if (FILESYSTEM_CHANGED_TB(tb)) { | 1026 | if (FILESYSTEM_CHANGED_TB(tb)) { |
1027 | brelse(*pp_s_com_father); | 1027 | brelse(*pcom_father); |
1028 | return REPEAT_SEARCH; | 1028 | return REPEAT_SEARCH; |
1029 | } | 1029 | } |
1030 | } | 1030 | } |
@@ -1034,7 +1034,7 @@ static int get_far_parent(struct tree_balance *tb, | |||
1034 | 1034 | ||
1035 | /* Form key to get parent of the left/right neighbor. */ | 1035 | /* Form key to get parent of the left/right neighbor. */ |
1036 | le_key2cpu_key(&s_lr_father_key, | 1036 | le_key2cpu_key(&s_lr_father_key, |
1037 | B_N_PDELIM_KEY(*pp_s_com_father, | 1037 | B_N_PDELIM_KEY(*pcom_father, |
1038 | (c_lr_par == | 1038 | (c_lr_par == |
1039 | LEFT_PARENTS) ? (tb->lkey[n_h - 1] = | 1039 | LEFT_PARENTS) ? (tb->lkey[n_h - 1] = |
1040 | n_position - | 1040 | n_position - |
@@ -1053,14 +1053,14 @@ static int get_far_parent(struct tree_balance *tb, | |||
1053 | 1053 | ||
1054 | if (FILESYSTEM_CHANGED_TB(tb)) { | 1054 | if (FILESYSTEM_CHANGED_TB(tb)) { |
1055 | pathrelse(&s_path_to_neighbor_father); | 1055 | pathrelse(&s_path_to_neighbor_father); |
1056 | brelse(*pp_s_com_father); | 1056 | brelse(*pcom_father); |
1057 | return REPEAT_SEARCH; | 1057 | return REPEAT_SEARCH; |
1058 | } | 1058 | } |
1059 | 1059 | ||
1060 | *pp_s_father = PATH_PLAST_BUFFER(&s_path_to_neighbor_father); | 1060 | *pfather = PATH_PLAST_BUFFER(&s_path_to_neighbor_father); |
1061 | 1061 | ||
1062 | RFALSE(B_LEVEL(*pp_s_father) != n_h + 1, | 1062 | RFALSE(B_LEVEL(*pfather) != n_h + 1, |
1063 | "PAP-8190: (%b %z) level too small", *pp_s_father, *pp_s_father); | 1063 | "PAP-8190: (%b %z) level too small", *pfather, *pfather); |
1064 | RFALSE(s_path_to_neighbor_father.path_length < | 1064 | RFALSE(s_path_to_neighbor_father.path_length < |
1065 | FIRST_PATH_ELEMENT_OFFSET, "PAP-8192: path length is too small"); | 1065 | FIRST_PATH_ELEMENT_OFFSET, "PAP-8192: path length is too small"); |
1066 | 1066 | ||
@@ -1078,11 +1078,11 @@ static int get_far_parent(struct tree_balance *tb, | |||
1078 | */ | 1078 | */ |
1079 | static int get_parents(struct tree_balance *tb, int n_h) | 1079 | static int get_parents(struct tree_balance *tb, int n_h) |
1080 | { | 1080 | { |
1081 | struct treepath *p_s_path = tb->tb_path; | 1081 | struct treepath *path = tb->tb_path; |
1082 | int n_position, | 1082 | int n_position, |
1083 | n_ret_value, | 1083 | n_ret_value, |
1084 | n_path_offset = PATH_H_PATH_OFFSET(tb->tb_path, n_h); | 1084 | n_path_offset = PATH_H_PATH_OFFSET(tb->tb_path, n_h); |
1085 | struct buffer_head *p_s_curf, *p_s_curcf; | 1085 | struct buffer_head *curf, *curcf; |
1086 | 1086 | ||
1087 | /* Current node is the root of the tree or will be root of the tree */ | 1087 | /* Current node is the root of the tree or will be root of the tree */ |
1088 | if (n_path_offset <= FIRST_PATH_ELEMENT_OFFSET) { | 1088 | if (n_path_offset <= FIRST_PATH_ELEMENT_OFFSET) { |
@@ -1100,66 +1100,65 @@ static int get_parents(struct tree_balance *tb, int n_h) | |||
1100 | } | 1100 | } |
1101 | 1101 | ||
1102 | /* Get parent FL[n_path_offset] of L[n_path_offset]. */ | 1102 | /* Get parent FL[n_path_offset] of L[n_path_offset]. */ |
1103 | if ((n_position = PATH_OFFSET_POSITION(p_s_path, n_path_offset - 1))) { | 1103 | n_position = PATH_OFFSET_POSITION(path, n_path_offset - 1); |
1104 | if (n_position) { | ||
1104 | /* Current node is not the first child of its parent. */ | 1105 | /* Current node is not the first child of its parent. */ |
1105 | /*(p_s_curf = p_s_curcf = PATH_OFFSET_PBUFFER(p_s_path, n_path_offset - 1))->b_count += 2; */ | 1106 | curf = PATH_OFFSET_PBUFFER(path, n_path_offset - 1); |
1106 | p_s_curf = p_s_curcf = | 1107 | curcf = PATH_OFFSET_PBUFFER(path, n_path_offset - 1); |
1107 | PATH_OFFSET_PBUFFER(p_s_path, n_path_offset - 1); | 1108 | get_bh(curf); |
1108 | get_bh(p_s_curf); | 1109 | get_bh(curf); |
1109 | get_bh(p_s_curf); | ||
1110 | tb->lkey[n_h] = n_position - 1; | 1110 | tb->lkey[n_h] = n_position - 1; |
1111 | } else { | 1111 | } else { |
1112 | /* Calculate current parent of L[n_path_offset], which is the left neighbor of the current node. | 1112 | /* Calculate current parent of L[n_path_offset], which is the left neighbor of the current node. |
1113 | Calculate current common parent of L[n_path_offset] and the current node. Note that | 1113 | Calculate current common parent of L[n_path_offset] and the current node. Note that |
1114 | CFL[n_path_offset] not equal FL[n_path_offset] and CFL[n_path_offset] not equal F[n_path_offset]. | 1114 | CFL[n_path_offset] not equal FL[n_path_offset] and CFL[n_path_offset] not equal F[n_path_offset]. |
1115 | Calculate lkey[n_path_offset]. */ | 1115 | Calculate lkey[n_path_offset]. */ |
1116 | if ((n_ret_value = get_far_parent(tb, n_h + 1, &p_s_curf, | 1116 | if ((n_ret_value = get_far_parent(tb, n_h + 1, &curf, |
1117 | &p_s_curcf, | 1117 | &curcf, |
1118 | LEFT_PARENTS)) != CARRY_ON) | 1118 | LEFT_PARENTS)) != CARRY_ON) |
1119 | return n_ret_value; | 1119 | return n_ret_value; |
1120 | } | 1120 | } |
1121 | 1121 | ||
1122 | brelse(tb->FL[n_h]); | 1122 | brelse(tb->FL[n_h]); |
1123 | tb->FL[n_h] = p_s_curf; /* New initialization of FL[n_h]. */ | 1123 | tb->FL[n_h] = curf; /* New initialization of FL[n_h]. */ |
1124 | brelse(tb->CFL[n_h]); | 1124 | brelse(tb->CFL[n_h]); |
1125 | tb->CFL[n_h] = p_s_curcf; /* New initialization of CFL[n_h]. */ | 1125 | tb->CFL[n_h] = curcf; /* New initialization of CFL[n_h]. */ |
1126 | 1126 | ||
1127 | RFALSE((p_s_curf && !B_IS_IN_TREE(p_s_curf)) || | 1127 | RFALSE((curf && !B_IS_IN_TREE(curf)) || |
1128 | (p_s_curcf && !B_IS_IN_TREE(p_s_curcf)), | 1128 | (curcf && !B_IS_IN_TREE(curcf)), |
1129 | "PAP-8195: FL (%b) or CFL (%b) is invalid", p_s_curf, p_s_curcf); | 1129 | "PAP-8195: FL (%b) or CFL (%b) is invalid", curf, curcf); |
1130 | 1130 | ||
1131 | /* Get parent FR[n_h] of R[n_h]. */ | 1131 | /* Get parent FR[n_h] of R[n_h]. */ |
1132 | 1132 | ||
1133 | /* Current node is the last child of F[n_h]. FR[n_h] != F[n_h]. */ | 1133 | /* Current node is the last child of F[n_h]. FR[n_h] != F[n_h]. */ |
1134 | if (n_position == B_NR_ITEMS(PATH_H_PBUFFER(p_s_path, n_h + 1))) { | 1134 | if (n_position == B_NR_ITEMS(PATH_H_PBUFFER(path, n_h + 1))) { |
1135 | /* Calculate current parent of R[n_h], which is the right neighbor of F[n_h]. | 1135 | /* Calculate current parent of R[n_h], which is the right neighbor of F[n_h]. |
1136 | Calculate current common parent of R[n_h] and current node. Note that CFR[n_h] | 1136 | Calculate current common parent of R[n_h] and current node. Note that CFR[n_h] |
1137 | not equal FR[n_path_offset] and CFR[n_h] not equal F[n_h]. */ | 1137 | not equal FR[n_path_offset] and CFR[n_h] not equal F[n_h]. */ |
1138 | if ((n_ret_value = | 1138 | if ((n_ret_value = |
1139 | get_far_parent(tb, n_h + 1, &p_s_curf, &p_s_curcf, | 1139 | get_far_parent(tb, n_h + 1, &curf, &curcf, |
1140 | RIGHT_PARENTS)) != CARRY_ON) | 1140 | RIGHT_PARENTS)) != CARRY_ON) |
1141 | return n_ret_value; | 1141 | return n_ret_value; |
1142 | } else { | 1142 | } else { |
1143 | /* Current node is not the last child of its parent F[n_h]. */ | 1143 | /* Current node is not the last child of its parent F[n_h]. */ |
1144 | /*(p_s_curf = p_s_curcf = PATH_OFFSET_PBUFFER(p_s_path, n_path_offset - 1))->b_count += 2; */ | 1144 | curf = PATH_OFFSET_PBUFFER(path, n_path_offset - 1); |
1145 | p_s_curf = p_s_curcf = | 1145 | curcf = PATH_OFFSET_PBUFFER(path, n_path_offset - 1); |
1146 | PATH_OFFSET_PBUFFER(p_s_path, n_path_offset - 1); | 1146 | get_bh(curf); |
1147 | get_bh(p_s_curf); | 1147 | get_bh(curf); |
1148 | get_bh(p_s_curf); | ||
1149 | tb->rkey[n_h] = n_position; | 1148 | tb->rkey[n_h] = n_position; |
1150 | } | 1149 | } |
1151 | 1150 | ||
1152 | brelse(tb->FR[n_h]); | 1151 | brelse(tb->FR[n_h]); |
1153 | /* New initialization of FR[n_path_offset]. */ | 1152 | /* New initialization of FR[n_path_offset]. */ |
1154 | tb->FR[n_h] = p_s_curf; | 1153 | tb->FR[n_h] = curf; |
1155 | 1154 | ||
1156 | brelse(tb->CFR[n_h]); | 1155 | brelse(tb->CFR[n_h]); |
1157 | /* New initialization of CFR[n_path_offset]. */ | 1156 | /* New initialization of CFR[n_path_offset]. */ |
1158 | tb->CFR[n_h] = p_s_curcf; | 1157 | tb->CFR[n_h] = curcf; |
1159 | 1158 | ||
1160 | RFALSE((p_s_curf && !B_IS_IN_TREE(p_s_curf)) || | 1159 | RFALSE((curf && !B_IS_IN_TREE(curf)) || |
1161 | (p_s_curcf && !B_IS_IN_TREE(p_s_curcf)), | 1160 | (curcf && !B_IS_IN_TREE(curcf)), |
1162 | "PAP-8205: FR (%b) or CFR (%b) is invalid", p_s_curf, p_s_curcf); | 1161 | "PAP-8205: FR (%b) or CFR (%b) is invalid", curf, curcf); |
1163 | 1162 | ||
1164 | return CARRY_ON; | 1163 | return CARRY_ON; |
1165 | } | 1164 | } |
@@ -1893,7 +1892,7 @@ static int check_balance(int mode, | |||
1893 | static int get_direct_parent(struct tree_balance *tb, int n_h) | 1892 | static int get_direct_parent(struct tree_balance *tb, int n_h) |
1894 | { | 1893 | { |
1895 | struct buffer_head *bh; | 1894 | struct buffer_head *bh; |
1896 | struct treepath *p_s_path = tb->tb_path; | 1895 | struct treepath *path = tb->tb_path; |
1897 | int n_position, | 1896 | int n_position, |
1898 | n_path_offset = PATH_H_PATH_OFFSET(tb->tb_path, n_h); | 1897 | n_path_offset = PATH_H_PATH_OFFSET(tb->tb_path, n_h); |
1899 | 1898 | ||
@@ -1903,27 +1902,27 @@ static int get_direct_parent(struct tree_balance *tb, int n_h) | |||
1903 | RFALSE(n_path_offset < FIRST_PATH_ELEMENT_OFFSET - 1, | 1902 | RFALSE(n_path_offset < FIRST_PATH_ELEMENT_OFFSET - 1, |
1904 | "PAP-8260: invalid offset in the path"); | 1903 | "PAP-8260: invalid offset in the path"); |
1905 | 1904 | ||
1906 | if (PATH_OFFSET_PBUFFER(p_s_path, FIRST_PATH_ELEMENT_OFFSET)-> | 1905 | if (PATH_OFFSET_PBUFFER(path, FIRST_PATH_ELEMENT_OFFSET)-> |
1907 | b_blocknr == SB_ROOT_BLOCK(tb->tb_sb)) { | 1906 | b_blocknr == SB_ROOT_BLOCK(tb->tb_sb)) { |
1908 | /* Root is not changed. */ | 1907 | /* Root is not changed. */ |
1909 | PATH_OFFSET_PBUFFER(p_s_path, n_path_offset - 1) = NULL; | 1908 | PATH_OFFSET_PBUFFER(path, n_path_offset - 1) = NULL; |
1910 | PATH_OFFSET_POSITION(p_s_path, n_path_offset - 1) = 0; | 1909 | PATH_OFFSET_POSITION(path, n_path_offset - 1) = 0; |
1911 | return CARRY_ON; | 1910 | return CARRY_ON; |
1912 | } | 1911 | } |
1913 | return REPEAT_SEARCH; /* Root is changed and we must recalculate the path. */ | 1912 | return REPEAT_SEARCH; /* Root is changed and we must recalculate the path. */ |
1914 | } | 1913 | } |
1915 | 1914 | ||
1916 | if (!B_IS_IN_TREE | 1915 | if (!B_IS_IN_TREE |
1917 | (bh = PATH_OFFSET_PBUFFER(p_s_path, n_path_offset - 1))) | 1916 | (bh = PATH_OFFSET_PBUFFER(path, n_path_offset - 1))) |
1918 | return REPEAT_SEARCH; /* Parent in the path is not in the tree. */ | 1917 | return REPEAT_SEARCH; /* Parent in the path is not in the tree. */ |
1919 | 1918 | ||
1920 | if ((n_position = | 1919 | if ((n_position = |
1921 | PATH_OFFSET_POSITION(p_s_path, | 1920 | PATH_OFFSET_POSITION(path, |
1922 | n_path_offset - 1)) > B_NR_ITEMS(bh)) | 1921 | n_path_offset - 1)) > B_NR_ITEMS(bh)) |
1923 | return REPEAT_SEARCH; | 1922 | return REPEAT_SEARCH; |
1924 | 1923 | ||
1925 | if (B_N_CHILD_NUM(bh, n_position) != | 1924 | if (B_N_CHILD_NUM(bh, n_position) != |
1926 | PATH_OFFSET_PBUFFER(p_s_path, n_path_offset)->b_blocknr) | 1925 | PATH_OFFSET_PBUFFER(path, n_path_offset)->b_blocknr) |
1927 | /* Parent in the path is not parent of the current node in the tree. */ | 1926 | /* Parent in the path is not parent of the current node in the tree. */ |
1928 | return REPEAT_SEARCH; | 1927 | return REPEAT_SEARCH; |
1929 | 1928 | ||
@@ -2319,7 +2318,7 @@ static int wait_tb_buffers_until_unlocked(struct tree_balance *tb) | |||
2319 | */ | 2318 | */ |
2320 | 2319 | ||
2321 | int fix_nodes(int n_op_mode, struct tree_balance *tb, | 2320 | int fix_nodes(int n_op_mode, struct tree_balance *tb, |
2322 | struct item_head *p_s_ins_ih, const void *data) | 2321 | struct item_head *ins_ih, const void *data) |
2323 | { | 2322 | { |
2324 | int n_ret_value, n_h, n_item_num = PATH_LAST_POSITION(tb->tb_path); | 2323 | int n_ret_value, n_h, n_item_num = PATH_LAST_POSITION(tb->tb_path); |
2325 | int n_pos_in_item; | 2324 | int n_pos_in_item; |
@@ -2405,7 +2404,7 @@ int fix_nodes(int n_op_mode, struct tree_balance *tb, | |||
2405 | goto repeat; | 2404 | goto repeat; |
2406 | 2405 | ||
2407 | n_ret_value = check_balance(n_op_mode, tb, n_h, n_item_num, | 2406 | n_ret_value = check_balance(n_op_mode, tb, n_h, n_item_num, |
2408 | n_pos_in_item, p_s_ins_ih, data); | 2407 | n_pos_in_item, ins_ih, data); |
2409 | if (n_ret_value != CARRY_ON) { | 2408 | if (n_ret_value != CARRY_ON) { |
2410 | if (n_ret_value == NO_BALANCING_NEEDED) { | 2409 | if (n_ret_value == NO_BALANCING_NEEDED) { |
2411 | /* No balancing for higher levels needed. */ | 2410 | /* No balancing for higher levels needed. */ |
diff --git a/fs/reiserfs/stree.c b/fs/reiserfs/stree.c index 5e867be559ea..fd769c8dac32 100644 --- a/fs/reiserfs/stree.c +++ b/fs/reiserfs/stree.c | |||
@@ -68,10 +68,10 @@ inline int B_IS_IN_TREE(const struct buffer_head *bh) | |||
68 | // | 68 | // |
69 | // to gets item head in le form | 69 | // to gets item head in le form |
70 | // | 70 | // |
71 | inline void copy_item_head(struct item_head *p_v_to, | 71 | inline void copy_item_head(struct item_head *to, |
72 | const struct item_head *p_v_from) | 72 | const struct item_head *from) |
73 | { | 73 | { |
74 | memcpy(p_v_to, p_v_from, IH_SIZE); | 74 | memcpy(to, from, IH_SIZE); |
75 | } | 75 | } |
76 | 76 | ||
77 | /* k1 is pointer to on-disk structure which is stored in little-endian | 77 | /* k1 is pointer to on-disk structure which is stored in little-endian |
@@ -135,15 +135,15 @@ static inline int comp_keys(const struct reiserfs_key *le_key, | |||
135 | inline int comp_short_le_keys(const struct reiserfs_key *key1, | 135 | inline int comp_short_le_keys(const struct reiserfs_key *key1, |
136 | const struct reiserfs_key *key2) | 136 | const struct reiserfs_key *key2) |
137 | { | 137 | { |
138 | __u32 *p_s_1_u32, *p_s_2_u32; | 138 | __u32 *k1_u32, *k2_u32; |
139 | int n_key_length = REISERFS_SHORT_KEY_LEN; | 139 | int n_key_length = REISERFS_SHORT_KEY_LEN; |
140 | 140 | ||
141 | p_s_1_u32 = (__u32 *) key1; | 141 | k1_u32 = (__u32 *) key1; |
142 | p_s_2_u32 = (__u32 *) key2; | 142 | k2_u32 = (__u32 *) key2; |
143 | for (; n_key_length--; ++p_s_1_u32, ++p_s_2_u32) { | 143 | for (; n_key_length--; ++k1_u32, ++k2_u32) { |
144 | if (le32_to_cpu(*p_s_1_u32) < le32_to_cpu(*p_s_2_u32)) | 144 | if (le32_to_cpu(*k1_u32) < le32_to_cpu(*k2_u32)) |
145 | return -1; | 145 | return -1; |
146 | if (le32_to_cpu(*p_s_1_u32) > le32_to_cpu(*p_s_2_u32)) | 146 | if (le32_to_cpu(*k1_u32) > le32_to_cpu(*k2_u32)) |
147 | return 1; | 147 | return 1; |
148 | } | 148 | } |
149 | return 0; | 149 | return 0; |
@@ -174,8 +174,8 @@ inline int comp_le_keys(const struct reiserfs_key *k1, | |||
174 | * Binary search toolkit function * | 174 | * Binary search toolkit function * |
175 | * Search for an item in the array by the item key * | 175 | * Search for an item in the array by the item key * |
176 | * Returns: 1 if found, 0 if not found; * | 176 | * Returns: 1 if found, 0 if not found; * |
177 | * *p_n_pos = number of the searched element if found, else the * | 177 | * *pos = number of the searched element if found, else the * |
178 | * number of the first element that is larger than p_v_key. * | 178 | * number of the first element that is larger than key. * |
179 | **************************************************************************/ | 179 | **************************************************************************/ |
180 | /* For those not familiar with binary search: n_lbound is the leftmost item that it | 180 | /* For those not familiar with binary search: n_lbound is the leftmost item that it |
181 | could be, n_rbound the rightmost item that it could be. We examine the item | 181 | could be, n_rbound the rightmost item that it could be. We examine the item |
@@ -184,28 +184,28 @@ inline int comp_le_keys(const struct reiserfs_key *k1, | |||
184 | there are no possible items, and we have not found it. With each examination we | 184 | there are no possible items, and we have not found it. With each examination we |
185 | cut the number of possible items it could be by one more than half rounded down, | 185 | cut the number of possible items it could be by one more than half rounded down, |
186 | or we find it. */ | 186 | or we find it. */ |
187 | static inline int bin_search(const void *p_v_key, /* Key to search for. */ | 187 | static inline int bin_search(const void *key, /* Key to search for. */ |
188 | const void *p_v_base, /* First item in the array. */ | 188 | const void *base, /* First item in the array. */ |
189 | int p_n_num, /* Number of items in the array. */ | 189 | int num, /* Number of items in the array. */ |
190 | int p_n_width, /* Item size in the array. | 190 | int width, /* Item size in the array. |
191 | searched. Lest the reader be | 191 | searched. Lest the reader be |
192 | confused, note that this is crafted | 192 | confused, note that this is crafted |
193 | as a general function, and when it | 193 | as a general function, and when it |
194 | is applied specifically to the array | 194 | is applied specifically to the array |
195 | of item headers in a node, p_n_width | 195 | of item headers in a node, width |
196 | is actually the item header size not | 196 | is actually the item header size not |
197 | the item size. */ | 197 | the item size. */ |
198 | int *p_n_pos /* Number of the searched for element. */ | 198 | int *pos /* Number of the searched for element. */ |
199 | ) | 199 | ) |
200 | { | 200 | { |
201 | int n_rbound, n_lbound, n_j; | 201 | int n_rbound, n_lbound, n_j; |
202 | 202 | ||
203 | for (n_j = ((n_rbound = p_n_num - 1) + (n_lbound = 0)) / 2; | 203 | for (n_j = ((n_rbound = num - 1) + (n_lbound = 0)) / 2; |
204 | n_lbound <= n_rbound; n_j = (n_rbound + n_lbound) / 2) | 204 | n_lbound <= n_rbound; n_j = (n_rbound + n_lbound) / 2) |
205 | switch (comp_keys | 205 | switch (comp_keys |
206 | ((struct reiserfs_key *)((char *)p_v_base + | 206 | ((struct reiserfs_key *)((char *)base + |
207 | n_j * p_n_width), | 207 | n_j * width), |
208 | (struct cpu_key *)p_v_key)) { | 208 | (struct cpu_key *)key)) { |
209 | case -1: | 209 | case -1: |
210 | n_lbound = n_j + 1; | 210 | n_lbound = n_j + 1; |
211 | continue; | 211 | continue; |
@@ -213,13 +213,13 @@ static inline int bin_search(const void *p_v_key, /* Key to search for. | |||
213 | n_rbound = n_j - 1; | 213 | n_rbound = n_j - 1; |
214 | continue; | 214 | continue; |
215 | case 0: | 215 | case 0: |
216 | *p_n_pos = n_j; | 216 | *pos = n_j; |
217 | return ITEM_FOUND; /* Key found in the array. */ | 217 | return ITEM_FOUND; /* Key found in the array. */ |
218 | } | 218 | } |
219 | 219 | ||
220 | /* bin_search did not find given key, it returns position of key, | 220 | /* bin_search did not find given key, it returns position of key, |
221 | that is minimal and greater than the given one. */ | 221 | that is minimal and greater than the given one. */ |
222 | *p_n_pos = n_lbound; | 222 | *pos = n_lbound; |
223 | return ITEM_NOT_FOUND; | 223 | return ITEM_NOT_FOUND; |
224 | } | 224 | } |
225 | 225 | ||
@@ -243,12 +243,12 @@ static const struct reiserfs_key MAX_KEY = { | |||
243 | the path, there is no delimiting key in the tree (buffer is first or last buffer in tree), and in this | 243 | the path, there is no delimiting key in the tree (buffer is first or last buffer in tree), and in this |
244 | case we return a special key, either MIN_KEY or MAX_KEY. */ | 244 | case we return a special key, either MIN_KEY or MAX_KEY. */ |
245 | static inline const struct reiserfs_key *get_lkey(const struct treepath | 245 | static inline const struct reiserfs_key *get_lkey(const struct treepath |
246 | *p_s_chk_path, | 246 | *chk_path, |
247 | const struct super_block | 247 | const struct super_block |
248 | *sb) | 248 | *sb) |
249 | { | 249 | { |
250 | int n_position, n_path_offset = p_s_chk_path->path_length; | 250 | int n_position, n_path_offset = chk_path->path_length; |
251 | struct buffer_head *p_s_parent; | 251 | struct buffer_head *parent; |
252 | 252 | ||
253 | RFALSE(n_path_offset < FIRST_PATH_ELEMENT_OFFSET, | 253 | RFALSE(n_path_offset < FIRST_PATH_ELEMENT_OFFSET, |
254 | "PAP-5010: invalid offset in the path"); | 254 | "PAP-5010: invalid offset in the path"); |
@@ -257,42 +257,42 @@ static inline const struct reiserfs_key *get_lkey(const struct treepath | |||
257 | while (n_path_offset-- > FIRST_PATH_ELEMENT_OFFSET) { | 257 | while (n_path_offset-- > FIRST_PATH_ELEMENT_OFFSET) { |
258 | 258 | ||
259 | RFALSE(!buffer_uptodate | 259 | RFALSE(!buffer_uptodate |
260 | (PATH_OFFSET_PBUFFER(p_s_chk_path, n_path_offset)), | 260 | (PATH_OFFSET_PBUFFER(chk_path, n_path_offset)), |
261 | "PAP-5020: parent is not uptodate"); | 261 | "PAP-5020: parent is not uptodate"); |
262 | 262 | ||
263 | /* Parent at the path is not in the tree now. */ | 263 | /* Parent at the path is not in the tree now. */ |
264 | if (!B_IS_IN_TREE | 264 | if (!B_IS_IN_TREE |
265 | (p_s_parent = | 265 | (parent = |
266 | PATH_OFFSET_PBUFFER(p_s_chk_path, n_path_offset))) | 266 | PATH_OFFSET_PBUFFER(chk_path, n_path_offset))) |
267 | return &MAX_KEY; | 267 | return &MAX_KEY; |
268 | /* Check whether position in the parent is correct. */ | 268 | /* Check whether position in the parent is correct. */ |
269 | if ((n_position = | 269 | if ((n_position = |
270 | PATH_OFFSET_POSITION(p_s_chk_path, | 270 | PATH_OFFSET_POSITION(chk_path, |
271 | n_path_offset)) > | 271 | n_path_offset)) > |
272 | B_NR_ITEMS(p_s_parent)) | 272 | B_NR_ITEMS(parent)) |
273 | return &MAX_KEY; | 273 | return &MAX_KEY; |
274 | /* Check whether parent at the path really points to the child. */ | 274 | /* Check whether parent at the path really points to the child. */ |
275 | if (B_N_CHILD_NUM(p_s_parent, n_position) != | 275 | if (B_N_CHILD_NUM(parent, n_position) != |
276 | PATH_OFFSET_PBUFFER(p_s_chk_path, | 276 | PATH_OFFSET_PBUFFER(chk_path, |
277 | n_path_offset + 1)->b_blocknr) | 277 | n_path_offset + 1)->b_blocknr) |
278 | return &MAX_KEY; | 278 | return &MAX_KEY; |
279 | /* Return delimiting key if position in the parent is not equal to zero. */ | 279 | /* Return delimiting key if position in the parent is not equal to zero. */ |
280 | if (n_position) | 280 | if (n_position) |
281 | return B_N_PDELIM_KEY(p_s_parent, n_position - 1); | 281 | return B_N_PDELIM_KEY(parent, n_position - 1); |
282 | } | 282 | } |
283 | /* Return MIN_KEY if we are in the root of the buffer tree. */ | 283 | /* Return MIN_KEY if we are in the root of the buffer tree. */ |
284 | if (PATH_OFFSET_PBUFFER(p_s_chk_path, FIRST_PATH_ELEMENT_OFFSET)-> | 284 | if (PATH_OFFSET_PBUFFER(chk_path, FIRST_PATH_ELEMENT_OFFSET)-> |
285 | b_blocknr == SB_ROOT_BLOCK(sb)) | 285 | b_blocknr == SB_ROOT_BLOCK(sb)) |
286 | return &MIN_KEY; | 286 | return &MIN_KEY; |
287 | return &MAX_KEY; | 287 | return &MAX_KEY; |
288 | } | 288 | } |
289 | 289 | ||
290 | /* Get delimiting key of the buffer at the path and its right neighbor. */ | 290 | /* Get delimiting key of the buffer at the path and its right neighbor. */ |
291 | inline const struct reiserfs_key *get_rkey(const struct treepath *p_s_chk_path, | 291 | inline const struct reiserfs_key *get_rkey(const struct treepath *chk_path, |
292 | const struct super_block *sb) | 292 | const struct super_block *sb) |
293 | { | 293 | { |
294 | int n_position, n_path_offset = p_s_chk_path->path_length; | 294 | int n_position, n_path_offset = chk_path->path_length; |
295 | struct buffer_head *p_s_parent; | 295 | struct buffer_head *parent; |
296 | 296 | ||
297 | RFALSE(n_path_offset < FIRST_PATH_ELEMENT_OFFSET, | 297 | RFALSE(n_path_offset < FIRST_PATH_ELEMENT_OFFSET, |
298 | "PAP-5030: invalid offset in the path"); | 298 | "PAP-5030: invalid offset in the path"); |
@@ -300,31 +300,31 @@ inline const struct reiserfs_key *get_rkey(const struct treepath *p_s_chk_path, | |||
300 | while (n_path_offset-- > FIRST_PATH_ELEMENT_OFFSET) { | 300 | while (n_path_offset-- > FIRST_PATH_ELEMENT_OFFSET) { |
301 | 301 | ||
302 | RFALSE(!buffer_uptodate | 302 | RFALSE(!buffer_uptodate |
303 | (PATH_OFFSET_PBUFFER(p_s_chk_path, n_path_offset)), | 303 | (PATH_OFFSET_PBUFFER(chk_path, n_path_offset)), |
304 | "PAP-5040: parent is not uptodate"); | 304 | "PAP-5040: parent is not uptodate"); |
305 | 305 | ||
306 | /* Parent at the path is not in the tree now. */ | 306 | /* Parent at the path is not in the tree now. */ |
307 | if (!B_IS_IN_TREE | 307 | if (!B_IS_IN_TREE |
308 | (p_s_parent = | 308 | (parent = |
309 | PATH_OFFSET_PBUFFER(p_s_chk_path, n_path_offset))) | 309 | PATH_OFFSET_PBUFFER(chk_path, n_path_offset))) |
310 | return &MIN_KEY; | 310 | return &MIN_KEY; |
311 | /* Check whether position in the parent is correct. */ | 311 | /* Check whether position in the parent is correct. */ |
312 | if ((n_position = | 312 | if ((n_position = |
313 | PATH_OFFSET_POSITION(p_s_chk_path, | 313 | PATH_OFFSET_POSITION(chk_path, |
314 | n_path_offset)) > | 314 | n_path_offset)) > |
315 | B_NR_ITEMS(p_s_parent)) | 315 | B_NR_ITEMS(parent)) |
316 | return &MIN_KEY; | 316 | return &MIN_KEY; |
317 | /* Check whether parent at the path really points to the child. */ | 317 | /* Check whether parent at the path really points to the child. */ |
318 | if (B_N_CHILD_NUM(p_s_parent, n_position) != | 318 | if (B_N_CHILD_NUM(parent, n_position) != |
319 | PATH_OFFSET_PBUFFER(p_s_chk_path, | 319 | PATH_OFFSET_PBUFFER(chk_path, |
320 | n_path_offset + 1)->b_blocknr) | 320 | n_path_offset + 1)->b_blocknr) |
321 | return &MIN_KEY; | 321 | return &MIN_KEY; |
322 | /* Return delimiting key if position in the parent is not the last one. */ | 322 | /* Return delimiting key if position in the parent is not the last one. */ |
323 | if (n_position != B_NR_ITEMS(p_s_parent)) | 323 | if (n_position != B_NR_ITEMS(parent)) |
324 | return B_N_PDELIM_KEY(p_s_parent, n_position); | 324 | return B_N_PDELIM_KEY(parent, n_position); |
325 | } | 325 | } |
326 | /* Return MAX_KEY if we are in the root of the buffer tree. */ | 326 | /* Return MAX_KEY if we are in the root of the buffer tree. */ |
327 | if (PATH_OFFSET_PBUFFER(p_s_chk_path, FIRST_PATH_ELEMENT_OFFSET)-> | 327 | if (PATH_OFFSET_PBUFFER(chk_path, FIRST_PATH_ELEMENT_OFFSET)-> |
328 | b_blocknr == SB_ROOT_BLOCK(sb)) | 328 | b_blocknr == SB_ROOT_BLOCK(sb)) |
329 | return &MAX_KEY; | 329 | return &MAX_KEY; |
330 | return &MIN_KEY; | 330 | return &MIN_KEY; |
@@ -335,25 +335,25 @@ inline const struct reiserfs_key *get_rkey(const struct treepath *p_s_chk_path, | |||
335 | the path. These delimiting keys are stored at least one level above that buffer in the tree. If the | 335 | the path. These delimiting keys are stored at least one level above that buffer in the tree. If the |
336 | buffer is the first or last node in the tree order then one of the delimiting keys may be absent, and in | 336 | buffer is the first or last node in the tree order then one of the delimiting keys may be absent, and in |
337 | this case get_lkey and get_rkey return a special key which is MIN_KEY or MAX_KEY. */ | 337 | this case get_lkey and get_rkey return a special key which is MIN_KEY or MAX_KEY. */ |
338 | static inline int key_in_buffer(struct treepath *p_s_chk_path, /* Path which should be checked. */ | 338 | static inline int key_in_buffer(struct treepath *chk_path, /* Path which should be checked. */ |
339 | const struct cpu_key *p_s_key, /* Key which should be checked. */ | 339 | const struct cpu_key *key, /* Key which should be checked. */ |
340 | struct super_block *sb /* Super block pointer. */ | 340 | struct super_block *sb |
341 | ) | 341 | ) |
342 | { | 342 | { |
343 | 343 | ||
344 | RFALSE(!p_s_key || p_s_chk_path->path_length < FIRST_PATH_ELEMENT_OFFSET | 344 | RFALSE(!key || chk_path->path_length < FIRST_PATH_ELEMENT_OFFSET |
345 | || p_s_chk_path->path_length > MAX_HEIGHT, | 345 | || chk_path->path_length > MAX_HEIGHT, |
346 | "PAP-5050: pointer to the key(%p) is NULL or invalid path length(%d)", | 346 | "PAP-5050: pointer to the key(%p) is NULL or invalid path length(%d)", |
347 | p_s_key, p_s_chk_path->path_length); | 347 | key, chk_path->path_length); |
348 | RFALSE(!PATH_PLAST_BUFFER(p_s_chk_path)->b_bdev, | 348 | RFALSE(!PATH_PLAST_BUFFER(chk_path)->b_bdev, |
349 | "PAP-5060: device must not be NODEV"); | 349 | "PAP-5060: device must not be NODEV"); |
350 | 350 | ||
351 | if (comp_keys(get_lkey(p_s_chk_path, sb), p_s_key) == 1) | 351 | if (comp_keys(get_lkey(chk_path, sb), key) == 1) |
352 | /* left delimiting key is bigger, that the key we look for */ | 352 | /* left delimiting key is bigger, that the key we look for */ |
353 | return 0; | 353 | return 0; |
354 | // if ( comp_keys(p_s_key, get_rkey(p_s_chk_path, sb)) != -1 ) | 354 | /* if ( comp_keys(key, get_rkey(chk_path, sb)) != -1 ) */ |
355 | if (comp_keys(get_rkey(p_s_chk_path, sb), p_s_key) != 1) | 355 | if (comp_keys(get_rkey(chk_path, sb), key) != 1) |
356 | /* p_s_key must be less than right delimitiing key */ | 356 | /* key must be less than right delimitiing key */ |
357 | return 0; | 357 | return 0; |
358 | return 1; | 358 | return 1; |
359 | } | 359 | } |
@@ -369,34 +369,34 @@ int reiserfs_check_path(struct treepath *p) | |||
369 | * dirty bits clean when preparing the buffer for the log. | 369 | * dirty bits clean when preparing the buffer for the log. |
370 | * This version should only be called from fix_nodes() */ | 370 | * This version should only be called from fix_nodes() */ |
371 | void pathrelse_and_restore(struct super_block *sb, | 371 | void pathrelse_and_restore(struct super_block *sb, |
372 | struct treepath *p_s_search_path) | 372 | struct treepath *search_path) |
373 | { | 373 | { |
374 | int n_path_offset = p_s_search_path->path_length; | 374 | int n_path_offset = search_path->path_length; |
375 | 375 | ||
376 | RFALSE(n_path_offset < ILLEGAL_PATH_ELEMENT_OFFSET, | 376 | RFALSE(n_path_offset < ILLEGAL_PATH_ELEMENT_OFFSET, |
377 | "clm-4000: invalid path offset"); | 377 | "clm-4000: invalid path offset"); |
378 | 378 | ||
379 | while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) { | 379 | while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) { |
380 | struct buffer_head *bh; | 380 | struct buffer_head *bh; |
381 | bh = PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--); | 381 | bh = PATH_OFFSET_PBUFFER(search_path, n_path_offset--); |
382 | reiserfs_restore_prepared_buffer(sb, bh); | 382 | reiserfs_restore_prepared_buffer(sb, bh); |
383 | brelse(bh); | 383 | brelse(bh); |
384 | } | 384 | } |
385 | p_s_search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET; | 385 | search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET; |
386 | } | 386 | } |
387 | 387 | ||
388 | /* Drop the reference to each buffer in a path */ | 388 | /* Drop the reference to each buffer in a path */ |
389 | void pathrelse(struct treepath *p_s_search_path) | 389 | void pathrelse(struct treepath *search_path) |
390 | { | 390 | { |
391 | int n_path_offset = p_s_search_path->path_length; | 391 | int n_path_offset = search_path->path_length; |
392 | 392 | ||
393 | RFALSE(n_path_offset < ILLEGAL_PATH_ELEMENT_OFFSET, | 393 | RFALSE(n_path_offset < ILLEGAL_PATH_ELEMENT_OFFSET, |
394 | "PAP-5090: invalid path offset"); | 394 | "PAP-5090: invalid path offset"); |
395 | 395 | ||
396 | while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) | 396 | while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) |
397 | brelse(PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--)); | 397 | brelse(PATH_OFFSET_PBUFFER(search_path, n_path_offset--)); |
398 | 398 | ||
399 | p_s_search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET; | 399 | search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET; |
400 | } | 400 | } |
401 | 401 | ||
402 | static int is_leaf(char *buf, int blocksize, struct buffer_head *bh) | 402 | static int is_leaf(char *buf, int blocksize, struct buffer_head *bh) |
@@ -547,9 +547,9 @@ static void search_by_key_reada(struct super_block *s, | |||
547 | * Algorithm SearchByKey * | 547 | * Algorithm SearchByKey * |
548 | * look for item in the Disk S+Tree by its key * | 548 | * look for item in the Disk S+Tree by its key * |
549 | * Input: sb - super block * | 549 | * Input: sb - super block * |
550 | * p_s_key - pointer to the key to search * | 550 | * key - pointer to the key to search * |
551 | * Output: ITEM_FOUND, ITEM_NOT_FOUND or IO_ERROR * | 551 | * Output: ITEM_FOUND, ITEM_NOT_FOUND or IO_ERROR * |
552 | * p_s_search_path - path from the root to the needed leaf * | 552 | * search_path - path from the root to the needed leaf * |
553 | **************************************************************************/ | 553 | **************************************************************************/ |
554 | 554 | ||
555 | /* This function fills up the path from the root to the leaf as it | 555 | /* This function fills up the path from the root to the leaf as it |
@@ -566,8 +566,8 @@ static void search_by_key_reada(struct super_block *s, | |||
566 | correctness of the top of the path but need not be checked for the | 566 | correctness of the top of the path but need not be checked for the |
567 | correctness of the bottom of the path */ | 567 | correctness of the bottom of the path */ |
568 | /* The function is NOT SCHEDULE-SAFE! */ | 568 | /* The function is NOT SCHEDULE-SAFE! */ |
569 | int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key to search. */ | 569 | int search_by_key(struct super_block *sb, const struct cpu_key *key, /* Key to search. */ |
570 | struct treepath *p_s_search_path,/* This structure was | 570 | struct treepath *search_path,/* This structure was |
571 | allocated and initialized | 571 | allocated and initialized |
572 | by the calling | 572 | by the calling |
573 | function. It is filled up | 573 | function. It is filled up |
@@ -580,7 +580,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
580 | b_blocknr_t n_block_number; | 580 | b_blocknr_t n_block_number; |
581 | int expected_level; | 581 | int expected_level; |
582 | struct buffer_head *bh; | 582 | struct buffer_head *bh; |
583 | struct path_element *p_s_last_element; | 583 | struct path_element *last_element; |
584 | int n_node_level, n_retval; | 584 | int n_node_level, n_retval; |
585 | int right_neighbor_of_leaf_node; | 585 | int right_neighbor_of_leaf_node; |
586 | int fs_gen; | 586 | int fs_gen; |
@@ -598,7 +598,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
598 | we must be careful to release all nodes in a path before we either | 598 | we must be careful to release all nodes in a path before we either |
599 | discard the path struct or re-use the path struct, as we do here. */ | 599 | discard the path struct or re-use the path struct, as we do here. */ |
600 | 600 | ||
601 | pathrelse(p_s_search_path); | 601 | pathrelse(search_path); |
602 | 602 | ||
603 | right_neighbor_of_leaf_node = 0; | 603 | right_neighbor_of_leaf_node = 0; |
604 | 604 | ||
@@ -615,18 +615,18 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
615 | "%s: there were %d iterations of " | 615 | "%s: there were %d iterations of " |
616 | "while loop looking for key %K", | 616 | "while loop looking for key %K", |
617 | current->comm, n_repeat_counter, | 617 | current->comm, n_repeat_counter, |
618 | p_s_key); | 618 | key); |
619 | #endif | 619 | #endif |
620 | 620 | ||
621 | /* prep path to have another element added to it. */ | 621 | /* prep path to have another element added to it. */ |
622 | p_s_last_element = | 622 | last_element = |
623 | PATH_OFFSET_PELEMENT(p_s_search_path, | 623 | PATH_OFFSET_PELEMENT(search_path, |
624 | ++p_s_search_path->path_length); | 624 | ++search_path->path_length); |
625 | fs_gen = get_generation(sb); | 625 | fs_gen = get_generation(sb); |
626 | 626 | ||
627 | /* Read the next tree node, and set the last element in the path to | 627 | /* Read the next tree node, and set the last element in the path to |
628 | have a pointer to it. */ | 628 | have a pointer to it. */ |
629 | if ((bh = p_s_last_element->pe_buffer = | 629 | if ((bh = last_element->pe_buffer = |
630 | sb_getblk(sb, n_block_number))) { | 630 | sb_getblk(sb, n_block_number))) { |
631 | if (!buffer_uptodate(bh) && reada_count > 1) | 631 | if (!buffer_uptodate(bh) && reada_count > 1) |
632 | search_by_key_reada(sb, reada_bh, | 632 | search_by_key_reada(sb, reada_bh, |
@@ -637,8 +637,8 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
637 | goto io_error; | 637 | goto io_error; |
638 | } else { | 638 | } else { |
639 | io_error: | 639 | io_error: |
640 | p_s_search_path->path_length--; | 640 | search_path->path_length--; |
641 | pathrelse(p_s_search_path); | 641 | pathrelse(search_path); |
642 | return IO_ERROR; | 642 | return IO_ERROR; |
643 | } | 643 | } |
644 | reada_count = 0; | 644 | reada_count = 0; |
@@ -652,12 +652,12 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
652 | if (fs_changed(fs_gen, sb) && | 652 | if (fs_changed(fs_gen, sb) && |
653 | (!B_IS_IN_TREE(bh) || | 653 | (!B_IS_IN_TREE(bh) || |
654 | B_LEVEL(bh) != expected_level || | 654 | B_LEVEL(bh) != expected_level || |
655 | !key_in_buffer(p_s_search_path, p_s_key, sb))) { | 655 | !key_in_buffer(search_path, key, sb))) { |
656 | PROC_INFO_INC(sb, search_by_key_fs_changed); | 656 | PROC_INFO_INC(sb, search_by_key_fs_changed); |
657 | PROC_INFO_INC(sb, search_by_key_restarted); | 657 | PROC_INFO_INC(sb, search_by_key_restarted); |
658 | PROC_INFO_INC(sb, | 658 | PROC_INFO_INC(sb, |
659 | sbk_restarted[expected_level - 1]); | 659 | sbk_restarted[expected_level - 1]); |
660 | pathrelse(p_s_search_path); | 660 | pathrelse(search_path); |
661 | 661 | ||
662 | /* Get the root block number so that we can repeat the search | 662 | /* Get the root block number so that we can repeat the search |
663 | starting from the root. */ | 663 | starting from the root. */ |
@@ -669,11 +669,11 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
669 | continue; | 669 | continue; |
670 | } | 670 | } |
671 | 671 | ||
672 | /* only check that the key is in the buffer if p_s_key is not | 672 | /* only check that the key is in the buffer if key is not |
673 | equal to the MAX_KEY. Latter case is only possible in | 673 | equal to the MAX_KEY. Latter case is only possible in |
674 | "finish_unfinished()" processing during mount. */ | 674 | "finish_unfinished()" processing during mount. */ |
675 | RFALSE(comp_keys(&MAX_KEY, p_s_key) && | 675 | RFALSE(comp_keys(&MAX_KEY, key) && |
676 | !key_in_buffer(p_s_search_path, p_s_key, sb), | 676 | !key_in_buffer(search_path, key, sb), |
677 | "PAP-5130: key is not in the buffer"); | 677 | "PAP-5130: key is not in the buffer"); |
678 | #ifdef CONFIG_REISERFS_CHECK | 678 | #ifdef CONFIG_REISERFS_CHECK |
679 | if (cur_tb) { | 679 | if (cur_tb) { |
@@ -689,7 +689,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
689 | reiserfs_error(sb, "vs-5150", | 689 | reiserfs_error(sb, "vs-5150", |
690 | "invalid format found in block %ld. " | 690 | "invalid format found in block %ld. " |
691 | "Fsck?", bh->b_blocknr); | 691 | "Fsck?", bh->b_blocknr); |
692 | pathrelse(p_s_search_path); | 692 | pathrelse(search_path); |
693 | return IO_ERROR; | 693 | return IO_ERROR; |
694 | } | 694 | } |
695 | 695 | ||
@@ -702,12 +702,12 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
702 | "vs-5152: tree level (%d) is less than stop level (%d)", | 702 | "vs-5152: tree level (%d) is less than stop level (%d)", |
703 | n_node_level, n_stop_level); | 703 | n_node_level, n_stop_level); |
704 | 704 | ||
705 | n_retval = bin_search(p_s_key, B_N_PITEM_HEAD(bh, 0), | 705 | n_retval = bin_search(key, B_N_PITEM_HEAD(bh, 0), |
706 | B_NR_ITEMS(bh), | 706 | B_NR_ITEMS(bh), |
707 | (n_node_level == | 707 | (n_node_level == |
708 | DISK_LEAF_NODE_LEVEL) ? IH_SIZE : | 708 | DISK_LEAF_NODE_LEVEL) ? IH_SIZE : |
709 | KEY_SIZE, | 709 | KEY_SIZE, |
710 | &(p_s_last_element->pe_position)); | 710 | &(last_element->pe_position)); |
711 | if (n_node_level == n_stop_level) { | 711 | if (n_node_level == n_stop_level) { |
712 | return n_retval; | 712 | return n_retval; |
713 | } | 713 | } |
@@ -715,7 +715,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
715 | /* we are not in the stop level */ | 715 | /* we are not in the stop level */ |
716 | if (n_retval == ITEM_FOUND) | 716 | if (n_retval == ITEM_FOUND) |
717 | /* item has been found, so we choose the pointer which is to the right of the found one */ | 717 | /* item has been found, so we choose the pointer which is to the right of the found one */ |
718 | p_s_last_element->pe_position++; | 718 | last_element->pe_position++; |
719 | 719 | ||
720 | /* if item was not found we choose the position which is to | 720 | /* if item was not found we choose the position which is to |
721 | the left of the found item. This requires no code, | 721 | the left of the found item. This requires no code, |
@@ -725,23 +725,23 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
725 | an internal node. Now we calculate child block number by | 725 | an internal node. Now we calculate child block number by |
726 | position in the node. */ | 726 | position in the node. */ |
727 | n_block_number = | 727 | n_block_number = |
728 | B_N_CHILD_NUM(bh, p_s_last_element->pe_position); | 728 | B_N_CHILD_NUM(bh, last_element->pe_position); |
729 | 729 | ||
730 | /* if we are going to read leaf nodes, try for read ahead as well */ | 730 | /* if we are going to read leaf nodes, try for read ahead as well */ |
731 | if ((p_s_search_path->reada & PATH_READA) && | 731 | if ((search_path->reada & PATH_READA) && |
732 | n_node_level == DISK_LEAF_NODE_LEVEL + 1) { | 732 | n_node_level == DISK_LEAF_NODE_LEVEL + 1) { |
733 | int pos = p_s_last_element->pe_position; | 733 | int pos = last_element->pe_position; |
734 | int limit = B_NR_ITEMS(bh); | 734 | int limit = B_NR_ITEMS(bh); |
735 | struct reiserfs_key *le_key; | 735 | struct reiserfs_key *le_key; |
736 | 736 | ||
737 | if (p_s_search_path->reada & PATH_READA_BACK) | 737 | if (search_path->reada & PATH_READA_BACK) |
738 | limit = 0; | 738 | limit = 0; |
739 | while (reada_count < SEARCH_BY_KEY_READA) { | 739 | while (reada_count < SEARCH_BY_KEY_READA) { |
740 | if (pos == limit) | 740 | if (pos == limit) |
741 | break; | 741 | break; |
742 | reada_blocks[reada_count++] = | 742 | reada_blocks[reada_count++] = |
743 | B_N_CHILD_NUM(bh, pos); | 743 | B_N_CHILD_NUM(bh, pos); |
744 | if (p_s_search_path->reada & PATH_READA_BACK) | 744 | if (search_path->reada & PATH_READA_BACK) |
745 | pos--; | 745 | pos--; |
746 | else | 746 | else |
747 | pos++; | 747 | pos++; |
@@ -751,7 +751,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
751 | */ | 751 | */ |
752 | le_key = B_N_PDELIM_KEY(bh, pos); | 752 | le_key = B_N_PDELIM_KEY(bh, pos); |
753 | if (le32_to_cpu(le_key->k_objectid) != | 753 | if (le32_to_cpu(le_key->k_objectid) != |
754 | p_s_key->on_disk_key.k_objectid) { | 754 | key->on_disk_key.k_objectid) { |
755 | break; | 755 | break; |
756 | } | 756 | } |
757 | } | 757 | } |
@@ -760,11 +760,11 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
760 | } | 760 | } |
761 | 761 | ||
762 | /* Form the path to an item and position in this item which contains | 762 | /* Form the path to an item and position in this item which contains |
763 | file byte defined by p_s_key. If there is no such item | 763 | file byte defined by key. If there is no such item |
764 | corresponding to the key, we point the path to the item with | 764 | corresponding to the key, we point the path to the item with |
765 | maximal key less than p_s_key, and *p_n_pos_in_item is set to one | 765 | maximal key less than key, and *pos_in_item is set to one |
766 | past the last entry/byte in the item. If searching for entry in a | 766 | past the last entry/byte in the item. If searching for entry in a |
767 | directory item, and it is not found, *p_n_pos_in_item is set to one | 767 | directory item, and it is not found, *pos_in_item is set to one |
768 | entry more than the entry with maximal key which is less than the | 768 | entry more than the entry with maximal key which is less than the |
769 | sought key. | 769 | sought key. |
770 | 770 | ||
@@ -777,7 +777,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key, /* Key | |||
777 | /* The function is NOT SCHEDULE-SAFE! */ | 777 | /* The function is NOT SCHEDULE-SAFE! */ |
778 | int search_for_position_by_key(struct super_block *sb, /* Pointer to the super block. */ | 778 | int search_for_position_by_key(struct super_block *sb, /* Pointer to the super block. */ |
779 | const struct cpu_key *p_cpu_key, /* Key to search (cpu variable) */ | 779 | const struct cpu_key *p_cpu_key, /* Key to search (cpu variable) */ |
780 | struct treepath *p_s_search_path /* Filled up by this function. */ | 780 | struct treepath *search_path /* Filled up by this function. */ |
781 | ) | 781 | ) |
782 | { | 782 | { |
783 | struct item_head *p_le_ih; /* pointer to on-disk structure */ | 783 | struct item_head *p_le_ih; /* pointer to on-disk structure */ |
@@ -788,34 +788,34 @@ int search_for_position_by_key(struct super_block *sb, /* Pointer to the super b | |||
788 | 788 | ||
789 | /* If searching for directory entry. */ | 789 | /* If searching for directory entry. */ |
790 | if (is_direntry_cpu_key(p_cpu_key)) | 790 | if (is_direntry_cpu_key(p_cpu_key)) |
791 | return search_by_entry_key(sb, p_cpu_key, p_s_search_path, | 791 | return search_by_entry_key(sb, p_cpu_key, search_path, |
792 | &de); | 792 | &de); |
793 | 793 | ||
794 | /* If not searching for directory entry. */ | 794 | /* If not searching for directory entry. */ |
795 | 795 | ||
796 | /* If item is found. */ | 796 | /* If item is found. */ |
797 | retval = search_item(sb, p_cpu_key, p_s_search_path); | 797 | retval = search_item(sb, p_cpu_key, search_path); |
798 | if (retval == IO_ERROR) | 798 | if (retval == IO_ERROR) |
799 | return retval; | 799 | return retval; |
800 | if (retval == ITEM_FOUND) { | 800 | if (retval == ITEM_FOUND) { |
801 | 801 | ||
802 | RFALSE(!ih_item_len | 802 | RFALSE(!ih_item_len |
803 | (B_N_PITEM_HEAD | 803 | (B_N_PITEM_HEAD |
804 | (PATH_PLAST_BUFFER(p_s_search_path), | 804 | (PATH_PLAST_BUFFER(search_path), |
805 | PATH_LAST_POSITION(p_s_search_path))), | 805 | PATH_LAST_POSITION(search_path))), |
806 | "PAP-5165: item length equals zero"); | 806 | "PAP-5165: item length equals zero"); |
807 | 807 | ||
808 | pos_in_item(p_s_search_path) = 0; | 808 | pos_in_item(search_path) = 0; |
809 | return POSITION_FOUND; | 809 | return POSITION_FOUND; |
810 | } | 810 | } |
811 | 811 | ||
812 | RFALSE(!PATH_LAST_POSITION(p_s_search_path), | 812 | RFALSE(!PATH_LAST_POSITION(search_path), |
813 | "PAP-5170: position equals zero"); | 813 | "PAP-5170: position equals zero"); |
814 | 814 | ||
815 | /* Item is not found. Set path to the previous item. */ | 815 | /* Item is not found. Set path to the previous item. */ |
816 | p_le_ih = | 816 | p_le_ih = |
817 | B_N_PITEM_HEAD(PATH_PLAST_BUFFER(p_s_search_path), | 817 | B_N_PITEM_HEAD(PATH_PLAST_BUFFER(search_path), |
818 | --PATH_LAST_POSITION(p_s_search_path)); | 818 | --PATH_LAST_POSITION(search_path)); |
819 | n_blk_size = sb->s_blocksize; | 819 | n_blk_size = sb->s_blocksize; |
820 | 820 | ||
821 | if (comp_short_keys(&(p_le_ih->ih_key), p_cpu_key)) { | 821 | if (comp_short_keys(&(p_le_ih->ih_key), p_cpu_key)) { |
@@ -829,9 +829,9 @@ int search_for_position_by_key(struct super_block *sb, /* Pointer to the super b | |||
829 | /* Needed byte is contained in the item pointed to by the path. */ | 829 | /* Needed byte is contained in the item pointed to by the path. */ |
830 | if (item_offset <= offset && | 830 | if (item_offset <= offset && |
831 | item_offset + op_bytes_number(p_le_ih, n_blk_size) > offset) { | 831 | item_offset + op_bytes_number(p_le_ih, n_blk_size) > offset) { |
832 | pos_in_item(p_s_search_path) = offset - item_offset; | 832 | pos_in_item(search_path) = offset - item_offset; |
833 | if (is_indirect_le_ih(p_le_ih)) { | 833 | if (is_indirect_le_ih(p_le_ih)) { |
834 | pos_in_item(p_s_search_path) /= n_blk_size; | 834 | pos_in_item(search_path) /= n_blk_size; |
835 | } | 835 | } |
836 | return POSITION_FOUND; | 836 | return POSITION_FOUND; |
837 | } | 837 | } |
@@ -839,18 +839,18 @@ int search_for_position_by_key(struct super_block *sb, /* Pointer to the super b | |||
839 | /* Needed byte is not contained in the item pointed to by the | 839 | /* Needed byte is not contained in the item pointed to by the |
840 | path. Set pos_in_item out of the item. */ | 840 | path. Set pos_in_item out of the item. */ |
841 | if (is_indirect_le_ih(p_le_ih)) | 841 | if (is_indirect_le_ih(p_le_ih)) |
842 | pos_in_item(p_s_search_path) = | 842 | pos_in_item(search_path) = |
843 | ih_item_len(p_le_ih) / UNFM_P_SIZE; | 843 | ih_item_len(p_le_ih) / UNFM_P_SIZE; |
844 | else | 844 | else |
845 | pos_in_item(p_s_search_path) = ih_item_len(p_le_ih); | 845 | pos_in_item(search_path) = ih_item_len(p_le_ih); |
846 | 846 | ||
847 | return POSITION_NOT_FOUND; | 847 | return POSITION_NOT_FOUND; |
848 | } | 848 | } |
849 | 849 | ||
850 | /* Compare given item and item pointed to by the path. */ | 850 | /* Compare given item and item pointed to by the path. */ |
851 | int comp_items(const struct item_head *stored_ih, const struct treepath *p_s_path) | 851 | int comp_items(const struct item_head *stored_ih, const struct treepath *path) |
852 | { | 852 | { |
853 | struct buffer_head *bh = PATH_PLAST_BUFFER(p_s_path); | 853 | struct buffer_head *bh = PATH_PLAST_BUFFER(path); |
854 | struct item_head *ih; | 854 | struct item_head *ih; |
855 | 855 | ||
856 | /* Last buffer at the path is not in the tree. */ | 856 | /* Last buffer at the path is not in the tree. */ |
@@ -858,11 +858,11 @@ int comp_items(const struct item_head *stored_ih, const struct treepath *p_s_pat | |||
858 | return 1; | 858 | return 1; |
859 | 859 | ||
860 | /* Last path position is invalid. */ | 860 | /* Last path position is invalid. */ |
861 | if (PATH_LAST_POSITION(p_s_path) >= B_NR_ITEMS(bh)) | 861 | if (PATH_LAST_POSITION(path) >= B_NR_ITEMS(bh)) |
862 | return 1; | 862 | return 1; |
863 | 863 | ||
864 | /* we need only to know, whether it is the same item */ | 864 | /* we need only to know, whether it is the same item */ |
865 | ih = get_ih(p_s_path); | 865 | ih = get_ih(path); |
866 | return memcmp(stored_ih, ih, IH_SIZE); | 866 | return memcmp(stored_ih, ih, IH_SIZE); |
867 | } | 867 | } |
868 | 868 | ||
@@ -951,14 +951,14 @@ static inline int prepare_for_direntry_item(struct treepath *path, | |||
951 | In case of file truncate calculate whether this item must be deleted/truncated or last | 951 | In case of file truncate calculate whether this item must be deleted/truncated or last |
952 | unformatted node of this item will be converted to a direct item. | 952 | unformatted node of this item will be converted to a direct item. |
953 | This function returns a determination of what balance mode the calling function should employ. */ | 953 | This function returns a determination of what balance mode the calling function should employ. */ |
954 | static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, struct inode *inode, struct treepath *p_s_path, const struct cpu_key *p_s_item_key, int *p_n_removed, /* Number of unformatted nodes which were removed | 954 | static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, struct inode *inode, struct treepath *path, const struct cpu_key *item_key, int *removed, /* Number of unformatted nodes which were removed |
955 | from end of the file. */ | 955 | from end of the file. */ |
956 | int *p_n_cut_size, unsigned long long n_new_file_length /* MAX_KEY_OFFSET in case of delete. */ | 956 | int *cut_size, unsigned long long n_new_file_length /* MAX_KEY_OFFSET in case of delete. */ |
957 | ) | 957 | ) |
958 | { | 958 | { |
959 | struct super_block *sb = inode->i_sb; | 959 | struct super_block *sb = inode->i_sb; |
960 | struct item_head *p_le_ih = PATH_PITEM_HEAD(p_s_path); | 960 | struct item_head *p_le_ih = PATH_PITEM_HEAD(path); |
961 | struct buffer_head *bh = PATH_PLAST_BUFFER(p_s_path); | 961 | struct buffer_head *bh = PATH_PLAST_BUFFER(path); |
962 | 962 | ||
963 | BUG_ON(!th->t_trans_id); | 963 | BUG_ON(!th->t_trans_id); |
964 | 964 | ||
@@ -968,20 +968,20 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st | |||
968 | RFALSE(n_new_file_length != max_reiserfs_offset(inode), | 968 | RFALSE(n_new_file_length != max_reiserfs_offset(inode), |
969 | "PAP-5210: mode must be M_DELETE"); | 969 | "PAP-5210: mode must be M_DELETE"); |
970 | 970 | ||
971 | *p_n_cut_size = -(IH_SIZE + ih_item_len(p_le_ih)); | 971 | *cut_size = -(IH_SIZE + ih_item_len(p_le_ih)); |
972 | return M_DELETE; | 972 | return M_DELETE; |
973 | } | 973 | } |
974 | 974 | ||
975 | /* Directory item. */ | 975 | /* Directory item. */ |
976 | if (is_direntry_le_ih(p_le_ih)) | 976 | if (is_direntry_le_ih(p_le_ih)) |
977 | return prepare_for_direntry_item(p_s_path, p_le_ih, inode, | 977 | return prepare_for_direntry_item(path, p_le_ih, inode, |
978 | n_new_file_length, | 978 | n_new_file_length, |
979 | p_n_cut_size); | 979 | cut_size); |
980 | 980 | ||
981 | /* Direct item. */ | 981 | /* Direct item. */ |
982 | if (is_direct_le_ih(p_le_ih)) | 982 | if (is_direct_le_ih(p_le_ih)) |
983 | return prepare_for_direct_item(p_s_path, p_le_ih, inode, | 983 | return prepare_for_direct_item(path, p_le_ih, inode, |
984 | n_new_file_length, p_n_cut_size); | 984 | n_new_file_length, cut_size); |
985 | 985 | ||
986 | /* Case of an indirect item. */ | 986 | /* Case of an indirect item. */ |
987 | { | 987 | { |
@@ -1001,9 +1001,9 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st | |||
1001 | 1001 | ||
1002 | do { | 1002 | do { |
1003 | need_re_search = 0; | 1003 | need_re_search = 0; |
1004 | *p_n_cut_size = 0; | 1004 | *cut_size = 0; |
1005 | bh = PATH_PLAST_BUFFER(p_s_path); | 1005 | bh = PATH_PLAST_BUFFER(path); |
1006 | copy_item_head(&s_ih, PATH_PITEM_HEAD(p_s_path)); | 1006 | copy_item_head(&s_ih, PATH_PITEM_HEAD(path)); |
1007 | pos = I_UNFM_NUM(&s_ih); | 1007 | pos = I_UNFM_NUM(&s_ih); |
1008 | 1008 | ||
1009 | while (le_ih_k_offset (&s_ih) + (pos - 1) * blk_size > n_new_file_length) { | 1009 | while (le_ih_k_offset (&s_ih) + (pos - 1) * blk_size > n_new_file_length) { |
@@ -1013,10 +1013,9 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st | |||
1013 | /* Each unformatted block deletion may involve one additional | 1013 | /* Each unformatted block deletion may involve one additional |
1014 | * bitmap block into the transaction, thereby the initial | 1014 | * bitmap block into the transaction, thereby the initial |
1015 | * journal space reservation might not be enough. */ | 1015 | * journal space reservation might not be enough. */ |
1016 | if (!delete && (*p_n_cut_size) != 0 && | 1016 | if (!delete && (*cut_size) != 0 && |
1017 | reiserfs_transaction_free_space(th) < JOURNAL_FOR_FREE_BLOCK_AND_UPDATE_SD) { | 1017 | reiserfs_transaction_free_space(th) < JOURNAL_FOR_FREE_BLOCK_AND_UPDATE_SD) |
1018 | break; | 1018 | break; |
1019 | } | ||
1020 | 1019 | ||
1021 | unfm = (__le32 *)B_I_PITEM(bh, &s_ih) + pos - 1; | 1020 | unfm = (__le32 *)B_I_PITEM(bh, &s_ih) + pos - 1; |
1022 | block = get_block_num(unfm, 0); | 1021 | block = get_block_num(unfm, 0); |
@@ -1030,17 +1029,17 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st | |||
1030 | 1029 | ||
1031 | cond_resched(); | 1030 | cond_resched(); |
1032 | 1031 | ||
1033 | if (item_moved (&s_ih, p_s_path)) { | 1032 | if (item_moved (&s_ih, path)) { |
1034 | need_re_search = 1; | 1033 | need_re_search = 1; |
1035 | break; | 1034 | break; |
1036 | } | 1035 | } |
1037 | 1036 | ||
1038 | pos --; | 1037 | pos --; |
1039 | (*p_n_removed) ++; | 1038 | (*removed)++; |
1040 | (*p_n_cut_size) -= UNFM_P_SIZE; | 1039 | (*cut_size) -= UNFM_P_SIZE; |
1041 | 1040 | ||
1042 | if (pos == 0) { | 1041 | if (pos == 0) { |
1043 | (*p_n_cut_size) -= IH_SIZE; | 1042 | (*cut_size) -= IH_SIZE; |
1044 | result = M_DELETE; | 1043 | result = M_DELETE; |
1045 | break; | 1044 | break; |
1046 | } | 1045 | } |
@@ -1050,10 +1049,10 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st | |||
1050 | ** buffer */ | 1049 | ** buffer */ |
1051 | reiserfs_restore_prepared_buffer(sb, bh); | 1050 | reiserfs_restore_prepared_buffer(sb, bh); |
1052 | } while (need_re_search && | 1051 | } while (need_re_search && |
1053 | search_for_position_by_key(sb, p_s_item_key, p_s_path) == POSITION_FOUND); | 1052 | search_for_position_by_key(sb, item_key, path) == POSITION_FOUND); |
1054 | pos_in_item(p_s_path) = pos * UNFM_P_SIZE; | 1053 | pos_in_item(path) = pos * UNFM_P_SIZE; |
1055 | 1054 | ||
1056 | if (*p_n_cut_size == 0) { | 1055 | if (*cut_size == 0) { |
1057 | /* Nothing were cut. maybe convert last unformatted node to the | 1056 | /* Nothing were cut. maybe convert last unformatted node to the |
1058 | * direct item? */ | 1057 | * direct item? */ |
1059 | result = M_CONVERT; | 1058 | result = M_CONVERT; |
@@ -1091,7 +1090,7 @@ static int calc_deleted_bytes_number(struct tree_balance *tb, char c_mode) | |||
1091 | static void init_tb_struct(struct reiserfs_transaction_handle *th, | 1090 | static void init_tb_struct(struct reiserfs_transaction_handle *th, |
1092 | struct tree_balance *tb, | 1091 | struct tree_balance *tb, |
1093 | struct super_block *sb, | 1092 | struct super_block *sb, |
1094 | struct treepath *p_s_path, int n_size) | 1093 | struct treepath *path, int n_size) |
1095 | { | 1094 | { |
1096 | 1095 | ||
1097 | BUG_ON(!th->t_trans_id); | 1096 | BUG_ON(!th->t_trans_id); |
@@ -1099,9 +1098,9 @@ static void init_tb_struct(struct reiserfs_transaction_handle *th, | |||
1099 | memset(tb, '\0', sizeof(struct tree_balance)); | 1098 | memset(tb, '\0', sizeof(struct tree_balance)); |
1100 | tb->transaction_handle = th; | 1099 | tb->transaction_handle = th; |
1101 | tb->tb_sb = sb; | 1100 | tb->tb_sb = sb; |
1102 | tb->tb_path = p_s_path; | 1101 | tb->tb_path = path; |
1103 | PATH_OFFSET_PBUFFER(p_s_path, ILLEGAL_PATH_ELEMENT_OFFSET) = NULL; | 1102 | PATH_OFFSET_PBUFFER(path, ILLEGAL_PATH_ELEMENT_OFFSET) = NULL; |
1104 | PATH_OFFSET_POSITION(p_s_path, ILLEGAL_PATH_ELEMENT_OFFSET) = 0; | 1103 | PATH_OFFSET_POSITION(path, ILLEGAL_PATH_ELEMENT_OFFSET) = 0; |
1105 | tb->insert_size[0] = n_size; | 1104 | tb->insert_size[0] = n_size; |
1106 | } | 1105 | } |
1107 | 1106 | ||
@@ -1141,13 +1140,17 @@ char head2type(struct item_head *ih) | |||
1141 | } | 1140 | } |
1142 | #endif | 1141 | #endif |
1143 | 1142 | ||
1144 | /* Delete object item. */ | 1143 | /* Delete object item. |
1145 | int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath *p_s_path, /* Path to the deleted item. */ | 1144 | * th - active transaction handle |
1146 | const struct cpu_key *p_s_item_key, /* Key to search for the deleted item. */ | 1145 | * path - path to the deleted item |
1147 | struct inode *inode, /* inode is here just to update | 1146 | * item_key - key to search for the deleted item |
1148 | * i_blocks and quotas */ | 1147 | * indode - used for updating i_blocks and quotas |
1149 | struct buffer_head *p_s_un_bh) | 1148 | * un_bh - NULL or unformatted node pointer |
1150 | { /* NULL or unformatted node pointer. */ | 1149 | */ |
1150 | int reiserfs_delete_item(struct reiserfs_transaction_handle *th, | ||
1151 | struct treepath *path, const struct cpu_key *item_key, | ||
1152 | struct inode *inode, struct buffer_head *un_bh) | ||
1153 | { | ||
1151 | struct super_block *sb = inode->i_sb; | 1154 | struct super_block *sb = inode->i_sb; |
1152 | struct tree_balance s_del_balance; | 1155 | struct tree_balance s_del_balance; |
1153 | struct item_head s_ih; | 1156 | struct item_head s_ih; |
@@ -1162,7 +1165,7 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath | |||
1162 | 1165 | ||
1163 | BUG_ON(!th->t_trans_id); | 1166 | BUG_ON(!th->t_trans_id); |
1164 | 1167 | ||
1165 | init_tb_struct(th, &s_del_balance, sb, p_s_path, | 1168 | init_tb_struct(th, &s_del_balance, sb, path, |
1166 | 0 /*size is unknown */ ); | 1169 | 0 /*size is unknown */ ); |
1167 | 1170 | ||
1168 | while (1) { | 1171 | while (1) { |
@@ -1172,14 +1175,14 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath | |||
1172 | n_iter++; | 1175 | n_iter++; |
1173 | c_mode = | 1176 | c_mode = |
1174 | #endif | 1177 | #endif |
1175 | prepare_for_delete_or_cut(th, inode, p_s_path, | 1178 | prepare_for_delete_or_cut(th, inode, path, |
1176 | p_s_item_key, &n_removed, | 1179 | item_key, &n_removed, |
1177 | &n_del_size, | 1180 | &n_del_size, |
1178 | max_reiserfs_offset(inode)); | 1181 | max_reiserfs_offset(inode)); |
1179 | 1182 | ||
1180 | RFALSE(c_mode != M_DELETE, "PAP-5320: mode must be M_DELETE"); | 1183 | RFALSE(c_mode != M_DELETE, "PAP-5320: mode must be M_DELETE"); |
1181 | 1184 | ||
1182 | copy_item_head(&s_ih, PATH_PITEM_HEAD(p_s_path)); | 1185 | copy_item_head(&s_ih, PATH_PITEM_HEAD(path)); |
1183 | s_del_balance.insert_size[0] = n_del_size; | 1186 | s_del_balance.insert_size[0] = n_del_size; |
1184 | 1187 | ||
1185 | n_ret_value = fix_nodes(M_DELETE, &s_del_balance, NULL, NULL); | 1188 | n_ret_value = fix_nodes(M_DELETE, &s_del_balance, NULL, NULL); |
@@ -1190,13 +1193,13 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath | |||
1190 | 1193 | ||
1191 | // file system changed, repeat search | 1194 | // file system changed, repeat search |
1192 | n_ret_value = | 1195 | n_ret_value = |
1193 | search_for_position_by_key(sb, p_s_item_key, p_s_path); | 1196 | search_for_position_by_key(sb, item_key, path); |
1194 | if (n_ret_value == IO_ERROR) | 1197 | if (n_ret_value == IO_ERROR) |
1195 | break; | 1198 | break; |
1196 | if (n_ret_value == FILE_NOT_FOUND) { | 1199 | if (n_ret_value == FILE_NOT_FOUND) { |
1197 | reiserfs_warning(sb, "vs-5340", | 1200 | reiserfs_warning(sb, "vs-5340", |
1198 | "no items of the file %K found", | 1201 | "no items of the file %K found", |
1199 | p_s_item_key); | 1202 | item_key); |
1200 | break; | 1203 | break; |
1201 | } | 1204 | } |
1202 | } /* while (1) */ | 1205 | } /* while (1) */ |
@@ -1207,7 +1210,7 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath | |||
1207 | } | 1210 | } |
1208 | // reiserfs_delete_item returns item length when success | 1211 | // reiserfs_delete_item returns item length when success |
1209 | n_ret_value = calc_deleted_bytes_number(&s_del_balance, M_DELETE); | 1212 | n_ret_value = calc_deleted_bytes_number(&s_del_balance, M_DELETE); |
1210 | q_ih = get_ih(p_s_path); | 1213 | q_ih = get_ih(path); |
1211 | quota_cut_bytes = ih_item_len(q_ih); | 1214 | quota_cut_bytes = ih_item_len(q_ih); |
1212 | 1215 | ||
1213 | /* hack so the quota code doesn't have to guess if the file | 1216 | /* hack so the quota code doesn't have to guess if the file |
@@ -1224,7 +1227,7 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath | |||
1224 | } | 1227 | } |
1225 | } | 1228 | } |
1226 | 1229 | ||
1227 | if (p_s_un_bh) { | 1230 | if (un_bh) { |
1228 | int off; | 1231 | int off; |
1229 | char *data; | 1232 | char *data; |
1230 | 1233 | ||
@@ -1242,16 +1245,16 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath | |||
1242 | ** The unformatted node must be dirtied later on. We can't be | 1245 | ** The unformatted node must be dirtied later on. We can't be |
1243 | ** sure here if the entire tail has been deleted yet. | 1246 | ** sure here if the entire tail has been deleted yet. |
1244 | ** | 1247 | ** |
1245 | ** p_s_un_bh is from the page cache (all unformatted nodes are | 1248 | ** un_bh is from the page cache (all unformatted nodes are |
1246 | ** from the page cache) and might be a highmem page. So, we | 1249 | ** from the page cache) and might be a highmem page. So, we |
1247 | ** can't use p_s_un_bh->b_data. | 1250 | ** can't use un_bh->b_data. |
1248 | ** -clm | 1251 | ** -clm |
1249 | */ | 1252 | */ |
1250 | 1253 | ||
1251 | data = kmap_atomic(p_s_un_bh->b_page, KM_USER0); | 1254 | data = kmap_atomic(un_bh->b_page, KM_USER0); |
1252 | off = ((le_ih_k_offset(&s_ih) - 1) & (PAGE_CACHE_SIZE - 1)); | 1255 | off = ((le_ih_k_offset(&s_ih) - 1) & (PAGE_CACHE_SIZE - 1)); |
1253 | memcpy(data + off, | 1256 | memcpy(data + off, |
1254 | B_I_PITEM(PATH_PLAST_BUFFER(p_s_path), &s_ih), | 1257 | B_I_PITEM(PATH_PLAST_BUFFER(path), &s_ih), |
1255 | n_ret_value); | 1258 | n_ret_value); |
1256 | kunmap_atomic(data, KM_USER0); | 1259 | kunmap_atomic(data, KM_USER0); |
1257 | } | 1260 | } |
@@ -1427,9 +1430,9 @@ static void unmap_buffers(struct page *page, loff_t pos) | |||
1427 | static int maybe_indirect_to_direct(struct reiserfs_transaction_handle *th, | 1430 | static int maybe_indirect_to_direct(struct reiserfs_transaction_handle *th, |
1428 | struct inode *inode, | 1431 | struct inode *inode, |
1429 | struct page *page, | 1432 | struct page *page, |
1430 | struct treepath *p_s_path, | 1433 | struct treepath *path, |
1431 | const struct cpu_key *p_s_item_key, | 1434 | const struct cpu_key *item_key, |
1432 | loff_t n_new_file_size, char *p_c_mode) | 1435 | loff_t n_new_file_size, char *mode) |
1433 | { | 1436 | { |
1434 | struct super_block *sb = inode->i_sb; | 1437 | struct super_block *sb = inode->i_sb; |
1435 | int n_block_size = sb->s_blocksize; | 1438 | int n_block_size = sb->s_blocksize; |
@@ -1445,17 +1448,17 @@ static int maybe_indirect_to_direct(struct reiserfs_transaction_handle *th, | |||
1445 | !tail_has_to_be_packed(inode) || | 1448 | !tail_has_to_be_packed(inode) || |
1446 | !page || (REISERFS_I(inode)->i_flags & i_nopack_mask)) { | 1449 | !page || (REISERFS_I(inode)->i_flags & i_nopack_mask)) { |
1447 | /* leave tail in an unformatted node */ | 1450 | /* leave tail in an unformatted node */ |
1448 | *p_c_mode = M_SKIP_BALANCING; | 1451 | *mode = M_SKIP_BALANCING; |
1449 | cut_bytes = | 1452 | cut_bytes = |
1450 | n_block_size - (n_new_file_size & (n_block_size - 1)); | 1453 | n_block_size - (n_new_file_size & (n_block_size - 1)); |
1451 | pathrelse(p_s_path); | 1454 | pathrelse(path); |
1452 | return cut_bytes; | 1455 | return cut_bytes; |
1453 | } | 1456 | } |
1454 | /* Permorm the conversion to a direct_item. */ | 1457 | /* Perform the conversion to a direct_item. */ |
1455 | /* return indirect_to_direct(inode, p_s_path, p_s_item_key, | 1458 | /* return indirect_to_direct(inode, path, item_key, |
1456 | n_new_file_size, p_c_mode); */ | 1459 | n_new_file_size, mode); */ |
1457 | return indirect2direct(th, inode, page, p_s_path, p_s_item_key, | 1460 | return indirect2direct(th, inode, page, path, item_key, |
1458 | n_new_file_size, p_c_mode); | 1461 | n_new_file_size, mode); |
1459 | } | 1462 | } |
1460 | 1463 | ||
1461 | /* we did indirect_to_direct conversion. And we have inserted direct | 1464 | /* we did indirect_to_direct conversion. And we have inserted direct |
@@ -1506,8 +1509,8 @@ static void indirect_to_direct_roll_back(struct reiserfs_transaction_handle *th, | |||
1506 | 1509 | ||
1507 | /* (Truncate or cut entry) or delete object item. Returns < 0 on failure */ | 1510 | /* (Truncate or cut entry) or delete object item. Returns < 0 on failure */ |
1508 | int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | 1511 | int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, |
1509 | struct treepath *p_s_path, | 1512 | struct treepath *path, |
1510 | struct cpu_key *p_s_item_key, | 1513 | struct cpu_key *item_key, |
1511 | struct inode *inode, | 1514 | struct inode *inode, |
1512 | struct page *page, loff_t n_new_file_size) | 1515 | struct page *page, loff_t n_new_file_size) |
1513 | { | 1516 | { |
@@ -1528,7 +1531,7 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1528 | 1531 | ||
1529 | BUG_ON(!th->t_trans_id); | 1532 | BUG_ON(!th->t_trans_id); |
1530 | 1533 | ||
1531 | init_tb_struct(th, &s_cut_balance, inode->i_sb, p_s_path, | 1534 | init_tb_struct(th, &s_cut_balance, inode->i_sb, path, |
1532 | n_cut_size); | 1535 | n_cut_size); |
1533 | 1536 | ||
1534 | /* Repeat this loop until we either cut the item without needing | 1537 | /* Repeat this loop until we either cut the item without needing |
@@ -1540,8 +1543,8 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1540 | pointers. */ | 1543 | pointers. */ |
1541 | 1544 | ||
1542 | c_mode = | 1545 | c_mode = |
1543 | prepare_for_delete_or_cut(th, inode, p_s_path, | 1546 | prepare_for_delete_or_cut(th, inode, path, |
1544 | p_s_item_key, &n_removed, | 1547 | item_key, &n_removed, |
1545 | &n_cut_size, n_new_file_size); | 1548 | &n_cut_size, n_new_file_size); |
1546 | if (c_mode == M_CONVERT) { | 1549 | if (c_mode == M_CONVERT) { |
1547 | /* convert last unformatted node to direct item or leave | 1550 | /* convert last unformatted node to direct item or leave |
@@ -1551,7 +1554,7 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1551 | 1554 | ||
1552 | n_ret_value = | 1555 | n_ret_value = |
1553 | maybe_indirect_to_direct(th, inode, page, | 1556 | maybe_indirect_to_direct(th, inode, page, |
1554 | p_s_path, p_s_item_key, | 1557 | path, item_key, |
1555 | n_new_file_size, &c_mode); | 1558 | n_new_file_size, &c_mode); |
1556 | if (c_mode == M_SKIP_BALANCING) | 1559 | if (c_mode == M_SKIP_BALANCING) |
1557 | /* tail has been left in the unformatted node */ | 1560 | /* tail has been left in the unformatted node */ |
@@ -1568,26 +1571,26 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1568 | inserting the new direct item. Now we are removing the | 1571 | inserting the new direct item. Now we are removing the |
1569 | last unformatted node pointer. Set key to search for | 1572 | last unformatted node pointer. Set key to search for |
1570 | it. */ | 1573 | it. */ |
1571 | set_cpu_key_k_type(p_s_item_key, TYPE_INDIRECT); | 1574 | set_cpu_key_k_type(item_key, TYPE_INDIRECT); |
1572 | p_s_item_key->key_length = 4; | 1575 | item_key->key_length = 4; |
1573 | n_new_file_size -= | 1576 | n_new_file_size -= |
1574 | (n_new_file_size & (sb->s_blocksize - 1)); | 1577 | (n_new_file_size & (sb->s_blocksize - 1)); |
1575 | tail_pos = n_new_file_size; | 1578 | tail_pos = n_new_file_size; |
1576 | set_cpu_key_k_offset(p_s_item_key, n_new_file_size + 1); | 1579 | set_cpu_key_k_offset(item_key, n_new_file_size + 1); |
1577 | if (search_for_position_by_key | 1580 | if (search_for_position_by_key |
1578 | (sb, p_s_item_key, | 1581 | (sb, item_key, |
1579 | p_s_path) == POSITION_NOT_FOUND) { | 1582 | path) == POSITION_NOT_FOUND) { |
1580 | print_block(PATH_PLAST_BUFFER(p_s_path), 3, | 1583 | print_block(PATH_PLAST_BUFFER(path), 3, |
1581 | PATH_LAST_POSITION(p_s_path) - 1, | 1584 | PATH_LAST_POSITION(path) - 1, |
1582 | PATH_LAST_POSITION(p_s_path) + 1); | 1585 | PATH_LAST_POSITION(path) + 1); |
1583 | reiserfs_panic(sb, "PAP-5580", "item to " | 1586 | reiserfs_panic(sb, "PAP-5580", "item to " |
1584 | "convert does not exist (%K)", | 1587 | "convert does not exist (%K)", |
1585 | p_s_item_key); | 1588 | item_key); |
1586 | } | 1589 | } |
1587 | continue; | 1590 | continue; |
1588 | } | 1591 | } |
1589 | if (n_cut_size == 0) { | 1592 | if (n_cut_size == 0) { |
1590 | pathrelse(p_s_path); | 1593 | pathrelse(path); |
1591 | return 0; | 1594 | return 0; |
1592 | } | 1595 | } |
1593 | 1596 | ||
@@ -1600,12 +1603,12 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1600 | PROC_INFO_INC(sb, cut_from_item_restarted); | 1603 | PROC_INFO_INC(sb, cut_from_item_restarted); |
1601 | 1604 | ||
1602 | n_ret_value = | 1605 | n_ret_value = |
1603 | search_for_position_by_key(sb, p_s_item_key, p_s_path); | 1606 | search_for_position_by_key(sb, item_key, path); |
1604 | if (n_ret_value == POSITION_FOUND) | 1607 | if (n_ret_value == POSITION_FOUND) |
1605 | continue; | 1608 | continue; |
1606 | 1609 | ||
1607 | reiserfs_warning(sb, "PAP-5610", "item %K not found", | 1610 | reiserfs_warning(sb, "PAP-5610", "item %K not found", |
1608 | p_s_item_key); | 1611 | item_key); |
1609 | unfix_nodes(&s_cut_balance); | 1612 | unfix_nodes(&s_cut_balance); |
1610 | return (n_ret_value == IO_ERROR) ? -EIO : -ENOENT; | 1613 | return (n_ret_value == IO_ERROR) ? -EIO : -ENOENT; |
1611 | } /* while */ | 1614 | } /* while */ |
@@ -1615,7 +1618,7 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1615 | if (n_is_inode_locked) { | 1618 | if (n_is_inode_locked) { |
1616 | // FIXME: this seems to be not needed: we are always able | 1619 | // FIXME: this seems to be not needed: we are always able |
1617 | // to cut item | 1620 | // to cut item |
1618 | indirect_to_direct_roll_back(th, inode, p_s_path); | 1621 | indirect_to_direct_roll_back(th, inode, path); |
1619 | } | 1622 | } |
1620 | if (n_ret_value == NO_DISK_SPACE) | 1623 | if (n_ret_value == NO_DISK_SPACE) |
1621 | reiserfs_warning(sb, "reiserfs-5092", | 1624 | reiserfs_warning(sb, "reiserfs-5092", |
@@ -1631,7 +1634,7 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1631 | /* Calculate number of bytes that need to be cut from the item. */ | 1634 | /* Calculate number of bytes that need to be cut from the item. */ |
1632 | quota_cut_bytes = | 1635 | quota_cut_bytes = |
1633 | (c_mode == | 1636 | (c_mode == |
1634 | M_DELETE) ? ih_item_len(get_ih(p_s_path)) : -s_cut_balance. | 1637 | M_DELETE) ? ih_item_len(get_ih(path)) : -s_cut_balance. |
1635 | insert_size[0]; | 1638 | insert_size[0]; |
1636 | if (retval2 == -1) | 1639 | if (retval2 == -1) |
1637 | n_ret_value = calc_deleted_bytes_number(&s_cut_balance, c_mode); | 1640 | n_ret_value = calc_deleted_bytes_number(&s_cut_balance, c_mode); |
@@ -1878,7 +1881,7 @@ int reiserfs_do_truncate(struct reiserfs_transaction_handle *th, | |||
1878 | #ifdef CONFIG_REISERFS_CHECK | 1881 | #ifdef CONFIG_REISERFS_CHECK |
1879 | // this makes sure, that we __append__, not overwrite or add holes | 1882 | // this makes sure, that we __append__, not overwrite or add holes |
1880 | static void check_research_for_paste(struct treepath *path, | 1883 | static void check_research_for_paste(struct treepath *path, |
1881 | const struct cpu_key *p_s_key) | 1884 | const struct cpu_key *key) |
1882 | { | 1885 | { |
1883 | struct item_head *found_ih = get_ih(path); | 1886 | struct item_head *found_ih = get_ih(path); |
1884 | 1887 | ||
@@ -1886,35 +1889,35 @@ static void check_research_for_paste(struct treepath *path, | |||
1886 | if (le_ih_k_offset(found_ih) + | 1889 | if (le_ih_k_offset(found_ih) + |
1887 | op_bytes_number(found_ih, | 1890 | op_bytes_number(found_ih, |
1888 | get_last_bh(path)->b_size) != | 1891 | get_last_bh(path)->b_size) != |
1889 | cpu_key_k_offset(p_s_key) | 1892 | cpu_key_k_offset(key) |
1890 | || op_bytes_number(found_ih, | 1893 | || op_bytes_number(found_ih, |
1891 | get_last_bh(path)->b_size) != | 1894 | get_last_bh(path)->b_size) != |
1892 | pos_in_item(path)) | 1895 | pos_in_item(path)) |
1893 | reiserfs_panic(NULL, "PAP-5720", "found direct item " | 1896 | reiserfs_panic(NULL, "PAP-5720", "found direct item " |
1894 | "%h or position (%d) does not match " | 1897 | "%h or position (%d) does not match " |
1895 | "to key %K", found_ih, | 1898 | "to key %K", found_ih, |
1896 | pos_in_item(path), p_s_key); | 1899 | pos_in_item(path), key); |
1897 | } | 1900 | } |
1898 | if (is_indirect_le_ih(found_ih)) { | 1901 | if (is_indirect_le_ih(found_ih)) { |
1899 | if (le_ih_k_offset(found_ih) + | 1902 | if (le_ih_k_offset(found_ih) + |
1900 | op_bytes_number(found_ih, | 1903 | op_bytes_number(found_ih, |
1901 | get_last_bh(path)->b_size) != | 1904 | get_last_bh(path)->b_size) != |
1902 | cpu_key_k_offset(p_s_key) | 1905 | cpu_key_k_offset(key) |
1903 | || I_UNFM_NUM(found_ih) != pos_in_item(path) | 1906 | || I_UNFM_NUM(found_ih) != pos_in_item(path) |
1904 | || get_ih_free_space(found_ih) != 0) | 1907 | || get_ih_free_space(found_ih) != 0) |
1905 | reiserfs_panic(NULL, "PAP-5730", "found indirect " | 1908 | reiserfs_panic(NULL, "PAP-5730", "found indirect " |
1906 | "item (%h) or position (%d) does not " | 1909 | "item (%h) or position (%d) does not " |
1907 | "match to key (%K)", | 1910 | "match to key (%K)", |
1908 | found_ih, pos_in_item(path), p_s_key); | 1911 | found_ih, pos_in_item(path), key); |
1909 | } | 1912 | } |
1910 | } | 1913 | } |
1911 | #endif /* config reiserfs check */ | 1914 | #endif /* config reiserfs check */ |
1912 | 1915 | ||
1913 | /* Paste bytes to the existing item. Returns bytes number pasted into the item. */ | 1916 | /* Paste bytes to the existing item. Returns bytes number pasted into the item. */ |
1914 | int reiserfs_paste_into_item(struct reiserfs_transaction_handle *th, struct treepath *p_s_search_path, /* Path to the pasted item. */ | 1917 | int reiserfs_paste_into_item(struct reiserfs_transaction_handle *th, struct treepath *search_path, /* Path to the pasted item. */ |
1915 | const struct cpu_key *p_s_key, /* Key to search for the needed item. */ | 1918 | const struct cpu_key *key, /* Key to search for the needed item. */ |
1916 | struct inode *inode, /* Inode item belongs to */ | 1919 | struct inode *inode, /* Inode item belongs to */ |
1917 | const char *p_c_body, /* Pointer to the bytes to paste. */ | 1920 | const char *body, /* Pointer to the bytes to paste. */ |
1918 | int n_pasted_size) | 1921 | int n_pasted_size) |
1919 | { /* Size of pasted bytes. */ | 1922 | { /* Size of pasted bytes. */ |
1920 | struct tree_balance s_paste_balance; | 1923 | struct tree_balance s_paste_balance; |
@@ -1929,17 +1932,17 @@ int reiserfs_paste_into_item(struct reiserfs_transaction_handle *th, struct tree | |||
1929 | reiserfs_debug(inode->i_sb, REISERFS_DEBUG_CODE, | 1932 | reiserfs_debug(inode->i_sb, REISERFS_DEBUG_CODE, |
1930 | "reiserquota paste_into_item(): allocating %u id=%u type=%c", | 1933 | "reiserquota paste_into_item(): allocating %u id=%u type=%c", |
1931 | n_pasted_size, inode->i_uid, | 1934 | n_pasted_size, inode->i_uid, |
1932 | key2type(&(p_s_key->on_disk_key))); | 1935 | key2type(&(key->on_disk_key))); |
1933 | #endif | 1936 | #endif |
1934 | 1937 | ||
1935 | if (DQUOT_ALLOC_SPACE_NODIRTY(inode, n_pasted_size)) { | 1938 | if (DQUOT_ALLOC_SPACE_NODIRTY(inode, n_pasted_size)) { |
1936 | pathrelse(p_s_search_path); | 1939 | pathrelse(search_path); |
1937 | return -EDQUOT; | 1940 | return -EDQUOT; |
1938 | } | 1941 | } |
1939 | init_tb_struct(th, &s_paste_balance, th->t_super, p_s_search_path, | 1942 | init_tb_struct(th, &s_paste_balance, th->t_super, search_path, |
1940 | n_pasted_size); | 1943 | n_pasted_size); |
1941 | #ifdef DISPLACE_NEW_PACKING_LOCALITIES | 1944 | #ifdef DISPLACE_NEW_PACKING_LOCALITIES |
1942 | s_paste_balance.key = p_s_key->on_disk_key; | 1945 | s_paste_balance.key = key->on_disk_key; |
1943 | #endif | 1946 | #endif |
1944 | 1947 | ||
1945 | /* DQUOT_* can schedule, must check before the fix_nodes */ | 1948 | /* DQUOT_* can schedule, must check before the fix_nodes */ |
@@ -1949,13 +1952,13 @@ int reiserfs_paste_into_item(struct reiserfs_transaction_handle *th, struct tree | |||
1949 | 1952 | ||
1950 | while ((retval = | 1953 | while ((retval = |
1951 | fix_nodes(M_PASTE, &s_paste_balance, NULL, | 1954 | fix_nodes(M_PASTE, &s_paste_balance, NULL, |
1952 | p_c_body)) == REPEAT_SEARCH) { | 1955 | body)) == REPEAT_SEARCH) { |
1953 | search_again: | 1956 | search_again: |
1954 | /* file system changed while we were in the fix_nodes */ | 1957 | /* file system changed while we were in the fix_nodes */ |
1955 | PROC_INFO_INC(th->t_super, paste_into_item_restarted); | 1958 | PROC_INFO_INC(th->t_super, paste_into_item_restarted); |
1956 | retval = | 1959 | retval = |
1957 | search_for_position_by_key(th->t_super, p_s_key, | 1960 | search_for_position_by_key(th->t_super, key, |
1958 | p_s_search_path); | 1961 | search_path); |
1959 | if (retval == IO_ERROR) { | 1962 | if (retval == IO_ERROR) { |
1960 | retval = -EIO; | 1963 | retval = -EIO; |
1961 | goto error_out; | 1964 | goto error_out; |
@@ -1963,19 +1966,19 @@ int reiserfs_paste_into_item(struct reiserfs_transaction_handle *th, struct tree | |||
1963 | if (retval == POSITION_FOUND) { | 1966 | if (retval == POSITION_FOUND) { |
1964 | reiserfs_warning(inode->i_sb, "PAP-5710", | 1967 | reiserfs_warning(inode->i_sb, "PAP-5710", |
1965 | "entry or pasted byte (%K) exists", | 1968 | "entry or pasted byte (%K) exists", |
1966 | p_s_key); | 1969 | key); |
1967 | retval = -EEXIST; | 1970 | retval = -EEXIST; |
1968 | goto error_out; | 1971 | goto error_out; |
1969 | } | 1972 | } |
1970 | #ifdef CONFIG_REISERFS_CHECK | 1973 | #ifdef CONFIG_REISERFS_CHECK |
1971 | check_research_for_paste(p_s_search_path, p_s_key); | 1974 | check_research_for_paste(search_path, key); |
1972 | #endif | 1975 | #endif |
1973 | } | 1976 | } |
1974 | 1977 | ||
1975 | /* Perform balancing after all resources are collected by fix_nodes, and | 1978 | /* Perform balancing after all resources are collected by fix_nodes, and |
1976 | accessing them will not risk triggering schedule. */ | 1979 | accessing them will not risk triggering schedule. */ |
1977 | if (retval == CARRY_ON) { | 1980 | if (retval == CARRY_ON) { |
1978 | do_balance(&s_paste_balance, NULL /*ih */ , p_c_body, M_PASTE); | 1981 | do_balance(&s_paste_balance, NULL /*ih */ , body, M_PASTE); |
1979 | return 0; | 1982 | return 0; |
1980 | } | 1983 | } |
1981 | retval = (retval == NO_DISK_SPACE) ? -ENOSPC : -EIO; | 1984 | retval = (retval == NO_DISK_SPACE) ? -ENOSPC : -EIO; |
@@ -1986,17 +1989,23 @@ int reiserfs_paste_into_item(struct reiserfs_transaction_handle *th, struct tree | |||
1986 | reiserfs_debug(inode->i_sb, REISERFS_DEBUG_CODE, | 1989 | reiserfs_debug(inode->i_sb, REISERFS_DEBUG_CODE, |
1987 | "reiserquota paste_into_item(): freeing %u id=%u type=%c", | 1990 | "reiserquota paste_into_item(): freeing %u id=%u type=%c", |
1988 | n_pasted_size, inode->i_uid, | 1991 | n_pasted_size, inode->i_uid, |
1989 | key2type(&(p_s_key->on_disk_key))); | 1992 | key2type(&(key->on_disk_key))); |
1990 | #endif | 1993 | #endif |
1991 | DQUOT_FREE_SPACE_NODIRTY(inode, n_pasted_size); | 1994 | DQUOT_FREE_SPACE_NODIRTY(inode, n_pasted_size); |
1992 | return retval; | 1995 | return retval; |
1993 | } | 1996 | } |
1994 | 1997 | ||
1995 | /* Insert new item into the buffer at the path. */ | 1998 | /* Insert new item into the buffer at the path. |
1996 | int reiserfs_insert_item(struct reiserfs_transaction_handle *th, struct treepath *p_s_path, /* Path to the inserteded item. */ | 1999 | * th - active transaction handle |
1997 | const struct cpu_key *key, struct item_head *p_s_ih, /* Pointer to the item header to insert. */ | 2000 | * path - path to the inserted item |
1998 | struct inode *inode, const char *p_c_body) | 2001 | * ih - pointer to the item header to insert |
1999 | { /* Pointer to the bytes to insert. */ | 2002 | * body - pointer to the bytes to insert |
2003 | */ | ||
2004 | int reiserfs_insert_item(struct reiserfs_transaction_handle *th, | ||
2005 | struct treepath *path, const struct cpu_key *key, | ||
2006 | struct item_head *ih, struct inode *inode, | ||
2007 | const char *body) | ||
2008 | { | ||
2000 | struct tree_balance s_ins_balance; | 2009 | struct tree_balance s_ins_balance; |
2001 | int retval; | 2010 | int retval; |
2002 | int fs_gen = 0; | 2011 | int fs_gen = 0; |
@@ -2006,28 +2015,27 @@ int reiserfs_insert_item(struct reiserfs_transaction_handle *th, struct treepath | |||
2006 | 2015 | ||
2007 | if (inode) { /* Do we count quotas for item? */ | 2016 | if (inode) { /* Do we count quotas for item? */ |
2008 | fs_gen = get_generation(inode->i_sb); | 2017 | fs_gen = get_generation(inode->i_sb); |
2009 | quota_bytes = ih_item_len(p_s_ih); | 2018 | quota_bytes = ih_item_len(ih); |
2010 | 2019 | ||
2011 | /* hack so the quota code doesn't have to guess if the file has | 2020 | /* hack so the quota code doesn't have to guess if the file has |
2012 | ** a tail, links are always tails, so there's no guessing needed | 2021 | ** a tail, links are always tails, so there's no guessing needed |
2013 | */ | 2022 | */ |
2014 | if (!S_ISLNK(inode->i_mode) && is_direct_le_ih(p_s_ih)) { | 2023 | if (!S_ISLNK(inode->i_mode) && is_direct_le_ih(ih)) |
2015 | quota_bytes = inode->i_sb->s_blocksize + UNFM_P_SIZE; | 2024 | quota_bytes = inode->i_sb->s_blocksize + UNFM_P_SIZE; |
2016 | } | ||
2017 | #ifdef REISERQUOTA_DEBUG | 2025 | #ifdef REISERQUOTA_DEBUG |
2018 | reiserfs_debug(inode->i_sb, REISERFS_DEBUG_CODE, | 2026 | reiserfs_debug(inode->i_sb, REISERFS_DEBUG_CODE, |
2019 | "reiserquota insert_item(): allocating %u id=%u type=%c", | 2027 | "reiserquota insert_item(): allocating %u id=%u type=%c", |
2020 | quota_bytes, inode->i_uid, head2type(p_s_ih)); | 2028 | quota_bytes, inode->i_uid, head2type(ih)); |
2021 | #endif | 2029 | #endif |
2022 | /* We can't dirty inode here. It would be immediately written but | 2030 | /* We can't dirty inode here. It would be immediately written but |
2023 | * appropriate stat item isn't inserted yet... */ | 2031 | * appropriate stat item isn't inserted yet... */ |
2024 | if (DQUOT_ALLOC_SPACE_NODIRTY(inode, quota_bytes)) { | 2032 | if (DQUOT_ALLOC_SPACE_NODIRTY(inode, quota_bytes)) { |
2025 | pathrelse(p_s_path); | 2033 | pathrelse(path); |
2026 | return -EDQUOT; | 2034 | return -EDQUOT; |
2027 | } | 2035 | } |
2028 | } | 2036 | } |
2029 | init_tb_struct(th, &s_ins_balance, th->t_super, p_s_path, | 2037 | init_tb_struct(th, &s_ins_balance, th->t_super, path, |
2030 | IH_SIZE + ih_item_len(p_s_ih)); | 2038 | IH_SIZE + ih_item_len(ih)); |
2031 | #ifdef DISPLACE_NEW_PACKING_LOCALITIES | 2039 | #ifdef DISPLACE_NEW_PACKING_LOCALITIES |
2032 | s_ins_balance.key = key->on_disk_key; | 2040 | s_ins_balance.key = key->on_disk_key; |
2033 | #endif | 2041 | #endif |
@@ -2037,12 +2045,12 @@ int reiserfs_insert_item(struct reiserfs_transaction_handle *th, struct treepath | |||
2037 | } | 2045 | } |
2038 | 2046 | ||
2039 | while ((retval = | 2047 | while ((retval = |
2040 | fix_nodes(M_INSERT, &s_ins_balance, p_s_ih, | 2048 | fix_nodes(M_INSERT, &s_ins_balance, ih, |
2041 | p_c_body)) == REPEAT_SEARCH) { | 2049 | body)) == REPEAT_SEARCH) { |
2042 | search_again: | 2050 | search_again: |
2043 | /* file system changed while we were in the fix_nodes */ | 2051 | /* file system changed while we were in the fix_nodes */ |
2044 | PROC_INFO_INC(th->t_super, insert_item_restarted); | 2052 | PROC_INFO_INC(th->t_super, insert_item_restarted); |
2045 | retval = search_item(th->t_super, key, p_s_path); | 2053 | retval = search_item(th->t_super, key, path); |
2046 | if (retval == IO_ERROR) { | 2054 | if (retval == IO_ERROR) { |
2047 | retval = -EIO; | 2055 | retval = -EIO; |
2048 | goto error_out; | 2056 | goto error_out; |
@@ -2058,7 +2066,7 @@ int reiserfs_insert_item(struct reiserfs_transaction_handle *th, struct treepath | |||
2058 | 2066 | ||
2059 | /* make balancing after all resources will be collected at a time */ | 2067 | /* make balancing after all resources will be collected at a time */ |
2060 | if (retval == CARRY_ON) { | 2068 | if (retval == CARRY_ON) { |
2061 | do_balance(&s_ins_balance, p_s_ih, p_c_body, M_INSERT); | 2069 | do_balance(&s_ins_balance, ih, body, M_INSERT); |
2062 | return 0; | 2070 | return 0; |
2063 | } | 2071 | } |
2064 | 2072 | ||
@@ -2069,7 +2077,7 @@ int reiserfs_insert_item(struct reiserfs_transaction_handle *th, struct treepath | |||
2069 | #ifdef REISERQUOTA_DEBUG | 2077 | #ifdef REISERQUOTA_DEBUG |
2070 | reiserfs_debug(th->t_super, REISERFS_DEBUG_CODE, | 2078 | reiserfs_debug(th->t_super, REISERFS_DEBUG_CODE, |
2071 | "reiserquota insert_item(): freeing %u id=%u type=%c", | 2079 | "reiserquota insert_item(): freeing %u id=%u type=%c", |
2072 | quota_bytes, inode->i_uid, head2type(p_s_ih)); | 2080 | quota_bytes, inode->i_uid, head2type(ih)); |
2073 | #endif | 2081 | #endif |
2074 | if (inode) | 2082 | if (inode) |
2075 | DQUOT_FREE_SPACE_NODIRTY(inode, quota_bytes); | 2083 | DQUOT_FREE_SPACE_NODIRTY(inode, quota_bytes); |
diff --git a/fs/reiserfs/tail_conversion.c b/fs/reiserfs/tail_conversion.c index 5c5ee0d0d6a8..2b90c0e5697c 100644 --- a/fs/reiserfs/tail_conversion.c +++ b/fs/reiserfs/tail_conversion.c | |||
@@ -172,10 +172,12 @@ void reiserfs_unmap_buffer(struct buffer_head *bh) | |||
172 | inode */ | 172 | inode */ |
173 | int indirect2direct(struct reiserfs_transaction_handle *th, | 173 | int indirect2direct(struct reiserfs_transaction_handle *th, |
174 | struct inode *inode, struct page *page, | 174 | struct inode *inode, struct page *page, |
175 | struct treepath *p_s_path, /* path to the indirect item. */ | 175 | struct treepath *path, /* path to the indirect item. */ |
176 | const struct cpu_key *p_s_item_key, /* Key to look for unformatted node pointer to be cut. */ | 176 | const struct cpu_key *item_key, /* Key to look for |
177 | * unformatted node | ||
178 | * pointer to be cut. */ | ||
177 | loff_t n_new_file_size, /* New file size. */ | 179 | loff_t n_new_file_size, /* New file size. */ |
178 | char *p_c_mode) | 180 | char *mode) |
179 | { | 181 | { |
180 | struct super_block *sb = inode->i_sb; | 182 | struct super_block *sb = inode->i_sb; |
181 | struct item_head s_ih; | 183 | struct item_head s_ih; |
@@ -189,10 +191,10 @@ int indirect2direct(struct reiserfs_transaction_handle *th, | |||
189 | 191 | ||
190 | REISERFS_SB(sb)->s_indirect2direct++; | 192 | REISERFS_SB(sb)->s_indirect2direct++; |
191 | 193 | ||
192 | *p_c_mode = M_SKIP_BALANCING; | 194 | *mode = M_SKIP_BALANCING; |
193 | 195 | ||
194 | /* store item head path points to. */ | 196 | /* store item head path points to. */ |
195 | copy_item_head(&s_ih, PATH_PITEM_HEAD(p_s_path)); | 197 | copy_item_head(&s_ih, PATH_PITEM_HEAD(path)); |
196 | 198 | ||
197 | tail_len = (n_new_file_size & (n_block_size - 1)); | 199 | tail_len = (n_new_file_size & (n_block_size - 1)); |
198 | if (get_inode_sd_version(inode) == STAT_DATA_V2) | 200 | if (get_inode_sd_version(inode) == STAT_DATA_V2) |
@@ -211,14 +213,14 @@ int indirect2direct(struct reiserfs_transaction_handle *th, | |||
211 | 213 | ||
212 | tail = (char *)kmap(page); /* this can schedule */ | 214 | tail = (char *)kmap(page); /* this can schedule */ |
213 | 215 | ||
214 | if (path_changed(&s_ih, p_s_path)) { | 216 | if (path_changed(&s_ih, path)) { |
215 | /* re-search indirect item */ | 217 | /* re-search indirect item */ |
216 | if (search_for_position_by_key(sb, p_s_item_key, p_s_path) | 218 | if (search_for_position_by_key(sb, item_key, path) |
217 | == POSITION_NOT_FOUND) | 219 | == POSITION_NOT_FOUND) |
218 | reiserfs_panic(sb, "PAP-5520", | 220 | reiserfs_panic(sb, "PAP-5520", |
219 | "item to be converted %K does not exist", | 221 | "item to be converted %K does not exist", |
220 | p_s_item_key); | 222 | item_key); |
221 | copy_item_head(&s_ih, PATH_PITEM_HEAD(p_s_path)); | 223 | copy_item_head(&s_ih, PATH_PITEM_HEAD(path)); |
222 | #ifdef CONFIG_REISERFS_CHECK | 224 | #ifdef CONFIG_REISERFS_CHECK |
223 | pos = le_ih_k_offset(&s_ih) - 1 + | 225 | pos = le_ih_k_offset(&s_ih) - 1 + |
224 | (ih_item_len(&s_ih) / UNFM_P_SIZE - | 226 | (ih_item_len(&s_ih) / UNFM_P_SIZE - |
@@ -240,13 +242,13 @@ int indirect2direct(struct reiserfs_transaction_handle *th, | |||
240 | */ | 242 | */ |
241 | tail = tail + (pos & (PAGE_CACHE_SIZE - 1)); | 243 | tail = tail + (pos & (PAGE_CACHE_SIZE - 1)); |
242 | 244 | ||
243 | PATH_LAST_POSITION(p_s_path)++; | 245 | PATH_LAST_POSITION(path)++; |
244 | 246 | ||
245 | key = *p_s_item_key; | 247 | key = *item_key; |
246 | set_cpu_key_k_type(&key, TYPE_DIRECT); | 248 | set_cpu_key_k_type(&key, TYPE_DIRECT); |
247 | key.key_length = 4; | 249 | key.key_length = 4; |
248 | /* Insert tail as new direct item in the tree */ | 250 | /* Insert tail as new direct item in the tree */ |
249 | if (reiserfs_insert_item(th, p_s_path, &key, &s_ih, inode, | 251 | if (reiserfs_insert_item(th, path, &key, &s_ih, inode, |
250 | tail ? tail : NULL) < 0) { | 252 | tail ? tail : NULL) < 0) { |
251 | /* No disk memory. So we can not convert last unformatted node | 253 | /* No disk memory. So we can not convert last unformatted node |
252 | to the direct item. In this case we used to adjust | 254 | to the direct item. In this case we used to adjust |
@@ -268,7 +270,7 @@ int indirect2direct(struct reiserfs_transaction_handle *th, | |||
268 | 270 | ||
269 | /* We have inserted new direct item and must remove last | 271 | /* We have inserted new direct item and must remove last |
270 | unformatted node. */ | 272 | unformatted node. */ |
271 | *p_c_mode = M_CUT; | 273 | *mode = M_CUT; |
272 | 274 | ||
273 | /* we store position of first direct item in the in-core inode */ | 275 | /* we store position of first direct item in the in-core inode */ |
274 | /* mark_file_with_tail (inode, pos1 + 1); */ | 276 | /* mark_file_with_tail (inode, pos1 + 1); */ |
diff --git a/include/linux/reiserfs_fs.h b/include/linux/reiserfs_fs.h index b72dc2095478..e711c796e9d1 100644 --- a/include/linux/reiserfs_fs.h +++ b/include/linux/reiserfs_fs.h | |||
@@ -694,9 +694,9 @@ static inline void cpu_key_k_offset_dec(struct cpu_key *key) | |||
694 | #define is_indirect_cpu_ih(ih) (is_indirect_cpu_key (&((ih)->ih_key))) | 694 | #define is_indirect_cpu_ih(ih) (is_indirect_cpu_key (&((ih)->ih_key))) |
695 | #define is_statdata_cpu_ih(ih) (is_statdata_cpu_key (&((ih)->ih_key))) | 695 | #define is_statdata_cpu_ih(ih) (is_statdata_cpu_key (&((ih)->ih_key))) |
696 | 696 | ||
697 | #define I_K_KEY_IN_ITEM(p_s_ih, p_s_key, n_blocksize) \ | 697 | #define I_K_KEY_IN_ITEM(ih, key, n_blocksize) \ |
698 | ( ! COMP_SHORT_KEYS(p_s_ih, p_s_key) && \ | 698 | (!COMP_SHORT_KEYS(ih, key) && \ |
699 | I_OFF_BYTE_IN_ITEM(p_s_ih, k_offset (p_s_key), n_blocksize) ) | 699 | I_OFF_BYTE_IN_ITEM(ih, k_offset(key), n_blocksize)) |
700 | 700 | ||
701 | /* maximal length of item */ | 701 | /* maximal length of item */ |
702 | #define MAX_ITEM_LEN(block_size) (block_size - BLKH_SIZE - IH_SIZE) | 702 | #define MAX_ITEM_LEN(block_size) (block_size - BLKH_SIZE - IH_SIZE) |
@@ -1196,33 +1196,33 @@ struct treepath { | |||
1196 | struct treepath var = {.path_length = ILLEGAL_PATH_ELEMENT_OFFSET, .reada = 0,} | 1196 | struct treepath var = {.path_length = ILLEGAL_PATH_ELEMENT_OFFSET, .reada = 0,} |
1197 | 1197 | ||
1198 | /* Get path element by path and path position. */ | 1198 | /* Get path element by path and path position. */ |
1199 | #define PATH_OFFSET_PELEMENT(p_s_path,n_offset) ((p_s_path)->path_elements +(n_offset)) | 1199 | #define PATH_OFFSET_PELEMENT(path, n_offset) ((path)->path_elements + (n_offset)) |
1200 | 1200 | ||
1201 | /* Get buffer header at the path by path and path position. */ | 1201 | /* Get buffer header at the path by path and path position. */ |
1202 | #define PATH_OFFSET_PBUFFER(p_s_path,n_offset) (PATH_OFFSET_PELEMENT(p_s_path,n_offset)->pe_buffer) | 1202 | #define PATH_OFFSET_PBUFFER(path, n_offset) (PATH_OFFSET_PELEMENT(path, n_offset)->pe_buffer) |
1203 | 1203 | ||
1204 | /* Get position in the element at the path by path and path position. */ | 1204 | /* Get position in the element at the path by path and path position. */ |
1205 | #define PATH_OFFSET_POSITION(p_s_path,n_offset) (PATH_OFFSET_PELEMENT(p_s_path,n_offset)->pe_position) | 1205 | #define PATH_OFFSET_POSITION(path, n_offset) (PATH_OFFSET_PELEMENT(path, n_offset)->pe_position) |
1206 | 1206 | ||
1207 | #define PATH_PLAST_BUFFER(p_s_path) (PATH_OFFSET_PBUFFER((p_s_path), (p_s_path)->path_length)) | 1207 | #define PATH_PLAST_BUFFER(path) (PATH_OFFSET_PBUFFER((path), (path)->path_length)) |
1208 | /* you know, to the person who didn't | 1208 | /* you know, to the person who didn't |
1209 | write this the macro name does not | 1209 | write this the macro name does not |
1210 | at first suggest what it does. | 1210 | at first suggest what it does. |
1211 | Maybe POSITION_FROM_PATH_END? Or | 1211 | Maybe POSITION_FROM_PATH_END? Or |
1212 | maybe we should just focus on | 1212 | maybe we should just focus on |
1213 | dumping paths... -Hans */ | 1213 | dumping paths... -Hans */ |
1214 | #define PATH_LAST_POSITION(p_s_path) (PATH_OFFSET_POSITION((p_s_path), (p_s_path)->path_length)) | 1214 | #define PATH_LAST_POSITION(path) (PATH_OFFSET_POSITION((path), (path)->path_length)) |
1215 | 1215 | ||
1216 | #define PATH_PITEM_HEAD(p_s_path) B_N_PITEM_HEAD(PATH_PLAST_BUFFER(p_s_path),PATH_LAST_POSITION(p_s_path)) | 1216 | #define PATH_PITEM_HEAD(path) B_N_PITEM_HEAD(PATH_PLAST_BUFFER(path), PATH_LAST_POSITION(path)) |
1217 | 1217 | ||
1218 | /* in do_balance leaf has h == 0 in contrast with path structure, | 1218 | /* in do_balance leaf has h == 0 in contrast with path structure, |
1219 | where root has level == 0. That is why we need these defines */ | 1219 | where root has level == 0. That is why we need these defines */ |
1220 | #define PATH_H_PBUFFER(p_s_path, h) PATH_OFFSET_PBUFFER (p_s_path, p_s_path->path_length - (h)) /* tb->S[h] */ | 1220 | #define PATH_H_PBUFFER(path, h) PATH_OFFSET_PBUFFER (path, path->path_length - (h)) /* tb->S[h] */ |
1221 | #define PATH_H_PPARENT(path, h) PATH_H_PBUFFER (path, (h) + 1) /* tb->F[h] or tb->S[0]->b_parent */ | 1221 | #define PATH_H_PPARENT(path, h) PATH_H_PBUFFER (path, (h) + 1) /* tb->F[h] or tb->S[0]->b_parent */ |
1222 | #define PATH_H_POSITION(path, h) PATH_OFFSET_POSITION (path, path->path_length - (h)) | 1222 | #define PATH_H_POSITION(path, h) PATH_OFFSET_POSITION (path, path->path_length - (h)) |
1223 | #define PATH_H_B_ITEM_ORDER(path, h) PATH_H_POSITION(path, h + 1) /* tb->S[h]->b_item_order */ | 1223 | #define PATH_H_B_ITEM_ORDER(path, h) PATH_H_POSITION(path, h + 1) /* tb->S[h]->b_item_order */ |
1224 | 1224 | ||
1225 | #define PATH_H_PATH_OFFSET(p_s_path, n_h) ((p_s_path)->path_length - (n_h)) | 1225 | #define PATH_H_PATH_OFFSET(path, n_h) ((path)->path_length - (n_h)) |
1226 | 1226 | ||
1227 | #define get_last_bh(path) PATH_PLAST_BUFFER(path) | 1227 | #define get_last_bh(path) PATH_PLAST_BUFFER(path) |
1228 | #define get_ih(path) PATH_PITEM_HEAD(path) | 1228 | #define get_ih(path) PATH_PITEM_HEAD(path) |
@@ -1512,7 +1512,7 @@ extern struct item_operations *item_ops[TYPE_ANY + 1]; | |||
1512 | #define COMP_SHORT_KEYS comp_short_keys | 1512 | #define COMP_SHORT_KEYS comp_short_keys |
1513 | 1513 | ||
1514 | /* number of blocks pointed to by the indirect item */ | 1514 | /* number of blocks pointed to by the indirect item */ |
1515 | #define I_UNFM_NUM(p_s_ih) ( ih_item_len(p_s_ih) / UNFM_P_SIZE ) | 1515 | #define I_UNFM_NUM(ih) (ih_item_len(ih) / UNFM_P_SIZE) |
1516 | 1516 | ||
1517 | /* the used space within the unformatted node corresponding to pos within the item pointed to by ih */ | 1517 | /* the used space within the unformatted node corresponding to pos within the item pointed to by ih */ |
1518 | #define I_POS_UNFM_SIZE(ih,pos,size) (((pos) == I_UNFM_NUM(ih) - 1 ) ? (size) - ih_free_space(ih) : (size)) | 1518 | #define I_POS_UNFM_SIZE(ih,pos,size) (((pos) == I_UNFM_NUM(ih) - 1 ) ? (size) - ih_free_space(ih) : (size)) |
@@ -1793,8 +1793,8 @@ int reiserfs_convert_objectid_map_v1(struct super_block *); | |||
1793 | 1793 | ||
1794 | /* stree.c */ | 1794 | /* stree.c */ |
1795 | int B_IS_IN_TREE(const struct buffer_head *); | 1795 | int B_IS_IN_TREE(const struct buffer_head *); |
1796 | extern void copy_item_head(struct item_head *p_v_to, | 1796 | extern void copy_item_head(struct item_head *to, |
1797 | const struct item_head *p_v_from); | 1797 | const struct item_head *from); |
1798 | 1798 | ||
1799 | // first key is in cpu form, second - le | 1799 | // first key is in cpu form, second - le |
1800 | extern int comp_short_keys(const struct reiserfs_key *le_key, | 1800 | extern int comp_short_keys(const struct reiserfs_key *le_key, |
@@ -1829,20 +1829,20 @@ static inline void copy_key(struct reiserfs_key *to, | |||
1829 | memcpy(to, from, KEY_SIZE); | 1829 | memcpy(to, from, KEY_SIZE); |
1830 | } | 1830 | } |
1831 | 1831 | ||
1832 | int comp_items(const struct item_head *stored_ih, const struct treepath *p_s_path); | 1832 | int comp_items(const struct item_head *stored_ih, const struct treepath *path); |
1833 | const struct reiserfs_key *get_rkey(const struct treepath *p_s_chk_path, | 1833 | const struct reiserfs_key *get_rkey(const struct treepath *chk_path, |
1834 | const struct super_block *sb); | 1834 | const struct super_block *sb); |
1835 | int search_by_key(struct super_block *, const struct cpu_key *, | 1835 | int search_by_key(struct super_block *, const struct cpu_key *, |
1836 | struct treepath *, int); | 1836 | struct treepath *, int); |
1837 | #define search_item(s,key,path) search_by_key (s, key, path, DISK_LEAF_NODE_LEVEL) | 1837 | #define search_item(s,key,path) search_by_key (s, key, path, DISK_LEAF_NODE_LEVEL) |
1838 | int search_for_position_by_key(struct super_block *sb, | 1838 | int search_for_position_by_key(struct super_block *sb, |
1839 | const struct cpu_key *p_s_cpu_key, | 1839 | const struct cpu_key *cpu_key, |
1840 | struct treepath *p_s_search_path); | 1840 | struct treepath *search_path); |
1841 | extern void decrement_bcount(struct buffer_head *bh); | 1841 | extern void decrement_bcount(struct buffer_head *bh); |
1842 | void decrement_counters_in_path(struct treepath *p_s_search_path); | 1842 | void decrement_counters_in_path(struct treepath *search_path); |
1843 | void pathrelse(struct treepath *p_s_search_path); | 1843 | void pathrelse(struct treepath *search_path); |
1844 | int reiserfs_check_path(struct treepath *p); | 1844 | int reiserfs_check_path(struct treepath *p); |
1845 | void pathrelse_and_restore(struct super_block *s, struct treepath *p_s_search_path); | 1845 | void pathrelse_and_restore(struct super_block *s, struct treepath *search_path); |
1846 | 1846 | ||
1847 | int reiserfs_insert_item(struct reiserfs_transaction_handle *th, | 1847 | int reiserfs_insert_item(struct reiserfs_transaction_handle *th, |
1848 | struct treepath *path, | 1848 | struct treepath *path, |
@@ -1865,7 +1865,7 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th, | |||
1865 | int reiserfs_delete_item(struct reiserfs_transaction_handle *th, | 1865 | int reiserfs_delete_item(struct reiserfs_transaction_handle *th, |
1866 | struct treepath *path, | 1866 | struct treepath *path, |
1867 | const struct cpu_key *key, | 1867 | const struct cpu_key *key, |
1868 | struct inode *inode, struct buffer_head *p_s_un_bh); | 1868 | struct inode *inode, struct buffer_head *un_bh); |
1869 | 1869 | ||
1870 | void reiserfs_delete_solid_item(struct reiserfs_transaction_handle *th, | 1870 | void reiserfs_delete_solid_item(struct reiserfs_transaction_handle *th, |
1871 | struct inode *inode, struct reiserfs_key *key); | 1871 | struct inode *inode, struct reiserfs_key *key); |
@@ -2005,7 +2005,7 @@ extern const struct address_space_operations reiserfs_address_space_operations; | |||
2005 | /* fix_nodes.c */ | 2005 | /* fix_nodes.c */ |
2006 | 2006 | ||
2007 | int fix_nodes(int n_op_mode, struct tree_balance *tb, | 2007 | int fix_nodes(int n_op_mode, struct tree_balance *tb, |
2008 | struct item_head *p_s_ins_ih, const void *); | 2008 | struct item_head *ins_ih, const void *); |
2009 | void unfix_nodes(struct tree_balance *); | 2009 | void unfix_nodes(struct tree_balance *); |
2010 | 2010 | ||
2011 | /* prints.c */ | 2011 | /* prints.c */ |