xfs: Nuke XFS_ERROR macro

XFS_ERROR was designed long ago to trap return values, but it's not
runtime configurable, it's not consistently used, and we can do
similar error trapping with ftrace scripts and triggers from
userspace.

Just nuke XFS_ERROR and associated bits.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>

1 files changed, 8 insertions, 8 deletions

diff --git a/fs/xfs/xfs_ialloc.c b/fs/xfs/xfs_ialloc.c
index 5960e5593fe0..16fb63a9bc5e 100644
--- a/fs/xfs/xfs_ialloc.c
+++ b/fs/xfs/xfs_ialloc.c
@@ -380,7 +380,7 @@ xfs_ialloc_ag_alloc(
         newlen = args.mp->m_ialloc_inos;
         if (args.mp->m_maxicount &&
             args.mp->m_sb.sb_icount + newlen > args.mp->m_maxicount)
-                return XFS_ERROR(ENOSPC);
+                return ENOSPC;
         args.minlen = args.maxlen = args.mp->m_ialloc_blks;
         /*
          * First try to allocate inodes contiguous with the last-allocated
@@ -1425,7 +1425,7 @@ out_alloc:
         return xfs_dialloc_ag(tp, agbp, parent, inop);
 out_error:
         xfs_perag_put(pag);
-        return XFS_ERROR(error);
+        return error;
 }
 
 STATIC int
@@ -1682,7 +1682,7 @@ xfs_difree(
                 xfs_warn(mp, "%s: agno >= mp->m_sb.sb_agcount (%d >= %d).",
                         __func__, agno, mp->m_sb.sb_agcount);
                 ASSERT(0);
-                return XFS_ERROR(EINVAL);
+                return EINVAL;
         }
         agino = XFS_INO_TO_AGINO(mp, inode);
         if (inode != XFS_AGINO_TO_INO(mp, agno, agino))  {
@@ -1690,14 +1690,14 @@ xfs_difree(
                         __func__, (unsigned long long)inode,
                         (unsigned long long)XFS_AGINO_TO_INO(mp, agno, agino));
                 ASSERT(0);
-                return XFS_ERROR(EINVAL);
+                return EINVAL;
         }
         agbno = XFS_AGINO_TO_AGBNO(mp, agino);
         if (agbno >= mp->m_sb.sb_agblocks)  {
                 xfs_warn(mp, "%s: agbno >= mp->m_sb.sb_agblocks (%d >= %d).",
                         __func__, agbno, mp->m_sb.sb_agblocks);
                 ASSERT(0);
-                return XFS_ERROR(EINVAL);
+                return EINVAL;
         }
         /*
          * Get the allocation group header.
@@ -1829,7 +1829,7 @@ xfs_imap(
                  * as they can be invalid without implying corruption.
                  */
                 if (flags & XFS_IGET_UNTRUSTED)
-                        return XFS_ERROR(EINVAL);
+                        return EINVAL;
                 if (agno >= mp->m_sb.sb_agcount) {
                         xfs_alert(mp,
                                 "%s: agno (%d) >= mp->m_sb.sb_agcount (%d)",
@@ -1849,7 +1849,7 @@ xfs_imap(
                 }
                 xfs_stack_trace();
 #endif /* DEBUG */
-                return XFS_ERROR(EINVAL);
+                return EINVAL;
         }
 
         blks_per_cluster = xfs_icluster_size_fsb(mp);
@@ -1922,7 +1922,7 @@ out_map:
                         __func__, (unsigned long long) imap->im_blkno,
                         (unsigned long long) imap->im_len,
                         XFS_FSB_TO_BB(mp, mp->m_sb.sb_dblocks));
-                return XFS_ERROR(EINVAL);
+                return EINVAL;
         }
         return 0;
 }