* [PATCH V2] generic: require filesize to be greater than fs block size in generic/240
@ 2013-12-11 11:21 Stanislav Kholmanskikh
2013-12-11 11:28 ` Christoph Hellwig
2013-12-11 15:11 ` Rich Johnston
0 siblings, 2 replies; 3+ messages in thread
From: Stanislav Kholmanskikh @ 2013-12-11 11:21 UTC (permalink / raw)
To: xfs; +Cc: vasily.isaenko, hch
If we execute generic/240 on a fs which has its fs block size greater
than 64k (for example, NFS), this test will fail with:
io_submit failed: Invalid argument
This will happen because in src/aio-dio-regress/aiodio_sparse2.c this
expression
num_aio = filesize / step;
will set num_aio to 0 and this means that no io_prep_write() will happen
before calling io_submit().
Fixing filesize to be 8 * "fs block size".
Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
---
tests/generic/240 | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/tests/generic/240 b/tests/generic/240
index e26f495..74dbba6 100755
--- a/tests/generic/240
+++ b/tests/generic/240
@@ -62,14 +62,15 @@ rm -f $TEST_DIR/aiodio_sparse
logical_block_size=`_min_dio_alignment $TEST_DEV`
fs_block_size=`stat -f $TEST_DIR | grep "Block size:" | awk '{print $3}'`
+file_size=$((8 * $fs_block_size))
if [ $fs_block_size -le $logical_block_size ]; then
_notrun "fs block size must be larger than the device block size. fs block size: $fs_block_size, device block size: $logical_block_size"
fi
-# 2 threads, fs block sized writes, 64k filesize, stride through file by
+# 2 threads, fs block sized writes, filesize, stride through file by
# fs block size, start at logical block size offset
-$AIO_TEST -i 2 -w $fs_block_size -s 64k -n $fs_block_size -o $logical_block_size "$TEST_DIR/aiodio_sparse"
+$AIO_TEST -i 2 -w $fs_block_size -s $file_size -n $fs_block_size -o $logical_block_size "$TEST_DIR/aiodio_sparse"
status=$?
exit
--
1.7.1
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH V2] generic: require filesize to be greater than fs block size in generic/240
2013-12-11 11:21 [PATCH V2] generic: require filesize to be greater than fs block size in generic/240 Stanislav Kholmanskikh
@ 2013-12-11 11:28 ` Christoph Hellwig
2013-12-11 15:11 ` Rich Johnston
1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2013-12-11 11:28 UTC (permalink / raw)
To: Stanislav Kholmanskikh; +Cc: vasily.isaenko, hch, xfs
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V2] generic: require filesize to be greater than fs block size in generic/240
2013-12-11 11:21 [PATCH V2] generic: require filesize to be greater than fs block size in generic/240 Stanislav Kholmanskikh
2013-12-11 11:28 ` Christoph Hellwig
@ 2013-12-11 15:11 ` Rich Johnston
1 sibling, 0 replies; 3+ messages in thread
From: Rich Johnston @ 2013-12-11 15:11 UTC (permalink / raw)
To: Stanislav Kholmanskikh, xfs; +Cc: vasily.isaenko, hch
This has been committed.
Thanks
--Rich
commit 43fb49332d0f810c843dc01327b3ca203e997ae7
Author: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
Date: Wed Dec 11 11:21:28 2013 +0000
generic: require filesize to be greater than fs block size in
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-12-11 15:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-11 11:21 [PATCH V2] generic: require filesize to be greater than fs block size in generic/240 Stanislav Kholmanskikh
2013-12-11 11:28 ` Christoph Hellwig
2013-12-11 15:11 ` Rich Johnston
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox