]> pilppa.org Git - linux-2.6-omap-h63xx.git/blobdiff - fs/reiserfs/stree.c
reiserfs: rename p_s_bh to bh
[linux-2.6-omap-h63xx.git] / fs / reiserfs / stree.c
index abbc64dcc8d453cd8bd2281e382c85c37bd4fbf6..eb6856f6d3233f922ce75dd174fc2b5a57f52d00 100644 (file)
@@ -23,7 +23,6 @@
  * get_rkey
  * key_in_buffer
  * decrement_bcount
- * decrement_counters_in_path
  * reiserfs_check_path
  * pathrelse_and_restore
  * pathrelse
 #include <linux/quotaops.h>
 
 /* Does the buffer contain a disk block which is in the tree. */
-inline int B_IS_IN_TREE(const struct buffer_head *p_s_bh)
+inline int B_IS_IN_TREE(const struct buffer_head *bh)
 {
 
-       RFALSE(B_LEVEL(p_s_bh) > MAX_HEIGHT,
-              "PAP-1010: block (%b) has too big level (%z)", p_s_bh, p_s_bh);
+       RFALSE(B_LEVEL(bh) > MAX_HEIGHT,
+              "PAP-1010: block (%b) has too big level (%z)", bh, bh);
 
-       return (B_LEVEL(p_s_bh) != FREE_LEVEL);
+       return (B_LEVEL(bh) != FREE_LEVEL);
 }
 
 //
@@ -78,7 +77,7 @@ inline void copy_item_head(struct item_head *p_v_to,
 /* k1 is pointer to on-disk structure which is stored in little-endian
    form. k2 is pointer to cpu variable. For key of items of the same
    object this returns 0.
-   Returns: -1 if key1 < key2 
+   Returns: -1 if key1 < key2
    0 if key1 == key2
    1 if key1 > key2 */
 inline int comp_short_keys(const struct reiserfs_key *le_key,
@@ -246,7 +245,7 @@ static const struct reiserfs_key MAX_KEY = {
 static inline const struct reiserfs_key *get_lkey(const struct treepath
                                                  *p_s_chk_path,
                                                  const struct super_block
-                                                 *p_s_sb)
+                                                 *sb)
 {
        int n_position, n_path_offset = p_s_chk_path->path_length;
        struct buffer_head *p_s_parent;
@@ -283,14 +282,14 @@ static inline const struct reiserfs_key *get_lkey(const struct treepath
        }
        /* Return MIN_KEY if we are in the root of the buffer tree. */
        if (PATH_OFFSET_PBUFFER(p_s_chk_path, FIRST_PATH_ELEMENT_OFFSET)->
-           b_blocknr == SB_ROOT_BLOCK(p_s_sb))
+           b_blocknr == SB_ROOT_BLOCK(sb))
                return &MIN_KEY;
        return &MAX_KEY;
 }
 
 /* Get delimiting key of the buffer at the path and its right neighbor. */
 inline const struct reiserfs_key *get_rkey(const struct treepath *p_s_chk_path,
-                                          const struct super_block *p_s_sb)
+                                          const struct super_block *sb)
 {
        int n_position, n_path_offset = p_s_chk_path->path_length;
        struct buffer_head *p_s_parent;
@@ -326,7 +325,7 @@ inline const struct reiserfs_key *get_rkey(const struct treepath *p_s_chk_path,
        }
        /* Return MAX_KEY if we are in the root of the buffer tree. */
        if (PATH_OFFSET_PBUFFER(p_s_chk_path, FIRST_PATH_ELEMENT_OFFSET)->
-           b_blocknr == SB_ROOT_BLOCK(p_s_sb))
+           b_blocknr == SB_ROOT_BLOCK(sb))
                return &MAX_KEY;
        return &MIN_KEY;
 }
@@ -338,7 +337,7 @@ inline const struct reiserfs_key *get_rkey(const struct treepath *p_s_chk_path,
    this case get_lkey and get_rkey return a special key which is MIN_KEY or MAX_KEY. */
 static inline int key_in_buffer(struct treepath *p_s_chk_path, /* Path which should be checked.  */
                                const struct cpu_key *p_s_key,  /* Key which should be checked.   */
-                               struct super_block *p_s_sb      /* Super block pointer.           */
+                               struct super_block *sb  /* Super block pointer.    */
     )
 {
 
@@ -349,47 +348,16 @@ static inline int key_in_buffer(struct treepath *p_s_chk_path,    /* Path which sho
        RFALSE(!PATH_PLAST_BUFFER(p_s_chk_path)->b_bdev,
               "PAP-5060: device must not be NODEV");
 
-       if (comp_keys(get_lkey(p_s_chk_path, p_s_sb), p_s_key) == 1)
+       if (comp_keys(get_lkey(p_s_chk_path, sb), p_s_key) == 1)
                /* left delimiting key is bigger, that the key we look for */
                return 0;
-       //  if ( comp_keys(p_s_key, get_rkey(p_s_chk_path, p_s_sb)) != -1 )
-       if (comp_keys(get_rkey(p_s_chk_path, p_s_sb), p_s_key) != 1)
+       //  if ( comp_keys(p_s_key, get_rkey(p_s_chk_path, sb)) != -1 )
+       if (comp_keys(get_rkey(p_s_chk_path, sb), p_s_key) != 1)
                /* p_s_key must be less than right delimitiing key */
                return 0;
        return 1;
 }
 
-inline void decrement_bcount(struct buffer_head *p_s_bh)
-{
-       if (p_s_bh) {
-               if (atomic_read(&(p_s_bh->b_count))) {
-                       put_bh(p_s_bh);
-                       return;
-               }
-               reiserfs_panic(NULL,
-                              "PAP-5070: decrement_bcount: trying to free free buffer %b",
-                              p_s_bh);
-       }
-}
-
-/* Decrement b_count field of the all buffers in the path. */
-void decrement_counters_in_path(struct treepath *p_s_search_path)
-{
-       int n_path_offset = p_s_search_path->path_length;
-
-       RFALSE(n_path_offset < ILLEGAL_PATH_ELEMENT_OFFSET ||
-              n_path_offset > EXTENDED_MAX_HEIGHT - 1,
-              "PAP-5080: invalid path offset of %d", n_path_offset);
-
-       while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) {
-               struct buffer_head *bh;
-
-               bh = PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--);
-               decrement_bcount(bh);
-       }
-       p_s_search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET;
-}
-
 int reiserfs_check_path(struct treepath *p)
 {
        RFALSE(p->path_length != ILLEGAL_PATH_ELEMENT_OFFSET,
@@ -397,12 +365,11 @@ int reiserfs_check_path(struct treepath *p)
        return 0;
 }
 
-/* Release all buffers in the path. Restore dirty bits clean
-** when preparing the buffer for the log
-**
-** only called from fix_nodes()
-*/
-void pathrelse_and_restore(struct super_block *s, struct treepath *p_s_search_path)
+/* Drop the reference to each buffer in a path and restore
+ * dirty bits clean when preparing the buffer for the log.
+ * This version should only be called from fix_nodes() */
+void pathrelse_and_restore(struct super_block *sb,
+                          struct treepath *p_s_search_path)
 {
        int n_path_offset = p_s_search_path->path_length;
 
@@ -410,16 +377,15 @@ void pathrelse_and_restore(struct super_block *s, struct treepath *p_s_search_pa
               "clm-4000: invalid path offset");
 
        while (n_path_offset > ILLEGAL_PATH_ELEMENT_OFFSET) {
-               reiserfs_restore_prepared_buffer(s,
-                                                PATH_OFFSET_PBUFFER
-                                                (p_s_search_path,
-                                                 n_path_offset));
-               brelse(PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--));
+               struct buffer_head *bh;
+               bh = PATH_OFFSET_PBUFFER(p_s_search_path, n_path_offset--);
+               reiserfs_restore_prepared_buffer(sb, bh);
+               brelse(bh);
        }
        p_s_search_path->path_length = ILLEGAL_PATH_ELEMENT_OFFSET;
 }
 
-/* Release all buffers in the path. */
+/* Drop the reference to each buffer in a path */
 void pathrelse(struct treepath *p_s_search_path)
 {
        int n_path_offset = p_s_search_path->path_length;
@@ -444,23 +410,24 @@ static int is_leaf(char *buf, int blocksize, struct buffer_head *bh)
 
        blkh = (struct block_head *)buf;
        if (blkh_level(blkh) != DISK_LEAF_NODE_LEVEL) {
-               reiserfs_warning(NULL,
-                                "is_leaf: this should be caught earlier");
+               reiserfs_warning(NULL, "reiserfs-5080",
+                                "this should be caught earlier");
                return 0;
        }
 
        nr = blkh_nr_item(blkh);
        if (nr < 1 || nr > ((blocksize - BLKH_SIZE) / (IH_SIZE + MIN_ITEM_LEN))) {
                /* item number is too big or too small */
-               reiserfs_warning(NULL, "is_leaf: nr_item seems wrong: %z", bh);
+               reiserfs_warning(NULL, "reiserfs-5081",
+                                "nr_item seems wrong: %z", bh);
                return 0;
        }
        ih = (struct item_head *)(buf + BLKH_SIZE) + nr - 1;
        used_space = BLKH_SIZE + IH_SIZE * nr + (blocksize - ih_location(ih));
        if (used_space != blocksize - blkh_free_space(blkh)) {
                /* free space does not match to calculated amount of use space */
-               reiserfs_warning(NULL, "is_leaf: free space seems wrong: %z",
-                                bh);
+               reiserfs_warning(NULL, "reiserfs-5082",
+                                "free space seems wrong: %z", bh);
                return 0;
        }
        // FIXME: it is_leaf will hit performance too much - we may have
@@ -471,29 +438,29 @@ static int is_leaf(char *buf, int blocksize, struct buffer_head *bh)
        prev_location = blocksize;
        for (i = 0; i < nr; i++, ih++) {
                if (le_ih_k_type(ih) == TYPE_ANY) {
-                       reiserfs_warning(NULL,
-                                        "is_leaf: wrong item type for item %h",
+                       reiserfs_warning(NULL, "reiserfs-5083",
+                                        "wrong item type for item %h",
                                         ih);
                        return 0;
                }
                if (ih_location(ih) >= blocksize
                    || ih_location(ih) < IH_SIZE * nr) {
-                       reiserfs_warning(NULL,
-                                        "is_leaf: item location seems wrong: %h",
+                       reiserfs_warning(NULL, "reiserfs-5084",
+                                        "item location seems wrong: %h",
                                         ih);
                        return 0;
                }
                if (ih_item_len(ih) < 1
                    || ih_item_len(ih) > MAX_ITEM_LEN(blocksize)) {
-                       reiserfs_warning(NULL,
-                                        "is_leaf: item length seems wrong: %h",
+                       reiserfs_warning(NULL, "reiserfs-5085",
+                                        "item length seems wrong: %h",
                                         ih);
                        return 0;
                }
                if (prev_location - ih_location(ih) != ih_item_len(ih)) {
-                       reiserfs_warning(NULL,
-                                        "is_leaf: item location seems wrong (second one): %h",
-                                        ih);
+                       reiserfs_warning(NULL, "reiserfs-5086",
+                                        "item location seems wrong "
+                                        "(second one): %h", ih);
                        return 0;
                }
                prev_location = ih_location(ih);
@@ -514,24 +481,23 @@ static int is_internal(char *buf, int blocksize, struct buffer_head *bh)
        nr = blkh_level(blkh);
        if (nr <= DISK_LEAF_NODE_LEVEL || nr > MAX_HEIGHT) {
                /* this level is not possible for internal nodes */
-               reiserfs_warning(NULL,
-                                "is_internal: this should be caught earlier");
+               reiserfs_warning(NULL, "reiserfs-5087",
+                                "this should be caught earlier");
                return 0;
        }
 
        nr = blkh_nr_item(blkh);
        if (nr > (blocksize - BLKH_SIZE - DC_SIZE) / (KEY_SIZE + DC_SIZE)) {
                /* for internal which is not root we might check min number of keys */
-               reiserfs_warning(NULL,
-                                "is_internal: number of key seems wrong: %z",
-                                bh);
+               reiserfs_warning(NULL, "reiserfs-5088",
+                                "number of key seems wrong: %z", bh);
                return 0;
        }
 
        used_space = BLKH_SIZE + KEY_SIZE * nr + DC_SIZE * (nr + 1);
        if (used_space != blocksize - blkh_free_space(blkh)) {
-               reiserfs_warning(NULL,
-                                "is_internal: free space seems wrong: %z", bh);
+               reiserfs_warning(NULL, "reiserfs-5089",
+                                "free space seems wrong: %z", bh);
                return 0;
        }
        // one may imagine much more checks
@@ -543,8 +509,8 @@ static int is_internal(char *buf, int blocksize, struct buffer_head *bh)
 static int is_tree_node(struct buffer_head *bh, int level)
 {
        if (B_LEVEL(bh) != level) {
-               reiserfs_warning(NULL,
-                                "is_tree_node: node level %d does not match to the expected one %d",
+               reiserfs_warning(NULL, "reiserfs-5090", "node level %d does "
+                                "not match to the expected one %d",
                                 B_LEVEL(bh), level);
                return 0;
        }
@@ -580,7 +546,7 @@ static void search_by_key_reada(struct super_block *s,
 /**************************************************************************
  * Algorithm   SearchByKey                                                *
  *             look for item in the Disk S+Tree by its key                *
- * Input:  p_s_sb   -  super block                                        *
+ * Input:  sb   -  super block                                            *
  *         p_s_key  - pointer to the key to search                        *
  * Output: ITEM_FOUND, ITEM_NOT_FOUND or IO_ERROR                         *
  *         p_s_search_path - path from the root to the needed leaf        *
@@ -600,7 +566,7 @@ static void search_by_key_reada(struct super_block *s,
    correctness of the top of the path but need not be checked for the
    correctness of the bottom of the path */
 /* The function is NOT SCHEDULE-SAFE! */
-int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,   /* Key to search. */
+int search_by_key(struct super_block *sb, const struct cpu_key *p_s_key,       /* Key to search. */
                  struct treepath *p_s_search_path,/* This structure was
                                                   allocated and initialized
                                                   by the calling
@@ -613,7 +579,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,        /*
 {
        b_blocknr_t n_block_number;
        int expected_level;
-       struct buffer_head *p_s_bh;
+       struct buffer_head *bh;
        struct path_element *p_s_last_element;
        int n_node_level, n_retval;
        int right_neighbor_of_leaf_node;
@@ -626,28 +592,28 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,      /*
        int n_repeat_counter = 0;
 #endif
 
-       PROC_INFO_INC(p_s_sb, search_by_key);
+       PROC_INFO_INC(sb, search_by_key);
 
        /* As we add each node to a path we increase its count.  This means that
           we must be careful to release all nodes in a path before we either
           discard the path struct or re-use the path struct, as we do here. */
 
-       decrement_counters_in_path(p_s_search_path);
+       pathrelse(p_s_search_path);
 
        right_neighbor_of_leaf_node = 0;
 
        /* With each iteration of this loop we search through the items in the
           current node, and calculate the next current node(next path element)
           for the next iteration of this loop.. */
-       n_block_number = SB_ROOT_BLOCK(p_s_sb);
+       n_block_number = SB_ROOT_BLOCK(sb);
        expected_level = -1;
        while (1) {
 
 #ifdef CONFIG_REISERFS_CHECK
                if (!(++n_repeat_counter % 50000))
-                       reiserfs_warning(p_s_sb, "PAP-5100: search_by_key: %s:"
-                                        "there were %d iterations of while loop "
-                                        "looking for key %K",
+                       reiserfs_warning(sb, "PAP-5100",
+                                        "%s: there were %d iterations of "
+                                        "while loop looking for key %K",
                                         current->comm, n_repeat_counter,
                                         p_s_key);
 #endif
@@ -656,19 +622,18 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,      /*
                p_s_last_element =
                    PATH_OFFSET_PELEMENT(p_s_search_path,
                                         ++p_s_search_path->path_length);
-               fs_gen = get_generation(p_s_sb);
+               fs_gen = get_generation(sb);
 
                /* Read the next tree node, and set the last element in the path to
                   have a pointer to it. */
-               if ((p_s_bh = p_s_last_element->pe_buffer =
-                    sb_getblk(p_s_sb, n_block_number))) {
-                       if (!buffer_uptodate(p_s_bh) && reada_count > 1) {
-                               search_by_key_reada(p_s_sb, reada_bh,
+               if ((bh = p_s_last_element->pe_buffer =
+                    sb_getblk(sb, n_block_number))) {
+                       if (!buffer_uptodate(bh) && reada_count > 1)
+                               search_by_key_reada(sb, reada_bh,
                                                    reada_blocks, reada_count);
-                       }
-                       ll_rw_block(READ, 1, &p_s_bh);
-                       wait_on_buffer(p_s_bh);
-                       if (!buffer_uptodate(p_s_bh))
+                       ll_rw_block(READ, 1, &bh);
+                       wait_on_buffer(bh);
+                       if (!buffer_uptodate(bh))
                                goto io_error;
                } else {
                      io_error:
@@ -678,25 +643,25 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,      /*
                }
                reada_count = 0;
                if (expected_level == -1)
-                       expected_level = SB_TREE_HEIGHT(p_s_sb);
+                       expected_level = SB_TREE_HEIGHT(sb);
                expected_level--;
 
                /* It is possible that schedule occurred. We must check whether the key
                   to search is still in the tree rooted from the current buffer. If
                   not then repeat search from the root. */
-               if (fs_changed(fs_gen, p_s_sb) &&
-                   (!B_IS_IN_TREE(p_s_bh) ||
-                    B_LEVEL(p_s_bh) != expected_level ||
-                    !key_in_buffer(p_s_search_path, p_s_key, p_s_sb))) {
-                       PROC_INFO_INC(p_s_sb, search_by_key_fs_changed);
-                       PROC_INFO_INC(p_s_sb, search_by_key_restarted);
-                       PROC_INFO_INC(p_s_sb,
+               if (fs_changed(fs_gen, sb) &&
+                   (!B_IS_IN_TREE(bh) ||
+                    B_LEVEL(bh) != expected_level ||
+                    !key_in_buffer(p_s_search_path, p_s_key, sb))) {
+                       PROC_INFO_INC(sb, search_by_key_fs_changed);
+                       PROC_INFO_INC(sb, search_by_key_restarted);
+                       PROC_INFO_INC(sb,
                                      sbk_restarted[expected_level - 1]);
-                       decrement_counters_in_path(p_s_search_path);
+                       pathrelse(p_s_search_path);
 
                        /* Get the root block number so that we can repeat the search
                           starting from the root. */
-                       n_block_number = SB_ROOT_BLOCK(p_s_sb);
+                       n_block_number = SB_ROOT_BLOCK(sb);
                        expected_level = -1;
                        right_neighbor_of_leaf_node = 0;
 
@@ -708,37 +673,37 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,      /*
                   equal to the MAX_KEY. Latter case is only possible in
                   "finish_unfinished()" processing during mount. */
                RFALSE(comp_keys(&MAX_KEY, p_s_key) &&
-                      !key_in_buffer(p_s_search_path, p_s_key, p_s_sb),
+                      !key_in_buffer(p_s_search_path, p_s_key, sb),
                       "PAP-5130: key is not in the buffer");
 #ifdef CONFIG_REISERFS_CHECK
                if (cur_tb) {
                        print_cur_tb("5140");
-                       reiserfs_panic(p_s_sb,
-                                      "PAP-5140: search_by_key: schedule occurred in do_balance!");
+                       reiserfs_panic(sb, "PAP-5140",
+                                      "schedule occurred in do_balance!");
                }
 #endif
 
                // make sure, that the node contents look like a node of
                // certain level
-               if (!is_tree_node(p_s_bh, expected_level)) {
-                       reiserfs_warning(p_s_sb, "vs-5150: search_by_key: "
-                                        "invalid format found in block %ld. Fsck?",
-                                        p_s_bh->b_blocknr);
+               if (!is_tree_node(bh, expected_level)) {
+                       reiserfs_error(sb, "vs-5150",
+                                      "invalid format found in block %ld. "
+                                      "Fsck?", bh->b_blocknr);
                        pathrelse(p_s_search_path);
                        return IO_ERROR;
                }
 
                /* ok, we have acquired next formatted node in the tree */
-               n_node_level = B_LEVEL(p_s_bh);
+               n_node_level = B_LEVEL(bh);
 
-               PROC_INFO_BH_STAT(p_s_sb, p_s_bh, n_node_level - 1);
+               PROC_INFO_BH_STAT(sb, bh, n_node_level - 1);
 
                RFALSE(n_node_level < n_stop_level,
                       "vs-5152: tree level (%d) is less than stop level (%d)",
                       n_node_level, n_stop_level);
 
-               n_retval = bin_search(p_s_key, B_N_PITEM_HEAD(p_s_bh, 0),
-                                     B_NR_ITEMS(p_s_bh),
+               n_retval = bin_search(p_s_key, B_N_PITEM_HEAD(bh, 0),
+                                     B_NR_ITEMS(bh),
                                      (n_node_level ==
                                       DISK_LEAF_NODE_LEVEL) ? IH_SIZE :
                                      KEY_SIZE,
@@ -760,13 +725,13 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,      /*
                   an internal node.  Now we calculate child block number by
                   position in the node. */
                n_block_number =
-                   B_N_CHILD_NUM(p_s_bh, p_s_last_element->pe_position);
+                   B_N_CHILD_NUM(bh, p_s_last_element->pe_position);
 
                /* if we are going to read leaf nodes, try for read ahead as well */
                if ((p_s_search_path->reada & PATH_READA) &&
                    n_node_level == DISK_LEAF_NODE_LEVEL + 1) {
                        int pos = p_s_last_element->pe_position;
-                       int limit = B_NR_ITEMS(p_s_bh);
+                       int limit = B_NR_ITEMS(bh);
                        struct reiserfs_key *le_key;
 
                        if (p_s_search_path->reada & PATH_READA_BACK)
@@ -775,7 +740,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,        /*
                                if (pos == limit)
                                        break;
                                reada_blocks[reada_count++] =
-                                   B_N_CHILD_NUM(p_s_bh, pos);
+                                   B_N_CHILD_NUM(bh, pos);
                                if (p_s_search_path->reada & PATH_READA_BACK)
                                        pos--;
                                else
@@ -784,7 +749,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,        /*
                                /*
                                 * check to make sure we're in the same object
                                 */
-                               le_key = B_N_PDELIM_KEY(p_s_bh, pos);
+                               le_key = B_N_PDELIM_KEY(bh, pos);
                                if (le32_to_cpu(le_key->k_objectid) !=
                                    p_s_key->on_disk_key.k_objectid) {
                                        break;
@@ -810,7 +775,7 @@ int search_by_key(struct super_block *p_s_sb, const struct cpu_key *p_s_key,        /*
    units of directory entries.  */
 
 /* The function is NOT SCHEDULE-SAFE! */
-int search_for_position_by_key(struct super_block *p_s_sb,     /* Pointer to the super block.          */
+int search_for_position_by_key(struct super_block *sb, /* Pointer to the super block.          */
                               const struct cpu_key *p_cpu_key, /* Key to search (cpu variable)         */
                               struct treepath *p_s_search_path /* Filled up by this function.          */
     )
@@ -823,13 +788,13 @@ int search_for_position_by_key(struct super_block *p_s_sb,        /* Pointer to the sup
 
        /* If searching for directory entry. */
        if (is_direntry_cpu_key(p_cpu_key))
-               return search_by_entry_key(p_s_sb, p_cpu_key, p_s_search_path,
+               return search_by_entry_key(sb, p_cpu_key, p_s_search_path,
                                           &de);
 
        /* If not searching for directory entry. */
 
        /* If item is found. */
-       retval = search_item(p_s_sb, p_cpu_key, p_s_search_path);
+       retval = search_item(sb, p_cpu_key, p_s_search_path);
        if (retval == IO_ERROR)
                return retval;
        if (retval == ITEM_FOUND) {
@@ -851,7 +816,7 @@ int search_for_position_by_key(struct super_block *p_s_sb,  /* Pointer to the sup
        p_le_ih =
            B_N_PITEM_HEAD(PATH_PLAST_BUFFER(p_s_search_path),
                           --PATH_LAST_POSITION(p_s_search_path));
-       n_blk_size = p_s_sb->s_blocksize;
+       n_blk_size = sb->s_blocksize;
 
        if (comp_short_keys(&(p_le_ih->ih_key), p_cpu_key)) {
                return FILE_NOT_FOUND;
@@ -885,15 +850,15 @@ int search_for_position_by_key(struct super_block *p_s_sb,        /* Pointer to the sup
 /* Compare given item and item pointed to by the path. */
 int comp_items(const struct item_head *stored_ih, const struct treepath *p_s_path)
 {
-       struct buffer_head *p_s_bh;
+       struct buffer_head *bh = PATH_PLAST_BUFFER(p_s_path);
        struct item_head *ih;
 
        /* Last buffer at the path is not in the tree. */
-       if (!B_IS_IN_TREE(p_s_bh = PATH_PLAST_BUFFER(p_s_path)))
+       if (!B_IS_IN_TREE(bh))
                return 1;
 
        /* Last path position is invalid. */
-       if (PATH_LAST_POSITION(p_s_path) >= B_NR_ITEMS(p_s_bh))
+       if (PATH_LAST_POSITION(p_s_path) >= B_NR_ITEMS(bh))
                return 1;
 
        /* we need only to know, whether it is the same item */
@@ -924,7 +889,7 @@ static inline int prepare_for_direct_item(struct treepath *path,
        }
        // new file gets truncated
        if (get_inode_item_key_version(inode) == KEY_FORMAT_3_6) {
-               // 
+               //
                round_len = ROUND_UP(new_file_length);
                /* this was n_new_file_length < le_ih ... */
                if (round_len < le_ih_k_offset(le_ih)) {
@@ -991,9 +956,9 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st
                                      int *p_n_cut_size, unsigned long long n_new_file_length   /* MAX_KEY_OFFSET in case of delete. */
     )
 {
-       struct super_block *p_s_sb = inode->i_sb;
+       struct super_block *sb = inode->i_sb;
        struct item_head *p_le_ih = PATH_PITEM_HEAD(p_s_path);
-       struct buffer_head *p_s_bh = PATH_PLAST_BUFFER(p_s_path);
+       struct buffer_head *bh = PATH_PLAST_BUFFER(p_s_path);
 
        BUG_ON(!th->t_trans_id);
 
@@ -1020,7 +985,7 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st
 
        /* Case of an indirect item. */
        {
-           int blk_size = p_s_sb->s_blocksize;
+           int blk_size = sb->s_blocksize;
            struct item_head s_ih;
            int need_re_search;
            int delete = 0;
@@ -1037,7 +1002,7 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st
            do {
                need_re_search = 0;
                *p_n_cut_size = 0;
-               p_s_bh = PATH_PLAST_BUFFER(p_s_path);
+               bh = PATH_PLAST_BUFFER(p_s_path);
                copy_item_head(&s_ih, PATH_PITEM_HEAD(p_s_path));
                pos = I_UNFM_NUM(&s_ih);
 
@@ -1053,13 +1018,13 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st
                        break;
                    }
 
-                   unfm = (__le32 *)B_I_PITEM(p_s_bh, &s_ih) + pos - 1;
+                   unfm = (__le32 *)B_I_PITEM(bh, &s_ih) + pos - 1;
                    block = get_block_num(unfm, 0);
 
                    if (block != 0) {
-                       reiserfs_prepare_for_journal(p_s_sb, p_s_bh, 1);
+                       reiserfs_prepare_for_journal(sb, bh, 1);
                        put_block_num(unfm, 0, 0);
-                       journal_mark_dirty (th, p_s_sb, p_s_bh);
+                       journal_mark_dirty(th, sb, bh);
                        reiserfs_free_block(th, inode, block, 1);
                    }
 
@@ -1083,9 +1048,9 @@ static char prepare_for_delete_or_cut(struct reiserfs_transaction_handle *th, st
                /* a trick.  If the buffer has been logged, this will do nothing.  If
                ** we've broken the loop without logging it, it will restore the
                ** buffer */
-               reiserfs_restore_prepared_buffer(p_s_sb, p_s_bh);
+               reiserfs_restore_prepared_buffer(sb, bh);
            } while (need_re_search &&
-                    search_for_position_by_key(p_s_sb, p_s_item_key, p_s_path) == POSITION_FOUND);
+                    search_for_position_by_key(sb, p_s_item_key, p_s_path) == POSITION_FOUND);
            pos_in_item(p_s_path) = pos * UNFM_P_SIZE;
 
            if (*p_n_cut_size == 0) {
@@ -1124,7 +1089,7 @@ static int calc_deleted_bytes_number(struct tree_balance *p_s_tb, char c_mode)
 
 static void init_tb_struct(struct reiserfs_transaction_handle *th,
                           struct tree_balance *p_s_tb,
-                          struct super_block *p_s_sb,
+                          struct super_block *sb,
                           struct treepath *p_s_path, int n_size)
 {
 
@@ -1132,7 +1097,7 @@ static void init_tb_struct(struct reiserfs_transaction_handle *th,
 
        memset(p_s_tb, '\0', sizeof(struct tree_balance));
        p_s_tb->transaction_handle = th;
-       p_s_tb->tb_sb = p_s_sb;
+       p_s_tb->tb_sb = sb;
        p_s_tb->tb_path = p_s_path;
        PATH_OFFSET_PBUFFER(p_s_path, ILLEGAL_PATH_ELEMENT_OFFSET) = NULL;
        PATH_OFFSET_POSITION(p_s_path, ILLEGAL_PATH_ELEMENT_OFFSET) = 0;
@@ -1181,7 +1146,7 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath
                         struct inode *p_s_inode,       /* inode is here just to update i_blocks and quotas */
                         struct buffer_head *p_s_un_bh)
 {                              /* NULL or unformatted node pointer.    */
-       struct super_block *p_s_sb = p_s_inode->i_sb;
+       struct super_block *sb = p_s_inode->i_sb;
        struct tree_balance s_del_balance;
        struct item_head s_ih;
        struct item_head *q_ih;
@@ -1195,7 +1160,7 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath
 
        BUG_ON(!th->t_trans_id);
 
-       init_tb_struct(th, &s_del_balance, p_s_sb, p_s_path,
+       init_tb_struct(th, &s_del_balance, sb, p_s_path,
                       0 /*size is unknown */ );
 
        while (1) {
@@ -1219,16 +1184,15 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath
                if (n_ret_value != REPEAT_SEARCH)
                        break;
 
-               PROC_INFO_INC(p_s_sb, delete_item_restarted);
+               PROC_INFO_INC(sb, delete_item_restarted);
 
                // file system changed, repeat search
                n_ret_value =
-                   search_for_position_by_key(p_s_sb, p_s_item_key, p_s_path);
+                   search_for_position_by_key(sb, p_s_item_key, p_s_path);
                if (n_ret_value == IO_ERROR)
                        break;
                if (n_ret_value == FILE_NOT_FOUND) {
-                       reiserfs_warning(p_s_sb,
-                                        "vs-5340: reiserfs_delete_item: "
+                       reiserfs_warning(sb, "vs-5340",
                                         "no items of the file %K found",
                                         p_s_item_key);
                        break;
@@ -1251,8 +1215,8 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath
         ** the unfm node once
         */
        if (!S_ISLNK(p_s_inode->i_mode) && is_direct_le_ih(q_ih)) {
-               if ((le_ih_k_offset(q_ih) & (p_s_sb->s_blocksize - 1)) == 1) {
-                       quota_cut_bytes = p_s_sb->s_blocksize + UNFM_P_SIZE;
+               if ((le_ih_k_offset(q_ih) & (sb->s_blocksize - 1)) == 1) {
+                       quota_cut_bytes = sb->s_blocksize + UNFM_P_SIZE;
                } else {
                        quota_cut_bytes = 0;
                }
@@ -1293,7 +1257,7 @@ int reiserfs_delete_item(struct reiserfs_transaction_handle *th, struct treepath
        do_balance(&s_del_balance, NULL, NULL, M_DELETE);
 
 #ifdef REISERQUOTA_DEBUG
-       reiserfs_debug(p_s_sb, REISERFS_DEBUG_CODE,
+       reiserfs_debug(sb, REISERFS_DEBUG_CODE,
                       "reiserquota delete_item(): freeing %u, id=%u type=%c",
                       quota_cut_bytes, p_s_inode->i_uid, head2type(&s_ih));
 #endif
@@ -1338,10 +1302,9 @@ void reiserfs_delete_solid_item(struct reiserfs_transaction_handle *th,
        while (1) {
                retval = search_item(th->t_super, &cpu_key, &path);
                if (retval == IO_ERROR) {
-                       reiserfs_warning(th->t_super,
-                                        "vs-5350: reiserfs_delete_solid_item: "
-                                        "i/o failure occurred trying to delete %K",
-                                        &cpu_key);
+                       reiserfs_error(th->t_super, "vs-5350",
+                                      "i/o failure occurred trying "
+                                      "to delete %K", &cpu_key);
                        break;
                }
                if (retval != ITEM_FOUND) {
@@ -1355,9 +1318,8 @@ void reiserfs_delete_solid_item(struct reiserfs_transaction_handle *th,
                             GET_GENERATION_NUMBER(le_key_k_offset
                                                   (le_key_version(key),
                                                    key)) == 1))
-                               reiserfs_warning(th->t_super,
-                                                "vs-5355: reiserfs_delete_solid_item: %k not found",
-                                                key);
+                               reiserfs_warning(th->t_super, "vs-5355",
+                                                "%k not found", key);
                        break;
                }
                if (!tb_init) {
@@ -1389,8 +1351,7 @@ void reiserfs_delete_solid_item(struct reiserfs_transaction_handle *th,
                        break;
                }
                // IO_ERROR, NO_DISK_SPACE, etc
-               reiserfs_warning(th->t_super,
-                                "vs-5360: reiserfs_delete_solid_item: "
+               reiserfs_warning(th->t_super, "vs-5360",
                                 "could not delete %K due to fix_nodes failure",
                                 &cpu_key);
                unfix_nodes(&tb);
@@ -1468,8 +1429,8 @@ static int maybe_indirect_to_direct(struct reiserfs_transaction_handle *th,
                                    const struct cpu_key *p_s_item_key,
                                    loff_t n_new_file_size, char *p_c_mode)
 {
-       struct super_block *p_s_sb = p_s_inode->i_sb;
-       int n_block_size = p_s_sb->s_blocksize;
+       struct super_block *sb = p_s_inode->i_sb;
+       int n_block_size = sb->s_blocksize;
        int cut_bytes;
        BUG_ON(!th->t_trans_id);
        BUG_ON(n_new_file_size != p_s_inode->i_size);
@@ -1481,7 +1442,7 @@ static int maybe_indirect_to_direct(struct reiserfs_transaction_handle *th,
        if (atomic_read(&p_s_inode->i_count) > 1 ||
            !tail_has_to_be_packed(p_s_inode) ||
            !page || (REISERFS_I(p_s_inode)->i_flags & i_nopack_mask)) {
-               // leave tail in an unformatted node    
+               /* leave tail in an unformatted node */
                *p_c_mode = M_SKIP_BALANCING;
                cut_bytes =
                    n_block_size - (n_new_file_size & (n_block_size - 1));
@@ -1515,8 +1476,8 @@ static void indirect_to_direct_roll_back(struct reiserfs_transaction_handle *th,
                /* look for the last byte of the tail */
                if (search_for_position_by_key(inode->i_sb, &tail_key, path) ==
                    POSITION_NOT_FOUND)
-                       reiserfs_panic(inode->i_sb,
-                                      "vs-5615: indirect_to_direct_roll_back: found invalid item");
+                       reiserfs_panic(inode->i_sb, "vs-5615",
+                                      "found invalid item");
                RFALSE(path->pos_in_item !=
                       ih_item_len(PATH_PITEM_HEAD(path)) - 1,
                       "vs-5616: appended bytes found");
@@ -1533,8 +1494,9 @@ static void indirect_to_direct_roll_back(struct reiserfs_transaction_handle *th,
                set_cpu_key_k_offset(&tail_key,
                                     cpu_key_k_offset(&tail_key) - removed);
        }
-       reiserfs_warning(inode->i_sb,
-                        "indirect_to_direct_roll_back: indirect_to_direct conversion has been rolled back due to lack of disk space");
+       reiserfs_warning(inode->i_sb, "reiserfs-5091", "indirect_to_direct "
+                        "conversion has been rolled back due to "
+                        "lack of disk space");
        //mark_file_without_tail (inode);
        mark_inode_dirty(inode);
 }
@@ -1546,7 +1508,7 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th,
                           struct inode *p_s_inode,
                           struct page *page, loff_t n_new_file_size)
 {
-       struct super_block *p_s_sb = p_s_inode->i_sb;
+       struct super_block *sb = p_s_inode->i_sb;
        /* Every function which is going to call do_balance must first
           create a tree_balance structure.  Then it must fill up this
           structure by using the init_tb_struct and fix_nodes functions.
@@ -1597,7 +1559,7 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th,
                        /* removing of last unformatted node will change value we
                           have to return to truncate. Save it */
                        retval2 = n_ret_value;
-                       /*retval2 = p_s_sb->s_blocksize - (n_new_file_size & (p_s_sb->s_blocksize - 1)); */
+                       /*retval2 = sb->s_blocksize - (n_new_file_size & (sb->s_blocksize - 1)); */
 
                        /* So, we have performed the first part of the conversion:
                           inserting the new direct item.  Now we are removing the
@@ -1606,17 +1568,17 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th,
                        set_cpu_key_k_type(p_s_item_key, TYPE_INDIRECT);
                        p_s_item_key->key_length = 4;
                        n_new_file_size -=
-                           (n_new_file_size & (p_s_sb->s_blocksize - 1));
+                           (n_new_file_size & (sb->s_blocksize - 1));
                        tail_pos = n_new_file_size;
                        set_cpu_key_k_offset(p_s_item_key, n_new_file_size + 1);
                        if (search_for_position_by_key
-                           (p_s_sb, p_s_item_key,
+                           (sb, p_s_item_key,
                             p_s_path) == POSITION_NOT_FOUND) {
                                print_block(PATH_PLAST_BUFFER(p_s_path), 3,
                                            PATH_LAST_POSITION(p_s_path) - 1,
                                            PATH_LAST_POSITION(p_s_path) + 1);
-                               reiserfs_panic(p_s_sb,
-                                              "PAP-5580: reiserfs_cut_from_item: item to convert does not exist (%K)",
+                               reiserfs_panic(sb, "PAP-5580", "item to "
+                                              "convert does not exist (%K)",
                                               p_s_item_key);
                        }
                        continue;
@@ -1632,15 +1594,14 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th,
                if (n_ret_value != REPEAT_SEARCH)
                        break;
 
-               PROC_INFO_INC(p_s_sb, cut_from_item_restarted);
+               PROC_INFO_INC(sb, cut_from_item_restarted);
 
                n_ret_value =
-                   search_for_position_by_key(p_s_sb, p_s_item_key, p_s_path);
+                   search_for_position_by_key(sb, p_s_item_key, p_s_path);
                if (n_ret_value == POSITION_FOUND)
                        continue;
 
-               reiserfs_warning(p_s_sb,
-                                "PAP-5610: reiserfs_cut_from_item: item %K not found",
+               reiserfs_warning(sb, "PAP-5610", "item %K not found",
                                 p_s_item_key);
                unfix_nodes(&s_cut_balance);
                return (n_ret_value == IO_ERROR) ? -EIO : -ENOENT;
@@ -1654,7 +1615,8 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th,
                        indirect_to_direct_roll_back(th, p_s_inode, p_s_path);
                }
                if (n_ret_value == NO_DISK_SPACE)
-                       reiserfs_warning(p_s_sb, "NO_DISK_SPACE");
+                       reiserfs_warning(sb, "reiserfs-5092",
+                                        "NO_DISK_SPACE");
                unfix_nodes(&s_cut_balance);
                return -EIO;
        }
@@ -1679,11 +1641,11 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th,
        p_le_ih = PATH_PITEM_HEAD(s_cut_balance.tb_path);
        if (!S_ISLNK(p_s_inode->i_mode) && is_direct_le_ih(p_le_ih)) {
                if (c_mode == M_DELETE &&
-                   (le_ih_k_offset(p_le_ih) & (p_s_sb->s_blocksize - 1)) ==
+                   (le_ih_k_offset(p_le_ih) & (sb->s_blocksize - 1)) ==
                    1) {
                        // FIXME: this is to keep 3.5 happy
                        REISERFS_I(p_s_inode)->i_first_direct_byte = U32_MAX;
-                       quota_cut_bytes = p_s_sb->s_blocksize + UNFM_P_SIZE;
+                       quota_cut_bytes = sb->s_blocksize + UNFM_P_SIZE;
                } else {
                        quota_cut_bytes = 0;
                }
@@ -1696,22 +1658,20 @@ int reiserfs_cut_from_item(struct reiserfs_transaction_handle *th,
                   sure, that we exactly remove last unformatted node pointer
                   of the item */
                if (!is_indirect_le_ih(le_ih))
-                       reiserfs_panic(p_s_sb,
-                                      "vs-5652: reiserfs_cut_from_item: "
+                       reiserfs_panic(sb, "vs-5652",
                                       "item must be indirect %h", le_ih);
 
                if (c_mode == M_DELETE && ih_item_len(le_ih) != UNFM_P_SIZE)
-                       reiserfs_panic(p_s_sb,
-                                      "vs-5653: reiserfs_cut_from_item: "
-                                      "completing indirect2direct conversion indirect item %h "
-                                      "being deleted must be of 4 byte long",
-                                      le_ih);
+                       reiserfs_panic(sb, "vs-5653", "completing "
+                                      "indirect2direct conversion indirect "
+                                      "item %h being deleted must be of "
+                                      "4 byte long", le_ih);
 
                if (c_mode == M_CUT
                    && s_cut_balance.insert_size[0] != -UNFM_P_SIZE) {
-                       reiserfs_panic(p_s_sb,
-                                      "vs-5654: reiserfs_cut_from_item: "
-                                      "can not complete indirect2direct conversion of %h (CUT, insert_size==%d)",
+                       reiserfs_panic(sb, "vs-5654", "can not complete "
+                                      "indirect2direct conversion of %h "
+                                      "(CUT, insert_size==%d)",
                                       le_ih, s_cut_balance.insert_size[0]);
                }
                /* it would be useful to make sure, that right neighboring
@@ -1743,8 +1703,7 @@ static void truncate_directory(struct reiserfs_transaction_handle *th,
 {
        BUG_ON(!th->t_trans_id);
        if (inode->i_nlink)
-               reiserfs_warning(inode->i_sb,
-                                "vs-5655: truncate_directory: link count != 0");
+               reiserfs_error(inode->i_sb, "vs-5655", "link count != 0");
 
        set_le_key_k_offset(KEY_FORMAT_3_5, INODE_PKEY(inode), DOT_OFFSET);
        set_le_key_k_type(KEY_FORMAT_3_5, INODE_PKEY(inode), TYPE_DIRENTRY);
@@ -1797,18 +1756,16 @@ int reiserfs_do_truncate(struct reiserfs_transaction_handle *th, struct inode *p
            search_for_position_by_key(p_s_inode->i_sb, &s_item_key,
                                       &s_search_path);
        if (retval == IO_ERROR) {
-               reiserfs_warning(p_s_inode->i_sb,
-                                "vs-5657: reiserfs_do_truncate: "
-                                "i/o failure occurred trying to truncate %K",
-                                &s_item_key);
+               reiserfs_error(p_s_inode->i_sb, "vs-5657",
+                              "i/o failure occurred trying to truncate %K",
+                              &s_item_key);
                err = -EIO;
                goto out;
        }
        if (retval == POSITION_FOUND || retval == FILE_NOT_FOUND) {
-               reiserfs_warning(p_s_inode->i_sb,
-                                "PAP-5660: reiserfs_do_truncate: "
-                                "wrong result %d of search for %K", retval,
-                                &s_item_key);
+               reiserfs_error(p_s_inode->i_sb, "PAP-5660",
+                              "wrong result %d of search for %K", retval,
+                              &s_item_key);
 
                err = -EIO;
                goto out;
@@ -1850,8 +1807,8 @@ int reiserfs_do_truncate(struct reiserfs_transaction_handle *th, struct inode *p
                    reiserfs_cut_from_item(th, &s_search_path, &s_item_key,
                                           p_s_inode, page, n_new_file_size);
                if (n_deleted < 0) {
-                       reiserfs_warning(p_s_inode->i_sb,
-                                        "vs-5665: reiserfs_do_truncate: reiserfs_cut_from_item failed");
+                       reiserfs_warning(p_s_inode->i_sb, "vs-5665",
+                                        "reiserfs_cut_from_item failed");
                        reiserfs_check_path(&s_search_path);
                        return 0;
                }
@@ -1868,7 +1825,7 @@ int reiserfs_do_truncate(struct reiserfs_transaction_handle *th, struct inode *p
                /* While there are bytes to truncate and previous file item is presented in the tree. */
 
                /*
-                ** This loop could take a really long time, and could log 
+                ** This loop could take a really long time, and could log
                 ** many more blocks than a transaction can hold.  So, we do a polite
                 ** journal end here, and if the transaction needs ending, we make
                 ** sure the file is consistent before ending the current trans
@@ -1877,7 +1834,7 @@ int reiserfs_do_truncate(struct reiserfs_transaction_handle *th, struct inode *p
                if (journal_transaction_should_end(th, 0) ||
                    reiserfs_transaction_free_space(th) <= JOURNAL_FOR_FREE_BLOCK_AND_UPDATE_SD) {
                        int orig_len_alloc = th->t_blocks_allocated;
-                       decrement_counters_in_path(&s_search_path);
+                       pathrelse(&s_search_path);
 
                        if (update_timestamps) {
                                p_s_inode->i_mtime = p_s_inode->i_ctime =
@@ -1929,10 +1886,10 @@ static void check_research_for_paste(struct treepath *path,
                    || op_bytes_number(found_ih,
                                       get_last_bh(path)->b_size) !=
                    pos_in_item(path))
-                       reiserfs_panic(NULL,
-                                      "PAP-5720: check_research_for_paste: "
-                                      "found direct item %h or position (%d) does not match to key %K",
-                                      found_ih, pos_in_item(path), p_s_key);
+                       reiserfs_panic(NULL, "PAP-5720", "found direct item "
+                                      "%h or position (%d) does not match "
+                                      "to key %K", found_ih,
+                                      pos_in_item(path), p_s_key);
        }
        if (is_indirect_le_ih(found_ih)) {
                if (le_ih_k_offset(found_ih) +
@@ -1941,9 +1898,9 @@ static void check_research_for_paste(struct treepath *path,
                    cpu_key_k_offset(p_s_key)
                    || I_UNFM_NUM(found_ih) != pos_in_item(path)
                    || get_ih_free_space(found_ih) != 0)
-                       reiserfs_panic(NULL,
-                                      "PAP-5730: check_research_for_paste: "
-                                      "found indirect item (%h) or position (%d) does not match to key (%K)",
+                       reiserfs_panic(NULL, "PAP-5730", "found indirect "
+                                      "item (%h) or position (%d) does not "
+                                      "match to key (%K)",
                                       found_ih, pos_in_item(path), p_s_key);
        }
 }
@@ -2000,8 +1957,8 @@ int reiserfs_paste_into_item(struct reiserfs_transaction_handle *th, struct tree
                        goto error_out;
                }
                if (retval == POSITION_FOUND) {
-                       reiserfs_warning(inode->i_sb,
-                                        "PAP-5710: reiserfs_paste_into_item: entry or pasted byte (%K) exists",
+                       reiserfs_warning(inode->i_sb, "PAP-5710",
+                                        "entry or pasted byte (%K) exists",
                                         p_s_key);
                        retval = -EEXIST;
                        goto error_out;
@@ -2087,8 +2044,7 @@ int reiserfs_insert_item(struct reiserfs_transaction_handle *th, struct treepath
                        goto error_out;
                }
                if (retval == ITEM_FOUND) {
-                       reiserfs_warning(th->t_super,
-                                        "PAP-5760: reiserfs_insert_item: "
+                       reiserfs_warning(th->t_super, "PAP-5760",
                                         "key %K already exists in the tree",
                                         key);
                        retval = -EEXIST;