diff options
| author | Joe Perches <joe@perches.com> | 2011-10-10 01:08:05 -0700 | 
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2011-10-31 23:43:52 +0100 | 
| commit | 78ace70c4186c0d18314eb001637aa97d1585e65 (patch) | |
| tree | 34227a6d50b93eb66df22ba130bea8ed932a5896 /fs/udf/inode.c | |
| parent | a40ecd7b3ccf520ff02da93e8d1ba6cd55c2e359 (diff) | |
| download | olio-linux-3.10-78ace70c4186c0d18314eb001637aa97d1585e65.tar.xz olio-linux-3.10-78ace70c4186c0d18314eb001637aa97d1585e65.zip  | |
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>
Diffstat (limited to 'fs/udf/inode.c')
| -rw-r--r-- | fs/udf/inode.c | 35 | 
1 files changed, 15 insertions, 20 deletions
diff --git a/fs/udf/inode.c b/fs/udf/inode.c index f94d6f9febf..f67e7e268c4 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;  		}  	}  |