xfs: kill xfs_read_buf()

xfs_read_buf() is effectively the same as xfs_trans_read_buf() when called
outside a transaction context. The error handling is slightly different in that
xfs_read_buf stales the errored buffer it gets back, but there is probably good
reason for xfs_trans_read_buf() for doing this.

Hence update xfs_trans_read_buf() to the same error handling as xfs_read_buf(),
and convert all the callers of xfs_read_buf() to use the former function. We can
then remove xfs_read_buf().

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Mark Tinguely <tinguely@sgi.com>
Signed-off-by: Ben Myers <bpm@sgi.com>

1 files changed, 0 insertions, 50 deletions

diff --git a/fs/xfs/xfs_rw.c b/fs/xfs/xfs_rw.c
index 2ce97758f4c7..3c2488a11165 100644
--- a/fs/xfs/xfs_rw.c
+++ b/fs/xfs/xfs_rw.c
@@ -92,56 +92,6 @@ xfs_do_force_shutdown(
 }
 
 /*
- * This isn't an absolute requirement, but it is
- * just a good idea to call xfs_read_buf instead of
- * directly doing a read_buf call. For one, we shouldn't
- * be doing this disk read if we are in SHUTDOWN state anyway,
- * so this stops that from happening. Secondly, this does all
- * the error checking stuff and the brelse if appropriate for
- * the caller, so the code can be a little leaner.
- */
-
-int
-xfs_read_buf(
-        struct xfs_mount *mp,
-        xfs_buftarg_t    *target,
-        xfs_daddr_t      blkno,
-        int              len,
-        uint             flags,
-        xfs_buf_t        **bpp)
-{
-        xfs_buf_t        *bp;
-        int              error;
-
-        if (!flags)
-                flags = XBF_MAPPED;
-
-        bp = xfs_buf_read(target, blkno, len, flags);
-        if (!bp)
-                return XFS_ERROR(EIO);
-        error = bp->b_error;
-        if (!error && !XFS_FORCED_SHUTDOWN(mp)) {
-                *bpp = bp;
-        } else {
-                *bpp = NULL;
-                if (error) {
-                        xfs_buf_ioerror_alert(bp, __func__);
-                } else {
-                        error = XFS_ERROR(EIO);
-                }
-                if (bp) {
-                        XFS_BUF_UNDONE(bp);
-                        xfs_buf_stale(bp);
-                        /*
-                         * brelse clears B_ERROR and b_error
-                         */
-                        xfs_buf_relse(bp);
-                }
-        }
-        return (error);
-}
-
-/*
  * helper function to extract extent size hint from inode
  */
 xfs_extlen_t