diff mbox series

xfs/145: Remove useless -f option for xfs_quota command

Message ID 1619067943-3859-1-git-send-email-xuyang2018.jy@fujitsu.com (mailing list archive)
State New, archived
Headers show
Series xfs/145: Remove useless -f option for xfs_quota command | expand

Commit Message

Yang Xu (Fujitsu) April 22, 2021, 5:05 a.m. UTC 
Since it is a xfs specifal case, we don't need to use foreign filesystem mode.

Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com>
Signed-off-by: Jinhui Huang <huangjh.jy@fujitsu.com>
---
 tests/xfs/145 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Brian Foster April 23, 2021, 11:13 a.m. UTC | #1 
On Thu, Apr 22, 2021 at 01:05:43PM +0800, Yang Xu wrote:
> Since it is a xfs specifal case, we don't need to use foreign filesystem mode.
> 
> Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com>
> Signed-off-by: Jinhui Huang <huangjh.jy@fujitsu.com>
> ---

Seems reasonable. Any particular reason for this change, or just an
observation?

Reviewed-by: Brian Foster <bfoster@redhat.com>

>  tests/xfs/145 | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/xfs/145 b/tests/xfs/145
> index 3b3c375d..3ccbb923 100755
> --- a/tests/xfs/145
> +++ b/tests/xfs/145
> @@ -51,7 +51,7 @@ $FILEFRAG_PROG -v $SCRATCH_MNT/testy 2>&1 | grep -q delalloc || \
>  rm -f $SCRATCH_MNT/testy
>  
>  echo "Run test program"
> -$XFS_QUOTA_PROG -f -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
> +$XFS_QUOTA_PROG -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
>  $here/src/chprojid_fail $SCRATCH_MNT/blah
>  
>  # The regression we're testing for is an accounting bug involving delalloc
> @@ -64,7 +64,7 @@ $FILEFRAG_PROG -v $SCRATCH_MNT/blah 2>&1 | grep -q delalloc || \
>  	echo "file didn't get delalloc extents, test invalid?"
>  
>  # Make a note of current quota status for diagnostic purposes
> -$XFS_QUOTA_PROG -f -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
> +$XFS_QUOTA_PROG -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
>  
>  # success, all done
>  status=0
> -- 
> 2.23.0
>
Yang Xu (Fujitsu) April 25, 2021, 2:10 a.m. UTC | #2 
on 2021/4/23 19:13, Brian Foster wrote:
> On Thu, Apr 22, 2021 at 01:05:43PM +0800, Yang Xu wrote:
>> Since it is a xfs specifal case, we don't need to use foreign filesystem mode.
>>
>> Signed-off-by: Yang Xu<xuyang2018.jy@fujitsu.com>
>> Signed-off-by: Jinhui Huang<huangjh.jy@fujitsu.com>
>> ---
>
> Seems reasonable. Any particular reason for this change, or just an
> observation?
I tested this case on Centos7.9 and then I got this failure
/usr/sbin/xfs_quota: invalid option -- 'f'

so I sent this patch.

When I tested this case on unpatched upstream kernel, it still pass. So 
this case does need some special requirements(like xfsprogs version, or 
other kernel config)?

Best Regards
Yang Xu
>
> Reviewed-by: Brian Foster<bfoster@redhat.com>
>
>>   tests/xfs/145 | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/xfs/145 b/tests/xfs/145
>> index 3b3c375d..3ccbb923 100755
>> --- a/tests/xfs/145
>> +++ b/tests/xfs/145
>> @@ -51,7 +51,7 @@ $FILEFRAG_PROG -v $SCRATCH_MNT/testy 2>&1 | grep -q delalloc || \
>>   rm -f $SCRATCH_MNT/testy
>>
>>   echo "Run test program"
>> -$XFS_QUOTA_PROG -f -x -c 'report -ap' $SCRATCH_MNT>>  $seqres.full
>> +$XFS_QUOTA_PROG -x -c 'report -ap' $SCRATCH_MNT>>  $seqres.full
>>   $here/src/chprojid_fail $SCRATCH_MNT/blah
>>
>>   # The regression we're testing for is an accounting bug involving delalloc
>> @@ -64,7 +64,7 @@ $FILEFRAG_PROG -v $SCRATCH_MNT/blah 2>&1 | grep -q delalloc || \
>>   	echo "file didn't get delalloc extents, test invalid?"
>>
>>   # Make a note of current quota status for diagnostic purposes
>> -$XFS_QUOTA_PROG -f -x -c 'report -ap' $SCRATCH_MNT>>  $seqres.full
>> +$XFS_QUOTA_PROG -x -c 'report -ap' $SCRATCH_MNT>>  $seqres.full
>>
>>   # success, all done
>>   status=0
>> --
>> 2.23.0
>>
>
diff mbox series

Patch

diff --git a/tests/xfs/145 b/tests/xfs/145
index 3b3c375d..3ccbb923 100755
--- a/tests/xfs/145
+++ b/tests/xfs/145
@@ -51,7 +51,7 @@  $FILEFRAG_PROG -v $SCRATCH_MNT/testy 2>&1 | grep -q delalloc || \
 rm -f $SCRATCH_MNT/testy
 
 echo "Run test program"
-$XFS_QUOTA_PROG -f -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
+$XFS_QUOTA_PROG -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
 $here/src/chprojid_fail $SCRATCH_MNT/blah
 
 # The regression we're testing for is an accounting bug involving delalloc
@@ -64,7 +64,7 @@  $FILEFRAG_PROG -v $SCRATCH_MNT/blah 2>&1 | grep -q delalloc || \
 	echo "file didn't get delalloc extents, test invalid?"
 
 # Make a note of current quota status for diagnostic purposes
-$XFS_QUOTA_PROG -f -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
+$XFS_QUOTA_PROG -x -c 'report -ap' $SCRATCH_MNT >> $seqres.full
 
 # success, all done
 status=0