diff options
| author | Dave Chinner <david@fromorbit.com> | 2010-01-11 11:47:45 +0000 |
|---|---|---|
| committer | Alex Elder <aelder@sgi.com> | 2010-01-15 15:34:04 -0600 |
| commit | aed3bb90abaf0b42e8c8747e192f7bb97f445279 (patch) | |
| tree | 4619214ed42821792b05f4417cb95bde90abb562 /fs/xfs/xfs_mount.c | |
| parent | 1c1c6ebcf5284aee4910f3b906ac90c20e510c82 (diff) | |
| download | op-kernel-dev-aed3bb90abaf0b42e8c8747e192f7bb97f445279.zip op-kernel-dev-aed3bb90abaf0b42e8c8747e192f7bb97f445279.tar.gz | |
xfs: Reference count per-ag structures
Reference count the per-ag structures to ensure that we keep get/put
pairs balanced. Assert that the reference counts are zero at unmount
time to catch leaks. In future, reference counts will enable us to
safely remove perag structures by allowing us to detect when they
are no longer in use.
Signed-off-by: Dave Chinner <david@fromorbit.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Alex Elder <aelder@sgi.com>
Diffstat (limited to 'fs/xfs/xfs_mount.c')
| -rw-r--r-- | fs/xfs/xfs_mount.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c index c04dd83..f241fec 100644 --- a/fs/xfs/xfs_mount.c +++ b/fs/xfs/xfs_mount.c @@ -215,6 +215,7 @@ xfs_free_perag( for (agno = 0; agno < mp->m_sb.sb_agcount; agno++) { spin_lock(&mp->m_perag_lock); pag = radix_tree_delete(&mp->m_perag_tree, agno); + ASSERT(atomic_read(&pag->pag_ref) == 0); spin_unlock(&mp->m_perag_lock); ASSERT(pag); kmem_free(pag->pagb_list); |
