[XFS] xfs_iflush_fork() never returns an error.
xfs_iflush_fork() never returns an error. Mark it void and clean up the code calling it that checks for errors. SGI-PV: 980084 SGI-Modid: xfs-linux-melb:xfs-kern:30827a Signed-off-by: David Chinner <dgc@sgi.com> Signed-off-by: Niv Sardi <xaiki@sgi.com> Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
This commit is contained in:
parent
cc88466f3f
commit
e4ac967b11
|
@ -2883,7 +2883,7 @@ xfs_iextents_copy(
|
||||||
* format indicates the current state of the fork.
|
* format indicates the current state of the fork.
|
||||||
*/
|
*/
|
||||||
/*ARGSUSED*/
|
/*ARGSUSED*/
|
||||||
STATIC int
|
STATIC void
|
||||||
xfs_iflush_fork(
|
xfs_iflush_fork(
|
||||||
xfs_inode_t *ip,
|
xfs_inode_t *ip,
|
||||||
xfs_dinode_t *dip,
|
xfs_dinode_t *dip,
|
||||||
|
@ -2904,16 +2904,16 @@ xfs_iflush_fork(
|
||||||
static const short extflag[2] =
|
static const short extflag[2] =
|
||||||
{ XFS_ILOG_DEXT, XFS_ILOG_AEXT };
|
{ XFS_ILOG_DEXT, XFS_ILOG_AEXT };
|
||||||
|
|
||||||
if (iip == NULL)
|
if (!iip)
|
||||||
return 0;
|
return;
|
||||||
ifp = XFS_IFORK_PTR(ip, whichfork);
|
ifp = XFS_IFORK_PTR(ip, whichfork);
|
||||||
/*
|
/*
|
||||||
* This can happen if we gave up in iformat in an error path,
|
* This can happen if we gave up in iformat in an error path,
|
||||||
* for the attribute fork.
|
* for the attribute fork.
|
||||||
*/
|
*/
|
||||||
if (ifp == NULL) {
|
if (!ifp) {
|
||||||
ASSERT(whichfork == XFS_ATTR_FORK);
|
ASSERT(whichfork == XFS_ATTR_FORK);
|
||||||
return 0;
|
return;
|
||||||
}
|
}
|
||||||
cp = XFS_DFORK_PTR(dip, whichfork);
|
cp = XFS_DFORK_PTR(dip, whichfork);
|
||||||
mp = ip->i_mount;
|
mp = ip->i_mount;
|
||||||
|
@ -2974,8 +2974,6 @@ xfs_iflush_fork(
|
||||||
ASSERT(0);
|
ASSERT(0);
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
STATIC int
|
STATIC int
|
||||||
|
@ -3452,16 +3450,9 @@ xfs_iflush_int(
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (xfs_iflush_fork(ip, dip, iip, XFS_DATA_FORK, bp) == EFSCORRUPTED) {
|
xfs_iflush_fork(ip, dip, iip, XFS_DATA_FORK, bp);
|
||||||
goto corrupt_out;
|
if (XFS_IFORK_Q(ip))
|
||||||
}
|
xfs_iflush_fork(ip, dip, iip, XFS_ATTR_FORK, bp);
|
||||||
|
|
||||||
if (XFS_IFORK_Q(ip)) {
|
|
||||||
/*
|
|
||||||
* The only error from xfs_iflush_fork is on the data fork.
|
|
||||||
*/
|
|
||||||
(void) xfs_iflush_fork(ip, dip, iip, XFS_ATTR_FORK, bp);
|
|
||||||
}
|
|
||||||
xfs_inobp_check(mp, bp);
|
xfs_inobp_check(mp, bp);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
|
Loading…
Reference in New Issue