generic/015: Issue sync after deleting the fillup file
diff mbox

Message ID 1521038917-20863-1-git-send-email-nborisov@suse.com
State New
Headers show

Commit Message

Nikolay Borisov March 14, 2018, 2:48 p.m. UTC
This test fails on btrfs due to the presence of delayed processing
of file deletes if the file is smaller than 32mb. Initially commit
97575acd7495b412435d06229a6d94ed9a814ada tried to fix a similar
failure by bumping the size of the filesystem. However that change
had a knock-on effect in that the scratch filesystem created is
larger than 100mb and thus not created in mixed mode. This in turn
causes the fs to have only 20mb for file data (rest is taken by DUP
metadata). Naturally, this leads to file freeing taking up to
"transaction commit interval" (default 30 s) time to properly account
the freed space.

Not standards define when unlink operations should be accounted so
btrfs is well within its right to be implemented in that way. So
to avoid this edge case just issue a sync before taking the 2nd
free space reading.

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
---
 tests/generic/015 | 2 ++
 1 file changed, 2 insertions(+)

Patch
diff mbox

diff --git a/tests/generic/015 b/tests/generic/015
index bdae86dd..cb7252b9 100755
--- a/tests/generic/015
+++ b/tests/generic/015
@@ -115,6 +115,8 @@  fi
 
 echo "check free space:"
 
+sync
+
 free2=`_free`
 if [ -z "$free2" ]
 then