On 8/16/13 12:02 PM, Stefan Behrens wrote:
> The btrfs-progs tools changed the output:
> - 100GiB instead of 100GB
> - The number of spaces was changed
ugh.
>
> Signed-off-by: Stefan Behrens <sbehrens@xxxxxxxxxxxxxxxx>
> ---
> common/filter | 2 +-
> common/filter.btrfs | 3 ++-
> tests/btrfs/006 | 6 +++---
> tests/btrfs/006.out | 36 ++++++++++++++++++------------------
> 4 files changed, 24 insertions(+), 23 deletions(-)
>
> diff --git a/common/filter b/common/filter
> index dbb1674..ee738ca 100644
> --- a/common/filter
> +++ b/common/filter
> @@ -262,7 +262,7 @@ _filter_uuid()
> # Filter out sizes like 6.14MB etc
> _filter_size()
> {
> - sed -e "s/[0-9\.]\+\s\?[b|k|m|g|t][b]\?/<SIZE>/ig"
> + sed -e "s/[0-9\.]\+\s\?[b|k|m|g|t][i]\?[b]\?/<SIZE>/ig"
> }
makes sense
But for the rest, is the output change intentional, or sloppiness/accidental?
If it's really intentional, then:
Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx>
> # Convert string read from stdin like 128K to bytes and print it to stdout
> diff --git a/common/filter.btrfs b/common/filter.btrfs
> index e9a2bc2..1584596 100644
> --- a/common/filter.btrfs
> +++ b/common/filter.btrfs
> @@ -33,6 +33,7 @@ _filter_btrfs_filesystem_show()
> # the uniq collapses all device lines into 1
> _filter_uuid $UUID | _filter_scratch | _filter_scratch_pool | \
> _filter_size | _filter_btrfs_version | _filter_devid | \
> + _filter_spaces | \
> sed -e "s/\(Total devices\) $NUMDEVS/\1 $NUM_SUBST/g" | \
> uniq
> }
> @@ -51,7 +52,7 @@ _filter_btrfs_device_stats()
> UNIQ_OPT=""
> fi
>
> - _filter_scratch | _filter_scratch_pool | \
> + _filter_scratch | _filter_scratch_pool | _filter_spaces | \
> sed -e "s/[0-9]\+$/<NUM>/g" | sort | uniq $UNIQ_OPT | \
> sed -e "s/$NUMDEVS /<NUMDEVS> /g"
> }
> diff --git a/tests/btrfs/006 b/tests/btrfs/006
> index 9f7beff..f323cc4 100755
> --- a/tests/btrfs/006
> +++ b/tests/btrfs/006
> @@ -82,13 +82,13 @@ echo "== Sync filesystem"
> $BTRFS_UTIL_PROG filesystem sync $SCRATCH_MNT | _filter_scratch
>
> echo "== Show device stats by mountpoint"
> -$BTRFS_UTIL_PROG device stats $SCRATCH_MNT | _filter_btrfs_device_stats $TOTAL_DEVS
> +$BTRFS_UTIL_PROG device stats $SCRATCH_MNT | _filter_btrfs_device_stats $TOTAL_DEVS | _filter_spaces
> echo "== Show device stats by first/scratch dev"
> $BTRFS_UTIL_PROG device stats $SCRATCH_DEV | _filter_btrfs_device_stats
> echo "== Show device stats by second dev"
> -$BTRFS_UTIL_PROG device stats $FIRST_POOL_DEV | sed -e "s,$FIRST_POOL_DEV,FIRST_POOL_DEV,g"
> +$BTRFS_UTIL_PROG device stats $FIRST_POOL_DEV | sed -e "s,$FIRST_POOL_DEV,FIRST_POOL_DEV,g" | _filter_spaces
> echo "== Show device stats by last dev"
> -$BTRFS_UTIL_PROG device stats $LAST_POOL_DEV | sed -e "s,$LAST_POOL_DEV,LAST_POOL_DEV,g"
> +$BTRFS_UTIL_PROG device stats $LAST_POOL_DEV | sed -e "s,$LAST_POOL_DEV,LAST_POOL_DEV,g" | _filter_spaces
>
> # success, all done
> status=0
> diff --git a/tests/btrfs/006.out b/tests/btrfs/006.out
> index ab33b7e..413a5a8 100644
> --- a/tests/btrfs/006.out
> +++ b/tests/btrfs/006.out
> @@ -4,38 +4,38 @@
> TestLabel.006
> == Mount.
> == Show filesystem by label
> -Label: 'TestLabel.006' uuid: <UUID>
> +Label: 'TestLabel.006' uuid: <UUID>
> Total devices <EXACTNUM> FS bytes used <SIZE>
> - devid <DEVID> size <SIZE> used <SIZE> path SCRATCH_DEV
> + devid <DEVID> size <SIZE> used <SIZE> path SCRATCH_DEV
>
> == Show filesystem by UUID
> -Label: 'TestLabel.006' uuid: <EXACTUUID>
> +Label: 'TestLabel.006' uuid: <EXACTUUID>
> Total devices <EXACTNUM> FS bytes used <SIZE>
> - devid <DEVID> size <SIZE> used <SIZE> path SCRATCH_DEV
> + devid <DEVID> size <SIZE> used <SIZE> path SCRATCH_DEV
>
> == Sync filesystem
> FSSync 'SCRATCH_MNT'
> == Show device stats by mountpoint
> - <NUMDEVS> [SCRATCH_DEV].corruption_errs <NUM>
> - <NUMDEVS> [SCRATCH_DEV].flush_io_errs <NUM>
> - <NUMDEVS> [SCRATCH_DEV].generation_errs <NUM>
> - <NUMDEVS> [SCRATCH_DEV].read_io_errs <NUM>
> - <NUMDEVS> [SCRATCH_DEV].write_io_errs <NUM>
> + <NUMDEVS> [SCRATCH_DEV].corruption_errs <NUM>
> + <NUMDEVS> [SCRATCH_DEV].flush_io_errs <NUM>
> + <NUMDEVS> [SCRATCH_DEV].generation_errs <NUM>
> + <NUMDEVS> [SCRATCH_DEV].read_io_errs <NUM>
> + <NUMDEVS> [SCRATCH_DEV].write_io_errs <NUM>
> == Show device stats by first/scratch dev
> [SCRATCH_DEV].corruption_errs <NUM>
> -[SCRATCH_DEV].flush_io_errs <NUM>
> +[SCRATCH_DEV].flush_io_errs <NUM>
> [SCRATCH_DEV].generation_errs <NUM>
> -[SCRATCH_DEV].read_io_errs <NUM>
> -[SCRATCH_DEV].write_io_errs <NUM>
> +[SCRATCH_DEV].read_io_errs <NUM>
> +[SCRATCH_DEV].write_io_errs <NUM>
> == Show device stats by second dev
> -[FIRST_POOL_DEV].write_io_errs 0
> -[FIRST_POOL_DEV].read_io_errs 0
> -[FIRST_POOL_DEV].flush_io_errs 0
> +[FIRST_POOL_DEV].write_io_errs 0
> +[FIRST_POOL_DEV].read_io_errs 0
> +[FIRST_POOL_DEV].flush_io_errs 0
> [FIRST_POOL_DEV].corruption_errs 0
> [FIRST_POOL_DEV].generation_errs 0
> == Show device stats by last dev
> -[LAST_POOL_DEV].write_io_errs 0
> -[LAST_POOL_DEV].read_io_errs 0
> -[LAST_POOL_DEV].flush_io_errs 0
> +[LAST_POOL_DEV].write_io_errs 0
> +[LAST_POOL_DEV].read_io_errs 0
> +[LAST_POOL_DEV].flush_io_errs 0
> [LAST_POOL_DEV].corruption_errs 0
> [LAST_POOL_DEV].generation_errs 0
>
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
