From: Dave Chinner <david@fromorbit.com>
To: fstests@vger.kernel.org
Subject: [PATCH] generic/038: speed up file creation
Date: Thu, 6 Aug 2015 10:27:28 +1000 [thread overview]
Message-ID: <1438820848-13325-1-git-send-email-david@fromorbit.com> (raw)
From: Dave Chinner <dchinner@redhat.com>
Now that generic/038 is running on my test machine, I notice how
slow it is:
generic/038 692s
11-12 minutes for a single test is way too long.
The test is creating
400,000 single block files, which can be easily parallelised and
hence run much faster than the test is currently doing.
Split the file creation up into 4 threads that create 100,000 files
each. 4 is chosen because XFS defaults to 4AGs, ext4 still has decent
speedups at 4 concurrent creates, and other filesystems aren't hurt
by excessive concurrency. The result:
generic/038 237s
on the same machine, which is roughly 3x faster and so it (just)
fast enough to to be considered acceptible.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
---
tests/generic/038 | 25 ++++++++++++++++++-------
1 file changed, 18 insertions(+), 7 deletions(-)
diff --git a/tests/generic/038 b/tests/generic/038
index 4d108cf..3c94a3b 100755
--- a/tests/generic/038
+++ b/tests/generic/038
@@ -105,19 +105,30 @@ trim_loop()
# the fallocate calls happen. So we don't really care if they all succeed or
# not, the goal is just to keep metadata space usage growing while data block
# groups are deleted.
+#
+# reating 400,000 files sequentially is really slow, so speed it up a bit
+# by doing it concurrently with 4 threads in 4 separate directories.
create_files()
{
local prefix=$1
- for ((i = 1; i <= 400000; i++)); do
- $XFS_IO_PROG -f -c "pwrite -S 0xaa 0 3900" \
- $SCRATCH_MNT/"${prefix}_$i" &> /dev/null
- if [ $? -ne 0 ]; then
- echo "Failed creating file ${prefix}_$i" >>$seqres.full
- break
- fi
+ for ((n = 0; n < 4; n++)); do
+ mkdir $SCRATCH_MNT/$n
+ (
+ for ((i = 1; i <= 100000; i++)); do
+ $XFS_IO_PROG -f -c "pwrite -S 0xaa 0 3900" \
+ $SCRATCH_MNT/$n/"${prefix}_$i" &> /dev/null
+ if [ $? -ne 0 ]; then
+ echo "Failed creating file $n/${prefix}_$i" >>$seqres.full
+ break
+ fi
+ done
+ ) &
+ create_pids[$n]=$!
done
+ wait ${create_pids[@]}
+
}
_scratch_mkfs >>$seqres.full 2>&1
--
2.1.4
next reply other threads:[~2015-08-06 0:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-06 0:27 Dave Chinner [this message]
2015-08-06 14:17 ` [PATCH] generic/038: speed up file creation Eryu Guan
2015-08-06 22:21 ` Dave Chinner
2015-08-07 8:09 ` Filipe David Manana
2015-08-07 23:22 ` Dave Chinner
2015-08-09 10:45 ` Eryu Guan
2015-08-09 23:20 ` Dave Chinner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1438820848-13325-1-git-send-email-david@fromorbit.com \
--to=david@fromorbit.com \
--cc=fstests@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox