reiserfs: rework reiserfs_warning

ReiserFS warnings can be somewhat inconsistent.
In some cases:
 * a unique identifier may be associated with it
 * the function name may be included
 * the device may be printed separately

This patch aims to make warnings more consistent. reiserfs_warning() prints
the device name, so printing it a second time is not required. The function
name for a warning is always helpful in debugging, so it is now automatically
inserted into the output. Hans has stated that every warning should have
a unique identifier. Some cases lack them, others really shouldn't have them.
reiserfs_warning() now expects an id associated with each message. In the
rare case where one isn't needed, "" will suffice.

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

1 files changed, 21 insertions, 24 deletions

diff --git a/fs/reiserfs/namei.c b/fs/reiserfs/namei.c
index 738967f6c8ee..bb41c6e7c79b 100644
--- a/fs/reiserfs/namei.c
+++ b/fs/reiserfs/namei.c
@@ -120,8 +120,8 @@ int search_by_entry_key(struct super_block *sb, const struct cpu_key *key,
         switch (retval) {
         case ITEM_NOT_FOUND:
                 if (!PATH_LAST_POSITION(path)) {
-                        reiserfs_warning(sb,
-                                         "vs-7000: search_by_entry_key: search_by_key returned item position == 0");
+                        reiserfs_warning(sb, "vs-7000", "search_by_key "
+                                         "returned item position == 0");
                         pathrelse(path);
                         return IO_ERROR;
                 }
@@ -135,8 +135,7 @@ int search_by_entry_key(struct super_block *sb, const struct cpu_key *key,
 
         default:
                 pathrelse(path);
-                reiserfs_warning(sb,
-                                 "vs-7002: search_by_entry_key: no path to here");
+                reiserfs_warning(sb, "vs-7002", "no path to here");
                 return IO_ERROR;
         }
 
@@ -300,8 +299,7 @@ static int reiserfs_find_entry(struct inode *dir, const char *name, int namelen,
                     search_by_entry_key(dir->i_sb, &key_to_search,
                                         path_to_entry, de);
                 if (retval == IO_ERROR) {
-                        reiserfs_warning(dir->i_sb, "zam-7001: io error in %s",
-                                         __func__);
+                        reiserfs_warning(dir->i_sb, "zam-7001", "io error");
                         return IO_ERROR;
                 }
 
@@ -484,10 +482,9 @@ static int reiserfs_add_entry(struct reiserfs_transaction_handle *th,
                 }
 
                 if (retval != NAME_FOUND) {
-                        reiserfs_warning(dir->i_sb,
-                                         "zam-7002:%s: \"reiserfs_find_entry\" "
-                                         "has returned unexpected value (%d)",
-                                         __func__, retval);
+                        reiserfs_warning(dir->i_sb, "zam-7002",
+                                         "reiserfs_find_entry() returned "
+                                         "unexpected value (%d)", retval);
                 }
 
                 return -EEXIST;
@@ -498,8 +495,9 @@ static int reiserfs_add_entry(struct reiserfs_transaction_handle *th,
                                 MAX_GENERATION_NUMBER + 1);
         if (gen_number > MAX_GENERATION_NUMBER) {
                 /* there is no free generation number */
-                reiserfs_warning(dir->i_sb,
-                                 "reiserfs_add_entry: Congratulations! we have got hash function screwed up");
+                reiserfs_warning(dir->i_sb, "reiserfs-7010",
+                                 "Congratulations! we have got hash function "
+                                 "screwed up");
                 if (buffer != small_buf)
                         kfree(buffer);
                 pathrelse(&path);
@@ -515,10 +513,9 @@ static int reiserfs_add_entry(struct reiserfs_transaction_handle *th,
         if (gen_number != 0) {  /* we need to re-search for the insertion point */
                 if (search_by_entry_key(dir->i_sb, &entry_key, &path, &de) !=
                     NAME_NOT_FOUND) {
-                        reiserfs_warning(dir->i_sb,
-                                         "vs-7032: reiserfs_add_entry: "
-                                         "entry with this key (%K) already exists",
-                                         &entry_key);
+                        reiserfs_warning(dir->i_sb, "vs-7032",
+                                         "entry with this key (%K) already "
+                                         "exists", &entry_key);
 
                         if (buffer != small_buf)
                                 kfree(buffer);
@@ -903,8 +900,9 @@ static int reiserfs_rmdir(struct inode *dir, struct dentry *dentry)
                 goto end_rmdir;
 
         if (inode->i_nlink != 2 && inode->i_nlink != 1)
-                reiserfs_warning(inode->i_sb, "%s: empty directory has nlink "
-                                 "!= 2 (%d)", __func__, inode->i_nlink);
+                reiserfs_warning(inode->i_sb, "reiserfs-7040",
+                                 "empty directory has nlink != 2 (%d)",
+                                 inode->i_nlink);
 
         clear_nlink(inode);
         inode->i_ctime = dir->i_ctime = dir->i_mtime = CURRENT_TIME_SEC;
@@ -980,10 +978,9 @@ static int reiserfs_unlink(struct inode *dir, struct dentry *dentry)
         }
 
         if (!inode->i_nlink) {
-                reiserfs_warning(inode->i_sb, "%s: deleting nonexistent file "
-                                 "(%s:%lu), %d", __func__,
-                                 reiserfs_bdevname(inode->i_sb), inode->i_ino,
-                                 inode->i_nlink);
+                reiserfs_warning(inode->i_sb, "reiserfs-7042",
+                                 "deleting nonexistent file (%lu), %d",
+                                 inode->i_ino, inode->i_nlink);
                 inode->i_nlink = 1;
         }
 
@@ -1499,8 +1496,8 @@ static int reiserfs_rename(struct inode *old_dir, struct dentry *old_dentry,
         if (reiserfs_cut_from_item
             (&th, &old_entry_path, &(old_de.de_entry_key), old_dir, NULL,
              0) < 0)
-                reiserfs_warning(old_dir->i_sb,
-                                 "vs-7060: reiserfs_rename: couldn't not cut old name. Fsck later?");
+                reiserfs_warning(old_dir->i_sb, "vs-7060",
+                                 "couldn't not cut old name. Fsck later?");
 
         old_dir->i_size -= DEH_SIZE + old_de.de_entrylen;