diff options
author | David Chinner <david@fromorbit.com> | 2008-10-30 02:39:58 -0400 |
---|---|---|
committer | Lachlan McIlroy <lachlan@sgi.com> | 2008-10-30 02:39:58 -0400 |
commit | 783a2f656f9674c31d4019708a94af93fa1d1c22 (patch) | |
tree | 7ac2123b187c0522dbd8a0d94de3856b33147c46 /fs/xfs/xfs_trans.c | |
parent | fc1829f34d30899701dfd5890030d39e13e1f47d (diff) |
[XFS] Finish removing the mount pointer from the AIL API
Change all the remaining AIL API functions that are passed struct
xfs_mount pointers to pass pointers directly to the struct xfs_ail being
used. With this conversion, all external access to the AIL is via the
struct xfs_ail. Hence the operation and referencing of the AIL is almost
entirely independent of the xfs_mount that is using it - it is now much
more tightly tied to the log and the items it is tracking in the log than
it is tied to the xfs_mount.
SGI-PV: 988143
SGI-Modid: xfs-linux-melb:xfs-kern:32353a
Signed-off-by: David Chinner <david@fromorbit.com>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Signed-off-by: Christoph Hellwig <hch@infradead.org>
Diffstat (limited to 'fs/xfs/xfs_trans.c')
-rw-r--r-- | fs/xfs/xfs_trans.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/xfs/xfs_trans.c b/fs/xfs/xfs_trans.c index 5163e1216c8e..ad137efc8702 100644 --- a/fs/xfs/xfs_trans.c +++ b/fs/xfs/xfs_trans.c | |||
@@ -1387,7 +1387,6 @@ xfs_trans_chunk_committed( | |||
1387 | 1387 | ||
1388 | lidp = licp->lic_descs; | 1388 | lidp = licp->lic_descs; |
1389 | for (i = 0; i < licp->lic_unused; i++, lidp++) { | 1389 | for (i = 0; i < licp->lic_unused; i++, lidp++) { |
1390 | struct xfs_mount *mp; | ||
1391 | struct xfs_ail *ailp; | 1390 | struct xfs_ail *ailp; |
1392 | 1391 | ||
1393 | if (xfs_lic_isfree(licp, i)) { | 1392 | if (xfs_lic_isfree(licp, i)) { |
@@ -1426,7 +1425,6 @@ xfs_trans_chunk_committed( | |||
1426 | * This would cause the earlier transaction to fail | 1425 | * This would cause the earlier transaction to fail |
1427 | * the test below. | 1426 | * the test below. |
1428 | */ | 1427 | */ |
1429 | mp = lip->li_mountp; | ||
1430 | ailp = lip->li_ailp; | 1428 | ailp = lip->li_ailp; |
1431 | spin_lock(&ailp->xa_lock); | 1429 | spin_lock(&ailp->xa_lock); |
1432 | if (XFS_LSN_CMP(item_lsn, lip->li_lsn) > 0) { | 1430 | if (XFS_LSN_CMP(item_lsn, lip->li_lsn) > 0) { |
@@ -1435,9 +1433,9 @@ xfs_trans_chunk_committed( | |||
1435 | * and update the position of the item in | 1433 | * and update the position of the item in |
1436 | * the AIL. | 1434 | * the AIL. |
1437 | * | 1435 | * |
1438 | * xfs_trans_update_ail() drops the AIL lock. | 1436 | * xfs_trans_ail_update() drops the AIL lock. |
1439 | */ | 1437 | */ |
1440 | xfs_trans_update_ail(mp, lip, item_lsn); | 1438 | xfs_trans_ail_update(ailp, lip, item_lsn); |
1441 | } else { | 1439 | } else { |
1442 | spin_unlock(&ailp->xa_lock); | 1440 | spin_unlock(&ailp->xa_lock); |
1443 | } | 1441 | } |