* [PATCH] generic: detect preallocation support for fsx tests 075 and 112 @ 2019-02-05 18:47 Christoph Hellwig 2019-02-10 11:11 ` Eryu Guan 0 siblings, 1 reply; 5+ messages in thread From: Christoph Hellwig @ 2019-02-05 18:47 UTC (permalink / raw) To: fstests Currently generic/075 and generic/112 have two extra fsx passes each that exercise fsx with preallocation, which are only enabled for XFS. These tests can also be run with other file systems, given that the XFS prealloc ioctls are implemented in generic code since the addition of the fallocate system call. This also means a version of XFS that does not support preallocation (e.g. because it always writes out of place) can skip the prealloc tests while still completing the normal fsx tests just fine. Signed-off-by: Christoph Hellwig <hch@lst.de> --- tests/generic/075 | 16 +++++++++++----- tests/generic/112 | 20 +++++++++++++------- 2 files changed, 24 insertions(+), 12 deletions(-) diff --git a/tests/generic/075 b/tests/generic/075 index 2b957891..8981c560 100755 --- a/tests/generic/075 +++ b/tests/generic/075 @@ -48,11 +48,12 @@ _do_test() echo "fsx.$_n : $_filter_param" echo "-----------------------------------------------" - if [ "$FSTYP" != "xfs" ] - then - if [ "$_n" = "1" -o "$_n" = "3" ] - then - # HACK: only xfs handles preallocation (-x) so just skip this test + if [ "$_n" = "1" -o "$_n" = "3" ]; then + if echo $testio | grep -q "Operation not supported"; then + # + # Skip the prealloc runs if the file system does not support + # preallocation + # return fi fi @@ -128,6 +129,11 @@ echo "brevity is wit..." _check_test_fs +# check if preallocation is supported +testfile=$TEST_DIR/$$.xfs_io +testio=`$XFS_IO_PROG -F -f -c "falloc 0 1m" $testfile 2>&1` +rm -f $testfile 2>&1 > /dev/null + # Options: # -d: debug output for all operations # -l flen: the upper bound on file size (default 262144) diff --git a/tests/generic/112 b/tests/generic/112 index 1879e7b5..a6f8fc14 100755 --- a/tests/generic/112 +++ b/tests/generic/112 @@ -48,13 +48,14 @@ _do_test() echo "fsx.$_n : $_filter_param" echo "-----------------------------------------------" - if [ "$FSTYP" != "xfs" ] - then - if [ "$_n" = "1" -o "$_n" = "3" ] - then - # HACK: only xfs handles preallocation (-x) so just skip this test - return - fi + if [ "$_n" = "1" -o "$_n" = "3" ]; then + if echo $testio | grep -q "Operation not supported"; then + # + # Skip the prealloc runs if the file system does not support + # preallocation + # + return + fi fi # This cd and use of -P gets full debug on $here (not TEST_DEV) @@ -128,6 +129,11 @@ echo "brevity is wit..." _check_test_fs +# check if preallocation is supported +testfile=$TEST_DIR/$$.xfs_io +testio=`$XFS_IO_PROG -F -f -c "falloc 0 1m" $testfile 2>&1` +rm -f $testfile 2>&1 > /dev/null + # Options: # -d: debug output for all operations # -l flen: the upper bound on file size (default 262144) -- 2.20.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] generic: detect preallocation support for fsx tests 075 and 112 2019-02-05 18:47 [PATCH] generic: detect preallocation support for fsx tests 075 and 112 Christoph Hellwig @ 2019-02-10 11:11 ` Eryu Guan 2019-02-11 0:37 ` Dave Chinner 0 siblings, 1 reply; 5+ messages in thread From: Eryu Guan @ 2019-02-10 11:11 UTC (permalink / raw) To: Christoph Hellwig; +Cc: fstests On Tue, Feb 05, 2019 at 07:47:13PM +0100, Christoph Hellwig wrote: > Currently generic/075 and generic/112 have two extra fsx passes each that > exercise fsx with preallocation, which are only enabled for XFS. > > These tests can also be run with other file systems, given that the XFS > prealloc ioctls are implemented in generic code since the addition of > the fallocate system call. This also means a version of XFS that does > not support preallocation (e.g. because it always writes out of place) > can skip the prealloc tests while still completing the normal fsx tests > just fine. > > Signed-off-by: Christoph Hellwig <hch@lst.de> > --- > tests/generic/075 | 16 +++++++++++----- > tests/generic/112 | 20 +++++++++++++------- > 2 files changed, 24 insertions(+), 12 deletions(-) > > diff --git a/tests/generic/075 b/tests/generic/075 > index 2b957891..8981c560 100755 > --- a/tests/generic/075 > +++ b/tests/generic/075 > @@ -48,11 +48,12 @@ _do_test() > echo "fsx.$_n : $_filter_param" > echo "-----------------------------------------------" > > - if [ "$FSTYP" != "xfs" ] > - then > - if [ "$_n" = "1" -o "$_n" = "3" ] > - then > - # HACK: only xfs handles preallocation (-x) so just skip this test > + if [ "$_n" = "1" -o "$_n" = "3" ]; then > + if echo $testio | grep -q "Operation not supported"; then Hmm, I don't think checking ENOSUPP message is sufficient, old distros may ship xfs_io without falloc support or there's no generic prealloc ioctl support, these cases are rare but possible. How about adding a new XFS_IOC_RESVSP ioctl check in src/feature.c and skip the fsx's "-x" option if feature reports that the ioctl is not supported by the fs? Thanks, Eryu > + # > + # Skip the prealloc runs if the file system does not support > + # preallocation > + # > return > fi > fi > @@ -128,6 +129,11 @@ echo "brevity is wit..." > > _check_test_fs > > +# check if preallocation is supported > +testfile=$TEST_DIR/$$.xfs_io > +testio=`$XFS_IO_PROG -F -f -c "falloc 0 1m" $testfile 2>&1` > +rm -f $testfile 2>&1 > /dev/null > + > # Options: > # -d: debug output for all operations > # -l flen: the upper bound on file size (default 262144) > diff --git a/tests/generic/112 b/tests/generic/112 > index 1879e7b5..a6f8fc14 100755 > --- a/tests/generic/112 > +++ b/tests/generic/112 > @@ -48,13 +48,14 @@ _do_test() > echo "fsx.$_n : $_filter_param" > echo "-----------------------------------------------" > > - if [ "$FSTYP" != "xfs" ] > - then > - if [ "$_n" = "1" -o "$_n" = "3" ] > - then > - # HACK: only xfs handles preallocation (-x) so just skip this test > - return > - fi > + if [ "$_n" = "1" -o "$_n" = "3" ]; then > + if echo $testio | grep -q "Operation not supported"; then > + # > + # Skip the prealloc runs if the file system does not support > + # preallocation > + # > + return > + fi > fi > > # This cd and use of -P gets full debug on $here (not TEST_DEV) > @@ -128,6 +129,11 @@ echo "brevity is wit..." > > _check_test_fs > > +# check if preallocation is supported > +testfile=$TEST_DIR/$$.xfs_io > +testio=`$XFS_IO_PROG -F -f -c "falloc 0 1m" $testfile 2>&1` > +rm -f $testfile 2>&1 > /dev/null > + > # Options: > # -d: debug output for all operations > # -l flen: the upper bound on file size (default 262144) > -- > 2.20.1 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] generic: detect preallocation support for fsx tests 075 and 112 2019-02-10 11:11 ` Eryu Guan @ 2019-02-11 0:37 ` Dave Chinner 2019-02-14 5:47 ` Eryu Guan 0 siblings, 1 reply; 5+ messages in thread From: Dave Chinner @ 2019-02-11 0:37 UTC (permalink / raw) To: Eryu Guan; +Cc: Christoph Hellwig, fstests On Sun, Feb 10, 2019 at 07:11:42PM +0800, Eryu Guan wrote: > On Tue, Feb 05, 2019 at 07:47:13PM +0100, Christoph Hellwig wrote: > > Currently generic/075 and generic/112 have two extra fsx passes each that > > exercise fsx with preallocation, which are only enabled for XFS. > > > > These tests can also be run with other file systems, given that the XFS > > prealloc ioctls are implemented in generic code since the addition of > > the fallocate system call. This also means a version of XFS that does > > not support preallocation (e.g. because it always writes out of place) > > can skip the prealloc tests while still completing the normal fsx tests > > just fine. > > > > Signed-off-by: Christoph Hellwig <hch@lst.de> > > --- > > tests/generic/075 | 16 +++++++++++----- > > tests/generic/112 | 20 +++++++++++++------- > > 2 files changed, 24 insertions(+), 12 deletions(-) > > > > diff --git a/tests/generic/075 b/tests/generic/075 > > index 2b957891..8981c560 100755 > > --- a/tests/generic/075 > > +++ b/tests/generic/075 > > @@ -48,11 +48,12 @@ _do_test() > > echo "fsx.$_n : $_filter_param" > > echo "-----------------------------------------------" > > > > - if [ "$FSTYP" != "xfs" ] > > - then > > - if [ "$_n" = "1" -o "$_n" = "3" ] > > - then > > - # HACK: only xfs handles preallocation (-x) so just skip this test > > + if [ "$_n" = "1" -o "$_n" = "3" ]; then > > + if echo $testio | grep -q "Operation not supported"; then > > Hmm, I don't think checking ENOSUPP message is sufficient, old distros > may ship xfs_io without falloc support or there's no generic prealloc > ioctl support, these cases are rare but possible. > > How about adding a new XFS_IOC_RESVSP ioctl check in src/feature.c and xfs_io -c "resvsp 0 1g" ... Cheers, Dave. -- Dave Chinner david@fromorbit.com ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] generic: detect preallocation support for fsx tests 075 and 112 2019-02-11 0:37 ` Dave Chinner @ 2019-02-14 5:47 ` Eryu Guan 2019-02-14 22:27 ` Dave Chinner 0 siblings, 1 reply; 5+ messages in thread From: Eryu Guan @ 2019-02-14 5:47 UTC (permalink / raw) To: Dave Chinner; +Cc: Christoph Hellwig, fstests On Mon, Feb 11, 2019 at 11:37:41AM +1100, Dave Chinner wrote: > On Sun, Feb 10, 2019 at 07:11:42PM +0800, Eryu Guan wrote: > > On Tue, Feb 05, 2019 at 07:47:13PM +0100, Christoph Hellwig wrote: > > > Currently generic/075 and generic/112 have two extra fsx passes each that > > > exercise fsx with preallocation, which are only enabled for XFS. > > > > > > These tests can also be run with other file systems, given that the XFS > > > prealloc ioctls are implemented in generic code since the addition of > > > the fallocate system call. This also means a version of XFS that does > > > not support preallocation (e.g. because it always writes out of place) > > > can skip the prealloc tests while still completing the normal fsx tests > > > just fine. > > > > > > Signed-off-by: Christoph Hellwig <hch@lst.de> > > > --- > > > tests/generic/075 | 16 +++++++++++----- > > > tests/generic/112 | 20 +++++++++++++------- > > > 2 files changed, 24 insertions(+), 12 deletions(-) > > > > > > diff --git a/tests/generic/075 b/tests/generic/075 > > > index 2b957891..8981c560 100755 > > > --- a/tests/generic/075 > > > +++ b/tests/generic/075 > > > @@ -48,11 +48,12 @@ _do_test() > > > echo "fsx.$_n : $_filter_param" > > > echo "-----------------------------------------------" > > > > > > - if [ "$FSTYP" != "xfs" ] > > > - then > > > - if [ "$_n" = "1" -o "$_n" = "3" ] > > > - then > > > - # HACK: only xfs handles preallocation (-x) so just skip this test > > > + if [ "$_n" = "1" -o "$_n" = "3" ]; then > > > + if echo $testio | grep -q "Operation not supported"; then > > > > Hmm, I don't think checking ENOSUPP message is sufficient, old distros > > may ship xfs_io without falloc support or there's no generic prealloc > > ioctl support, these cases are rare but possible. > > > > How about adding a new XFS_IOC_RESVSP ioctl check in src/feature.c and > > xfs_io -c "resvsp 0 1g" ... But we have to check for different error messages caused by different errnos again, and xfs_io doesn't set non-zero return value on failure, so there's no easy way to check if XFS_IOC_RESVSP ioctl is succeeded or not. I think it would be easier to do it in feature.c and we can rely on the return value of 'feature'. Or can we assume 'xfs -c "resvsp 0 1"' prints nothing on success, then we could check if it prints out some kind error messages? Thanks, Eryu ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] generic: detect preallocation support for fsx tests 075 and 112 2019-02-14 5:47 ` Eryu Guan @ 2019-02-14 22:27 ` Dave Chinner 0 siblings, 0 replies; 5+ messages in thread From: Dave Chinner @ 2019-02-14 22:27 UTC (permalink / raw) To: Eryu Guan; +Cc: Christoph Hellwig, fstests On Thu, Feb 14, 2019 at 01:47:59PM +0800, Eryu Guan wrote: > On Mon, Feb 11, 2019 at 11:37:41AM +1100, Dave Chinner wrote: > > On Sun, Feb 10, 2019 at 07:11:42PM +0800, Eryu Guan wrote: > > > On Tue, Feb 05, 2019 at 07:47:13PM +0100, Christoph Hellwig wrote: > > > > Currently generic/075 and generic/112 have two extra fsx passes each that > > > > exercise fsx with preallocation, which are only enabled for XFS. > > > > > > > > These tests can also be run with other file systems, given that the XFS > > > > prealloc ioctls are implemented in generic code since the addition of > > > > the fallocate system call. This also means a version of XFS that does > > > > not support preallocation (e.g. because it always writes out of place) > > > > can skip the prealloc tests while still completing the normal fsx tests > > > > just fine. > > > > > > > > Signed-off-by: Christoph Hellwig <hch@lst.de> > > > > --- > > > > tests/generic/075 | 16 +++++++++++----- > > > > tests/generic/112 | 20 +++++++++++++------- > > > > 2 files changed, 24 insertions(+), 12 deletions(-) > > > > > > > > diff --git a/tests/generic/075 b/tests/generic/075 > > > > index 2b957891..8981c560 100755 > > > > --- a/tests/generic/075 > > > > +++ b/tests/generic/075 > > > > @@ -48,11 +48,12 @@ _do_test() > > > > echo "fsx.$_n : $_filter_param" > > > > echo "-----------------------------------------------" > > > > > > > > - if [ "$FSTYP" != "xfs" ] > > > > - then > > > > - if [ "$_n" = "1" -o "$_n" = "3" ] > > > > - then > > > > - # HACK: only xfs handles preallocation (-x) so just skip this test > > > > + if [ "$_n" = "1" -o "$_n" = "3" ]; then > > > > + if echo $testio | grep -q "Operation not supported"; then > > > > > > Hmm, I don't think checking ENOSUPP message is sufficient, old distros > > > may ship xfs_io without falloc support or there's no generic prealloc > > > ioctl support, these cases are rare but possible. > > > > > > How about adding a new XFS_IOC_RESVSP ioctl check in src/feature.c and > > > > xfs_io -c "resvsp 0 1g" ... > > But we have to check for different error messages caused by different > errnos again, and xfs_io doesn't set non-zero return value on failure, > so there's no easy way to check if XFS_IOC_RESVSP ioctl is succeeded or > not. I think it would be easier to do it in feature.c and we can rely on > the return value of 'feature'. > > Or can we assume 'xfs -c "resvsp 0 1"' prints nothing on success, then > we could check if it prints out some kind error messages? Ummm, look at the code or run the command I quoted? io/prealloc.c::resvsp_f() if (xfsctl(file->name, file->fd, XFS_IOC_RESVSP64, &segment) < 0) { perror("XFS_IOC_RESVSP64"); return 0; } On XFS (success): $ xfs_io -f -c "resvsp 0 1" /mnt/scratch/testfile $ On ext3 (failure): $ sudo xfs_io -f -c "resvsp 0 1" /testfile XFS_IOC_RESVSP64: Operation not supported $ -Dave. -- Dave Chinner david@fromorbit.com ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-02-14 22:27 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-02-05 18:47 [PATCH] generic: detect preallocation support for fsx tests 075 and 112 Christoph Hellwig 2019-02-10 11:11 ` Eryu Guan 2019-02-11 0:37 ` Dave Chinner 2019-02-14 5:47 ` Eryu Guan 2019-02-14 22:27 ` Dave Chinner
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox