Skip to content

Commit

Permalink
xfs: fold __xfs_trans_roll into xfs_trans_roll
Browse files Browse the repository at this point in the history
No one cares about the low-level helper anymore.

Signed-off-by: Christoph Hellwig <[email protected]>
Reviewed-by: Eric Sandeen <[email protected]>
Reviewed-by: Darrick J. Wong <[email protected]>
Signed-off-by: Darrick J. Wong <[email protected]>
  • Loading branch information
Christoph Hellwig authored and djwong committed Apr 6, 2017
1 parent 4c934c7 commit 254133f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 16 deletions.
17 changes: 2 additions & 15 deletions fs/xfs/xfs_trans.c
Original file line number Diff line number Diff line change
Expand Up @@ -1034,17 +1034,14 @@ xfs_trans_cancel(
* chunk we've been working on and get a new transaction to continue.
*/
int
__xfs_trans_roll(
xfs_trans_roll(
struct xfs_trans **tpp,
struct xfs_inode *dp,
int *committed)
struct xfs_inode *dp)
{
struct xfs_trans *trans;
struct xfs_trans_res tres;
int error;

*committed = 0;

/*
* Ensure that the inode is always logged.
*/
Expand All @@ -1070,7 +1067,6 @@ __xfs_trans_roll(
if (error)
return error;

*committed = 1;
trans = *tpp;

/*
Expand All @@ -1093,12 +1089,3 @@ __xfs_trans_roll(
xfs_trans_ijoin(trans, dp, 0);
return 0;
}

int
xfs_trans_roll(
struct xfs_trans **tpp,
struct xfs_inode *dp)
{
int committed;
return __xfs_trans_roll(tpp, dp, &committed);
}
1 change: 0 additions & 1 deletion fs/xfs/xfs_trans.h
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,6 @@ int xfs_trans_free_extent(struct xfs_trans *,
struct xfs_efd_log_item *, xfs_fsblock_t,
xfs_extlen_t, struct xfs_owner_info *);
int xfs_trans_commit(struct xfs_trans *);
int __xfs_trans_roll(struct xfs_trans **, struct xfs_inode *, int *);
int xfs_trans_roll(struct xfs_trans **, struct xfs_inode *);
void xfs_trans_cancel(xfs_trans_t *);
int xfs_trans_ail_init(struct xfs_mount *);
Expand Down

0 comments on commit 254133f

Please sign in to comment.