Re: [PATCH v3] geneirc/077 fix min size for btrfs

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]





On 10/11/2018 12:25 PM, Darrick J. Wong wrote:
On Thu, Oct 11, 2018 at 11:26:00AM +0800, Anand Jain wrote:
If btrfs need to be tested at its default blockgroup which is non-mixed,
then it needs at least 256mb.

Signed-off-by: Anand Jain <anand.jain@xxxxxxxxxx>
---
v2->v3:
	separated from the patch set of 9.
	notrun for the cases where filler is not big enough to fill the
		fssize.
v2->v1: ref the cover-letter of the set.

  tests/generic/077 | 11 +++++++----
  1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/tests/generic/077 b/tests/generic/077
index ef6af18c83e3..784afe448940 100755
--- a/tests/generic/077
+++ b/tests/generic/077
@@ -13,7 +13,7 @@ echo "QA output created by $seq"
  here=`pwd`
  tmp=/tmp/$$
  status=1
-# Something w/ enough data to fill 50M of fs...
+# Something w/ enough data to fill 256M of fs...
  filler=/lib/modules/
# fall back in case /lib/modules doesn't exist
@@ -38,6 +38,11 @@ _supported_os Linux
[ ! -d $filler ] && _notrun "No directory to source files from" +# check if two iterations of the assigned filler is big enough to fill fssize
+fs_size=$((256 * 1024 * 1024))
+[ $(( $(du -h -m /usr | tail -1| cut -f1) * 2 )) -lt 256 ] && \

Err... what does measuring /usr have to do with /lib/modules?

Ah. Thanks Darrick. Its a bug, I fixed it in v4.

Also, /lib/modules is 58M on my test VM, which means that a 256M
filesystem isn't going to ENOSPC.

 Yep as above its only checking for /usr its a bug fixed in v4.

Thanks, Anand

(Though weirdly it doesn't fail despite the lack of ENOSPC even at the
50M size, so I'm not sure what this test is actually supposed to do...)
>
--D

+	_notrun "filler $filler isn't big enough to fill fssize $fssize"
+
  _require_scratch
  _require_attrs
  _require_acls
@@ -49,9 +54,7 @@ rm -f $seqres.full
  _scratch_unmount >/dev/null 2>&1
  echo "*** MKFS ***"                         >>$seqres.full
  echo ""                                     >>$seqres.full
-SIZE=`expr 50 \* 1024 \* 1024`
-_scratch_mkfs_sized $SIZE                   >>$seqres.full 2>&1 \
-	|| _fail "mkfs failed"
+_scratch_mkfs_sized $fs_size >> $seqres.full 2>&1 || _fail "mkfs failed"
  _scratch_mount
  mkdir $SCRATCH_MNT/subdir
--
1.8.3.1




[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux