diff options
author | Dave Chinner <dchinner@redhat.com> | 2011-03-06 18:05:35 -0500 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2011-03-06 18:05:35 -0500 |
commit | 5348778699bba92bf28b79863e09e7181d8cf95c (patch) | |
tree | 77f5349bc85bf4d35b73a9a66de304535e507bf4 /fs/xfs/xfs_inode.c | |
parent | af34e09da42801c97f39f768c715f5511d914b52 (diff) |
xfs: convert xfs_fs_cmn_err to new error logging API
Continue to clean up the error logging code by converting all the
callers of xfs_fs_cmn_err() to the new API. Once done, remove the
unused old API function.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Alex Elder <aelder@sgi.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'fs/xfs/xfs_inode.c')
-rw-r--r-- | fs/xfs/xfs_inode.c | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c index 55169bbfc820..d820ada49b18 100644 --- a/fs/xfs/xfs_inode.c +++ b/fs/xfs/xfs_inode.c | |||
@@ -110,8 +110,8 @@ xfs_inobp_check( | |||
110 | dip = (xfs_dinode_t *)xfs_buf_offset(bp, | 110 | dip = (xfs_dinode_t *)xfs_buf_offset(bp, |
111 | i * mp->m_sb.sb_inodesize); | 111 | i * mp->m_sb.sb_inodesize); |
112 | if (!dip->di_next_unlinked) { | 112 | if (!dip->di_next_unlinked) { |
113 | xfs_fs_cmn_err(CE_ALERT, mp, | 113 | xfs_alert(mp, |
114 | "Detected a bogus zero next_unlinked field in incore inode buffer 0x%p. About to pop an ASSERT.", | 114 | "Detected bogus zero next_unlinked field in incore inode buffer 0x%p.", |
115 | bp); | 115 | bp); |
116 | ASSERT(dip->di_next_unlinked); | 116 | ASSERT(dip->di_next_unlinked); |
117 | } | 117 | } |
@@ -806,11 +806,9 @@ xfs_iread( | |||
806 | */ | 806 | */ |
807 | if (be16_to_cpu(dip->di_magic) != XFS_DINODE_MAGIC) { | 807 | if (be16_to_cpu(dip->di_magic) != XFS_DINODE_MAGIC) { |
808 | #ifdef DEBUG | 808 | #ifdef DEBUG |
809 | xfs_fs_cmn_err(CE_ALERT, mp, "xfs_iread: " | 809 | xfs_alert(mp, |
810 | "dip->di_magic (0x%x) != " | 810 | "%s: dip->di_magic (0x%x) != XFS_DINODE_MAGIC (0x%x)", |
811 | "XFS_DINODE_MAGIC (0x%x)", | 811 | __func__, be16_to_cpu(dip->di_magic), XFS_DINODE_MAGIC); |
812 | be16_to_cpu(dip->di_magic), | ||
813 | XFS_DINODE_MAGIC); | ||
814 | #endif /* DEBUG */ | 812 | #endif /* DEBUG */ |
815 | error = XFS_ERROR(EINVAL); | 813 | error = XFS_ERROR(EINVAL); |
816 | goto out_brelse; | 814 | goto out_brelse; |
@@ -828,9 +826,8 @@ xfs_iread( | |||
828 | error = xfs_iformat(ip, dip); | 826 | error = xfs_iformat(ip, dip); |
829 | if (error) { | 827 | if (error) { |
830 | #ifdef DEBUG | 828 | #ifdef DEBUG |
831 | xfs_fs_cmn_err(CE_ALERT, mp, "xfs_iread: " | 829 | xfs_alert(mp, "%s: xfs_iformat() returned error %d", |
832 | "xfs_iformat() returned error %d", | 830 | __func__, error); |
833 | error); | ||
834 | #endif /* DEBUG */ | 831 | #endif /* DEBUG */ |
835 | goto out_brelse; | 832 | goto out_brelse; |
836 | } | 833 | } |