xfs
[Top] [All Lists]

Re: [PATCH] xfstests: tests/generic: added -P option to df

To: Stanislav Kholmanskikh <stanislav.kholmanskikh@xxxxxxxxxx>
Subject: Re: [PATCH] xfstests: tests/generic: added -P option to df
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Tue, 29 Oct 2013 08:44:44 +1100
Cc: xfs@xxxxxxxxxxx, vasily.isaenko@xxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1382444035-9502-1-git-send-email-stanislav.kholmanskikh@xxxxxxxxxx>
References: <1382444035-9502-1-git-send-email-stanislav.kholmanskikh@xxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Tue, Oct 22, 2013 at 04:13:55PM +0400, Stanislav Kholmanskikh wrote:
> Otherwise the testcases will fail if the scratch
> device has a long name (for example, if it's an LVM volume).
> Because df outputs its usage stats with two lines:
> 
> /dev/mapper/xfstests-disk1
>                        3030800      4608   2868908   1% /tmp/mnt/disk1
> 
> Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@xxxxxxxxxx>
> ---
>  tests/generic/251 |    2 +-
>  tests/generic/260 |    2 +-
>  tests/generic/273 |    2 +-
>  tests/generic/275 |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/generic/251 b/tests/generic/251
> index 4126f15..eca3b0d 100755
> --- a/tests/generic/251
> +++ b/tests/generic/251
> @@ -89,7 +89,7 @@ _guess_max_minlen()
>  fstrim_loop()
>  {
>       trap "_destroy_fstrim; exit \$status" 2 15
> -     fsize=$(df | grep $SCRATCH_MNT | grep $SCRATCH_DEV  | awk '{print $2}')
> +     fsize=$(df -P | grep $SCRATCH_MNT | grep $SCRATCH_DEV  | awk '{print 
> $2}')

Convert to $DF_PROG and add the -P flag to that variable, please.

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>