On 2016/12/13 18:44, Qu Wenruo wrote:
> Populate fs after convert so we can trigger data chunk allocation.
> This can expose too restrict old rollback condition
> 
> Reported-by: Chandan Rajendra <chan...@linux.vnet.ibm.com>
> Signed-off-by: Qu Wenruo <quwen...@cn.fujitsu.com>
> ---
>  tests/common         | 4 ++++
>  tests/common.convert | 3 +++
>  2 files changed, 7 insertions(+)
> 
> diff --git a/tests/common b/tests/common
> index 571118a..4a1330f 100644
> --- a/tests/common
> +++ b/tests/common
> @@ -486,6 +486,10 @@ generate_dataset() {
>                               run_check $SUDO_HELPER ln -s 
> "$dirpath/$long_filename" "$dirpath/slow_slink.$num"
>                       done
>                       ;;
> +             large)
> +                     run_check $SUDO_HELPER dd if=/dev/urandom bs=32M 
> count=1 \
> +                     of="$dirpath/$dataset_type" bs=32M >/dev/null 2>&1
> +                     ;;

Too many bs=.

Thanks,
Tsutomu

>       esac
>  }
>  
> diff --git a/tests/common.convert b/tests/common.convert
> index a2d3152..8c9242e 100644
> --- a/tests/common.convert
> +++ b/tests/common.convert
> @@ -160,6 +160,9 @@ convert_test_post_checks_all() {
>       convert_test_post_check_checksums "$1"
>       convert_test_post_check_permissions "$2"
>       convert_test_post_check_acl "$3"
> +
> +     # Create a large file to trigger data chunk allocation
> +     generate_dataset "large"
>       run_check_umount_test_dev
>  }
>  
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to