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