From: Darrick J. Wong <djwong@xxxxxxxxxx>
The new rtgroups feature implements a simplistic rotor to pick the
rtgroup for an initial allocation to a file. This causes test failures
if the preallocations are spread across two rtgroups, which happens if
there are more subtests than rtgroups.
One way to fix this would be to reset the rotor then each subtest starts
allocating from rtgroup 0, but the only way to do that is to cycle the
scratch mount, which is a bit gross.
Instead, report logically contiguous mappings as a single mapping even
if the physical space is not contiguous. Unfortunately, there's not
enough context in the comments to know if the test actually was checking
for physical contiguity? Or if this is just an exerciser of the old
preallocation calls, and it's fine as long as the file ranges are mapped
(or unmapped) as desired.
Messing with some awk is a lot cheaper than umount/mount cycling.
Signed-off-by: "Darrick J. Wong" <djwong@xxxxxxxxxx>
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
---
tests/xfs/009 | 29 ++++++++++++++++++++++++++++-
1 file changed, 28 insertions(+), 1 deletion(-)
diff --git a/tests/xfs/009 b/tests/xfs/009
index dde505f079f4f8..bb42ce32490df5 100755
--- a/tests/xfs/009
+++ b/tests/xfs/009
@@ -49,13 +49,26 @@ _filesize()
_block_filter()
{
$AWK_PROG -v bsize="$bsize" '
+ BEGIN {
+ br_pos = 0
+ br_len = 0
+ }
+ function dump_blockrange() {
+ if (br_len == 0)
+ return
+ printf(" [%d,%d]: BLOCKRANGE\n", br_pos, br_len)
+ br_pos = 0
+ br_len = 0
+ }
/blocksize/ {
+ dump_blockrange()
printf(" blocksize BSIZE\n")
next
}
/CMD/ {
+ dump_blockrange()
split($3, off, "=")
offset = strtonum(off[2])
if (offset != -1)
@@ -72,6 +85,7 @@ _block_filter()
}
/MAP/ {
+ dump_blockrange()
split($2, off, "=")
offset = strtonum(off[2])
if (offset != -1)
@@ -90,6 +104,7 @@ _block_filter()
}
/TRUNCATE/ {
+ dump_blockrange()
split($2, off, "=")
offset = strtonum(off[2]) / bsize
@@ -99,16 +114,28 @@ _block_filter()
}
/\[[0-9]+,[0-9]+\]:/ {
- printf(" %s BLOCKRANGE\n", $1)
+ rangestr = gensub(/\[([0-9]+),([0-9]+)\]:/, "\\1,\\2", "g", $1);
+ split(rangestr, off, ",")
+ if (br_pos + br_len == off[1]) {
+ br_len += off[2];
+ } else {
+ dump_blockrange()
+ br_pos = off[1];
+ br_len = off[2];
+ }
next
}
{
+ dump_blockrange()
print
next
}
+ END {
+ dump_blockrange()
+ }
'
}
