diff options
author | Eric Sandeen <sandeen@sandeen.net> | 2007-05-07 23:48:42 -0400 |
---|---|---|
committer | Tim Shimmin <tes@sgi.com> | 2007-05-07 23:48:42 -0400 |
commit | 1c72bf90037f32fc2b10e0a05dff2640abce8ee2 (patch) | |
tree | 3903e287eed6b6e942340d01bec360edebff594e /fs/xfs/xfs_attr.c | |
parent | 5b94f675f57e4ff16c8fda09088d7480a84dcd91 (diff) |
[XFS] The last argument "lsn" of xfs_trans_commit() is always called with
NULL.
Patch provided by Eric Sandeen.
SGI-PV: 961693
SGI-Modid: xfs-linux-melb:xfs-kern:28199a
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Signed-off-by: Tim Shimmin <tes@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_attr.c')
-rw-r--r-- | fs/xfs/xfs_attr.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/fs/xfs/xfs_attr.c b/fs/xfs/xfs_attr.c index 9d358ffce4e5..7ce44a7b88a2 100644 --- a/fs/xfs/xfs_attr.c +++ b/fs/xfs/xfs_attr.c | |||
@@ -328,8 +328,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen, | |||
328 | xfs_trans_set_sync(args.trans); | 328 | xfs_trans_set_sync(args.trans); |
329 | } | 329 | } |
330 | err2 = xfs_trans_commit(args.trans, | 330 | err2 = xfs_trans_commit(args.trans, |
331 | XFS_TRANS_RELEASE_LOG_RES, | 331 | XFS_TRANS_RELEASE_LOG_RES); |
332 | NULL); | ||
333 | xfs_iunlock(dp, XFS_ILOCK_EXCL); | 332 | xfs_iunlock(dp, XFS_ILOCK_EXCL); |
334 | 333 | ||
335 | /* | 334 | /* |
@@ -397,8 +396,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen, | |||
397 | * Commit the last in the sequence of transactions. | 396 | * Commit the last in the sequence of transactions. |
398 | */ | 397 | */ |
399 | xfs_trans_log_inode(args.trans, dp, XFS_ILOG_CORE); | 398 | xfs_trans_log_inode(args.trans, dp, XFS_ILOG_CORE); |
400 | error = xfs_trans_commit(args.trans, XFS_TRANS_RELEASE_LOG_RES, | 399 | error = xfs_trans_commit(args.trans, XFS_TRANS_RELEASE_LOG_RES); |
401 | NULL); | ||
402 | xfs_iunlock(dp, XFS_ILOCK_EXCL); | 400 | xfs_iunlock(dp, XFS_ILOCK_EXCL); |
403 | 401 | ||
404 | /* | 402 | /* |
@@ -544,8 +542,7 @@ xfs_attr_remove_int(xfs_inode_t *dp, const char *name, int namelen, int flags) | |||
544 | * Commit the last in the sequence of transactions. | 542 | * Commit the last in the sequence of transactions. |
545 | */ | 543 | */ |
546 | xfs_trans_log_inode(args.trans, dp, XFS_ILOG_CORE); | 544 | xfs_trans_log_inode(args.trans, dp, XFS_ILOG_CORE); |
547 | error = xfs_trans_commit(args.trans, XFS_TRANS_RELEASE_LOG_RES, | 545 | error = xfs_trans_commit(args.trans, XFS_TRANS_RELEASE_LOG_RES); |
548 | NULL); | ||
549 | xfs_iunlock(dp, XFS_ILOCK_EXCL); | 546 | xfs_iunlock(dp, XFS_ILOCK_EXCL); |
550 | 547 | ||
551 | /* | 548 | /* |
@@ -859,8 +856,7 @@ xfs_attr_inactive(xfs_inode_t *dp) | |||
859 | * Commit the last in the sequence of transactions. | 856 | * Commit the last in the sequence of transactions. |
860 | */ | 857 | */ |
861 | xfs_trans_log_inode(trans, dp, XFS_ILOG_CORE); | 858 | xfs_trans_log_inode(trans, dp, XFS_ILOG_CORE); |
862 | error = xfs_trans_commit(trans, XFS_TRANS_RELEASE_LOG_RES, | 859 | error = xfs_trans_commit(trans, XFS_TRANS_RELEASE_LOG_RES); |
863 | NULL); | ||
864 | xfs_iunlock(dp, XFS_ILOCK_EXCL); | 860 | xfs_iunlock(dp, XFS_ILOCK_EXCL); |
865 | 861 | ||
866 | return(error); | 862 | return(error); |