Skip to content

Commit

Permalink
xfs: replace xfs_buf_geterror() with bp->b_error
Browse files Browse the repository at this point in the history
Since we just checked bp for NULL, it is ok to replace
xfs_buf_geterror() with bp->b_error in these places.

Signed-off-by: Chandra Seetharaman <sekharan@us.ibm.com>
Signed-off-by: Alex Elder <aelder@sgi.com>
  • Loading branch information
Chandra Seetharaman authored and Alex Elder committed Aug 12, 2011
1 parent ac4d688 commit e570280
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions fs/xfs/xfs_log_recover.c
Original file line number Diff line number Diff line change
Expand Up @@ -2133,7 +2133,7 @@ xlog_recover_buffer_pass2(
buf_flags);
if (!bp)
return XFS_ERROR(ENOMEM);
error = xfs_buf_geterror(bp);
error = bp->b_error;
if (error) {
xfs_ioerror_alert("xlog_recover_do..(read#1)", mp,
bp, buf_f->blf_blkno);
Expand Down Expand Up @@ -2228,7 +2228,7 @@ xlog_recover_inode_pass2(
error = ENOMEM;
goto error;
}
error = xfs_buf_geterror(bp);
error = bp->b_error;
if (error) {
xfs_ioerror_alert("xlog_recover_do..(read#2)", mp,
bp, in_f->ilf_blkno);
Expand Down
2 changes: 1 addition & 1 deletion fs/xfs/xfs_vnodeops.c
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ xfs_readlink_bmap(
XBF_LOCK | XBF_MAPPED | XBF_DONT_BLOCK);
if (!bp)
return XFS_ERROR(ENOMEM);
error = xfs_buf_geterror(bp);
error = bp->b_error;
if (error) {
xfs_ioerror_alert("xfs_readlink",
ip->i_mount, bp, XFS_BUF_ADDR(bp));
Expand Down

0 comments on commit e570280

Please sign in to comment.