At 12/14/2016 07:51 AM, Tsutomu Itoh wrote:
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 <chandan@xxxxxxxxxxxxxxxxxx> Signed-off-by: Qu Wenruo <quwenruo@xxxxxxxxxxxxxx> --- 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
Oh, right. Thanks for pointing it out, I'll update it soon. Thanks, Qu
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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
