summaryrefslogtreecommitdiffstats
path: root/tools/perf
diff options
context:
space:
mode:
authorDave Chinner <david@fromorbit.com>2010-01-10 23:51:47 +0000
committerAlex Elder <aelder@sgi.com>2010-01-15 13:44:44 -0600
commit57817c68229984818fea9e614d6f95249c3fb098 (patch)
tree1c3265ae92ccf51617763a568c4c76be3a596578 /tools/perf
parent018027be90a6946e8cf3f9b17b5582384f7ed117 (diff)
downloadblackbird-op-linux-57817c68229984818fea9e614d6f95249c3fb098.tar.gz
blackbird-op-linux-57817c68229984818fea9e614d6f95249c3fb098.zip
xfs: reclaim all inodes by background tree walks
We cannot do direct inode reclaim without taking the flush lock to ensure that we do not reclaim an inode under IO. We check the inode is clean before doing direct reclaim, but this is not good enough because the inode flush code marks the inode clean once it has copied the in-core dirty state to the backing buffer. It is the flush lock that determines whether the inode is still under IO, even though it is marked clean, and the inode is still required at IO completion so we can't reclaim it even though it is clean in core. Hence the requirement that we need to take the flush lock even on clean inodes because this guarantees that the inode writeback IO has completed and it is safe to reclaim the inode. With delayed write inode flushing, we coul dend up waiting a long time on the flush lock even for a clean inode. The background reclaim already handles this efficiently, so avoid all the problems by killing the direct reclaim path altogether. 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 'tools/perf')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud