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/quota | |
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/quota')
-rw-r--r-- | fs/xfs/quota/xfs_dquot.c | 3 | ||||
-rw-r--r-- | fs/xfs/quota/xfs_qm.c | 5 | ||||
-rw-r--r-- | fs/xfs/quota/xfs_qm_syscalls.c | 6 |
3 files changed, 6 insertions, 8 deletions
diff --git a/fs/xfs/quota/xfs_dquot.c b/fs/xfs/quota/xfs_dquot.c index 4adaf13aac6f..cfdd35ee9f7a 100644 --- a/fs/xfs/quota/xfs_dquot.c +++ b/fs/xfs/quota/xfs_dquot.c | |||
@@ -753,8 +753,7 @@ xfs_qm_idtodq( | |||
753 | goto error0; | 753 | goto error0; |
754 | } | 754 | } |
755 | if (tp) { | 755 | if (tp) { |
756 | if ((error = xfs_trans_commit(tp, XFS_TRANS_RELEASE_LOG_RES, | 756 | if ((error = xfs_trans_commit(tp, XFS_TRANS_RELEASE_LOG_RES))) |
757 | NULL))) | ||
758 | goto error1; | 757 | goto error1; |
759 | } | 758 | } |
760 | 759 | ||
diff --git a/fs/xfs/quota/xfs_qm.c b/fs/xfs/quota/xfs_qm.c index 1de2acdc7f70..762af2d1b50f 100644 --- a/fs/xfs/quota/xfs_qm.c +++ b/fs/xfs/quota/xfs_qm.c | |||
@@ -1453,8 +1453,7 @@ xfs_qm_qino_alloc( | |||
1453 | XFS_SB_UNLOCK(mp, s); | 1453 | XFS_SB_UNLOCK(mp, s); |
1454 | xfs_mod_sb(tp, sbfields); | 1454 | xfs_mod_sb(tp, sbfields); |
1455 | 1455 | ||
1456 | if ((error = xfs_trans_commit(tp, XFS_TRANS_RELEASE_LOG_RES, | 1456 | if ((error = xfs_trans_commit(tp, XFS_TRANS_RELEASE_LOG_RES))) { |
1457 | NULL))) { | ||
1458 | xfs_fs_cmn_err(CE_ALERT, mp, "XFS qino_alloc failed!"); | 1457 | xfs_fs_cmn_err(CE_ALERT, mp, "XFS qino_alloc failed!"); |
1459 | return error; | 1458 | return error; |
1460 | } | 1459 | } |
@@ -2405,7 +2404,7 @@ xfs_qm_write_sb_changes( | |||
2405 | } | 2404 | } |
2406 | 2405 | ||
2407 | xfs_mod_sb(tp, flags); | 2406 | xfs_mod_sb(tp, flags); |
2408 | (void) xfs_trans_commit(tp, 0, NULL); | 2407 | (void) xfs_trans_commit(tp, 0); |
2409 | 2408 | ||
2410 | return 0; | 2409 | return 0; |
2411 | } | 2410 | } |
diff --git a/fs/xfs/quota/xfs_qm_syscalls.c b/fs/xfs/quota/xfs_qm_syscalls.c index 716f562aa8b2..44ab68de971d 100644 --- a/fs/xfs/quota/xfs_qm_syscalls.c +++ b/fs/xfs/quota/xfs_qm_syscalls.c | |||
@@ -735,7 +735,7 @@ xfs_qm_scall_setqlim( | |||
735 | xfs_trans_log_dquot(tp, dqp); | 735 | xfs_trans_log_dquot(tp, dqp); |
736 | 736 | ||
737 | xfs_dqtrace_entry(dqp, "Q_SETQLIM: COMMIT"); | 737 | xfs_dqtrace_entry(dqp, "Q_SETQLIM: COMMIT"); |
738 | xfs_trans_commit(tp, 0, NULL); | 738 | xfs_trans_commit(tp, 0); |
739 | xfs_qm_dqprint(dqp); | 739 | xfs_qm_dqprint(dqp); |
740 | xfs_qm_dqrele(dqp); | 740 | xfs_qm_dqrele(dqp); |
741 | mutex_unlock(&(XFS_QI_QOFFLOCK(mp))); | 741 | mutex_unlock(&(XFS_QI_QOFFLOCK(mp))); |
@@ -809,7 +809,7 @@ xfs_qm_log_quotaoff_end( | |||
809 | * We don't care about quotoff's performance. | 809 | * We don't care about quotoff's performance. |
810 | */ | 810 | */ |
811 | xfs_trans_set_sync(tp); | 811 | xfs_trans_set_sync(tp); |
812 | error = xfs_trans_commit(tp, 0, NULL); | 812 | error = xfs_trans_commit(tp, 0); |
813 | return (error); | 813 | return (error); |
814 | } | 814 | } |
815 | 815 | ||
@@ -852,7 +852,7 @@ xfs_qm_log_quotaoff( | |||
852 | * We don't care about quotoff's performance. | 852 | * We don't care about quotoff's performance. |
853 | */ | 853 | */ |
854 | xfs_trans_set_sync(tp); | 854 | xfs_trans_set_sync(tp); |
855 | error = xfs_trans_commit(tp, 0, NULL); | 855 | error = xfs_trans_commit(tp, 0); |
856 | 856 | ||
857 | error0: | 857 | error0: |
858 | if (error) { | 858 | if (error) { |