udf: Convert printks to pr_<level>

Use the current logging styles.

Convert a few printks that should have been udf_warn and udf_err.
Coalesce formats.  Add #define pr_fmt.
Move an #include "udfdecls.h" above other includes in udftime.c
so pr_fmt works correctly.  Strip prefixes from conversions as appropriate.
Reorder logging definitions in udfdecl.h

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jan Kara <jack@suse.cz>

1 files changed, 15 insertions, 20 deletions

diff --git a/fs/udf/inode.c b/fs/udf/inode.c
index f94d6f9febf5..f67e7e268c49 100644
--- a/fs/udf/inode.c
+++ b/fs/udf/inode.c
@@ -84,12 +84,10 @@ void udf_evict_inode(struct inode *inode)
         end_writeback(inode);
         if (iinfo->i_alloc_type != ICBTAG_FLAG_AD_IN_ICB &&
             inode->i_size != iinfo->i_lenExtents) {
-                printk(KERN_WARNING "UDF-fs (%s): Inode %lu (mode %o) has "
-                        "inode size %llu different from extent length %llu. "
-                        "Filesystem need not be standards compliant.\n",
-                        inode->i_sb->s_id, inode->i_ino, inode->i_mode,
-                        (unsigned long long)inode->i_size,
-                        (unsigned long long)iinfo->i_lenExtents);
+                udf_warn(inode->i_sb, "Inode %lu (mode %o) has inode size %llu different from extent length %llu. Filesystem need not be standards compliant.\n",
+                         inode->i_ino, inode->i_mode,
+                         (unsigned long long)inode->i_size,
+                         (unsigned long long)iinfo->i_lenExtents);
         }
         kfree(iinfo->i_ext.i_data);
         iinfo->i_ext.i_data = NULL;
@@ -1177,16 +1175,15 @@ static void __udf_read_inode(struct inode *inode)
          */
         bh = udf_read_ptagged(inode->i_sb, &iinfo->i_location, 0, &ident);
         if (!bh) {
-                printk(KERN_ERR "udf: udf_read_inode(ino %ld) failed !bh\n",
-                       inode->i_ino);
+                udf_err(inode->i_sb, "(ino %ld) failed !bh\n", inode->i_ino);
                 make_bad_inode(inode);
                 return;
         }
 
         if (ident != TAG_IDENT_FE && ident != TAG_IDENT_EFE &&
             ident != TAG_IDENT_USE) {
-                printk(KERN_ERR "udf: udf_read_inode(ino %ld) "
-                                "failed ident=%d\n", inode->i_ino, ident);
+                udf_err(inode->i_sb, "(ino %ld) failed ident=%d\n",
+                        inode->i_ino, ident);
                 brelse(bh);
                 make_bad_inode(inode);
                 return;
@@ -1226,8 +1223,8 @@ static void __udf_read_inode(struct inode *inode)
                 }
                 brelse(ibh);
         } else if (fe->icbTag.strategyType != cpu_to_le16(4)) {
-                printk(KERN_ERR "udf: unsupported strategy type: %d\n",
-                       le16_to_cpu(fe->icbTag.strategyType));
+                udf_err(inode->i_sb, "unsupported strategy type: %d\n",
+                        le16_to_cpu(fe->icbTag.strategyType));
                 brelse(bh);
                 make_bad_inode(inode);
                 return;
@@ -1421,9 +1418,8 @@ static void udf_fill_inode(struct inode *inode, struct buffer_head *bh)
                 udf_debug("METADATA BITMAP FILE-----\n");
                 break;
         default:
-                printk(KERN_ERR "udf: udf_fill_inode(ino %ld) failed unknown "
-                                "file type=%d\n", inode->i_ino,
-                                fe->icbTag.fileType);
+                udf_err(inode->i_sb, "(ino %ld) failed unknown file type=%d\n",
+                        inode->i_ino, fe->icbTag.fileType);
                 make_bad_inode(inode);
                 return;
         }
@@ -1446,8 +1442,8 @@ static int udf_alloc_i_data(struct inode *inode, size_t size)
         iinfo->i_ext.i_data = kmalloc(size, GFP_KERNEL);
 
         if (!iinfo->i_ext.i_data) {
-                printk(KERN_ERR "udf:udf_alloc_i_data (ino %ld) "
-                                "no free memory\n", inode->i_ino);
+                udf_err(inode->i_sb, "(ino %ld) no free memory\n",
+                        inode->i_ino);
                 return -ENOMEM;
         }
 
@@ -1697,9 +1693,8 @@ out:
         if (do_sync) {
                 sync_dirty_buffer(bh);
                 if (buffer_write_io_error(bh)) {
-                        printk(KERN_WARNING "IO error syncing udf inode "
-                                "[%s:%08lx]\n", inode->i_sb->s_id,
-                                inode->i_ino);
+                        udf_warn(inode->i_sb, "IO error syncing udf inode [%08lx]\n",
+                                 inode->i_ino);
                         err = -EIO;
                 }
         }