summaryrefslogtreecommitdiff
path: root/.clang-format
diff options
context:
space:
mode:
authorOmar Sandoval <osandov@fb.com>2018-12-12 19:46:32 +0300
committerDarrick J. Wong <darrick.wong@oracle.com>2018-12-12 19:47:17 +0300
commit355e3532132b487ebf6a4900fad8f3525fa3e137 (patch)
tree97b63d782ed5e36a9a0e953a0ceb7d1fca30f15b /.clang-format
parent2c2d9d3a205afa93bf6105e4ab6f1ff536291dc6 (diff)
downloadlinux-355e3532132b487ebf6a4900fad8f3525fa3e137.tar.xz
xfs: cache minimum realtime summary level
The realtime summary is a two-dimensional array on disk, effectively: u32 rsum[log2(number of realtime extents) + 1][number of blocks in the bitmap] rsum[log][bbno] is the number of extents of size 2**log which start in bitmap block bbno. xfs_rtallocate_extent_near() uses xfs_rtany_summary() to check whether rsum[log][bbno] != 0 for any log level. However, the summary array is stored in row-major order (i.e., like an array in C), so all of these entries are not adjacent, but rather spread across the entire summary file. In the worst case (a full bitmap block), xfs_rtany_summary() has to check every level. This means that on a moderately-used realtime device, an allocation will waste a lot of time finding, reading, and releasing buffers for the realtime summary. In particular, one of our storage services (which runs on servers with 8 very slow CPUs and 15 8 TB XFS realtime filesystems) spends almost 5% of its CPU cycles in xfs_rtbuf_get() and xfs_trans_brelse() called from xfs_rtany_summary(). One solution would be to also store the summary with the dimensions swapped. However, this would require a disk format change to a very old component of XFS. Instead, we can cache the minimum size which contains any extents. We do so lazily; rather than guaranteeing that the cache contains the precise minimum, it always contains a loose lower bound which we tighten when we read or update a summary block. This only uses a few kilobytes of memory and is already serialized via the realtime bitmap and summary inode locks, so the cost is minimal. With this change, the same workload only spends 0.2% of its CPU cycles in the realtime allocator. Signed-off-by: Omar Sandoval <osandov@fb.com> Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to '.clang-format')
0 files changed, 0 insertions, 0 deletions