[prev in list] [next in list] [prev in thread] [next in thread] 

List:       fstests
Subject:    Re: [PATCH 07/10] btrfs/215: use _btrfs_get_first_logical
From:       Anand Jain <anand.jain () oracle ! com>
Date:       2022-05-30 0:56:51
Message-ID: 5b08b4a6-f4e5-9143-9646-19079f49675a () oracle ! com
[Download RAW message or body]

looks good.

Reviewed-by: Anand Jain <anand.jain@oracle.com>


On 5/27/22 13:49, Christoph Hellwig wrote:
> Use the _btrfs_get_first_logical helper instead of open coding it.
> 
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> Reviewed-by: Qu Wenruo <wqu@suse.com>
> ---
>   tests/btrfs/215 | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/btrfs/215 b/tests/btrfs/215
> index 0dcbce2a..928f365c 100755
> --- a/tests/btrfs/215
> +++ b/tests/btrfs/215
> @@ -56,7 +56,7 @@ fi
>   #create an 8 block file
>   $XFS_IO_PROG -d -f -c "pwrite -S 0xbb -b $filesize 0 $filesize" "$SCRATCH_MNT/foobar" > /dev/null
>   
> -logical_extent=$($FILEFRAG_PROG -v "$SCRATCH_MNT/foobar" | _filter_filefrag | cut -d '#' -f 1)
> +logical_extent=$(_btrfs_get_first_logical $SCRATCH_MNT/foobar)
>   physical_extent=$(get_physical $logical_extent)
>   
>   echo "logical = $logical_extent physical=$physical_extent" >> $seqres.full

[prev in list] [next in list] [prev in thread] [next in thread]