generic/204: correct log size for XFS

generic/204: correct log size for XFS

[alexander.tsvetkov]

[-- Attachment #1.1: Type: text/plain, Size: 1458 bytes --]

generic/204 fails on device with Advanced Format of 4096 bytes per
physical sector and when partition starts at the 4K boundary/./In this case
filesystem sector/block size will be of 4096 bytes size and 
_scratch_mkfs_sized
fails because mkfs reports that 5Mb log size is not enough to create a 
filesystem,
for example attempt to make filesystem on such partition:

mkfs.xfs -f -bsize=4096 -l size=5m -d size=109051904 /dev/sdb2"

results to:

"log size 1280 blocks too small, minimum size is 1605 blocks"

and generic/204fails with ENOSPC before it has finished creating the 
necessary files:

"QA output created by 204
./tests/generic/204: line 86: echo: write error: No space left on device
..."

Log size of 7Mbis enough to make filesytem and pass this test.

diff --git a/tests/generic/204 b/tests/generic/204
index 13a762a..4ebab26 100755
--- a/tests/generic/204
+++ b/tests/generic/204
@@ -53,9 +53,9 @@ _scratch_mkfs 2> /dev/null | _filter_mkfs 2> $tmp.mkfs 
 > /dev/null

  # For xfs, we need to handle the different default log sizes that 
different
  # versions of mkfs create. All should be valid with a 5MB log, so use 
that.
-[ $FSTYP = "xfs" ] && MKFS_OPTIONS="$MKFS_OPTIONS -l size=5m"
+[ $FSTYP = "xfs" ] && MKFS_OPTIONS="$MKFS_OPTIONS -l size=7m"

-SIZE=`expr 104 \* 1024 \* 1024`
+SIZE=`expr 106 \* 1024 \* 1024`
  _scratch_mkfs_sized $SIZE $dbsize 2> /dev/null \
                 | _filter_mkfs 2> $tmp.mkfs > /dev/null
  _scratch_mount

[-- Attachment #1.2: Type: text/html, Size: 2532 bytes --]

[-- Attachment #2: Type: text/plain, Size: 121 bytes --]

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

Re: generic/204: correct log size for XFS

[Dave Chinner]

On Thu, Apr 10, 2014 at 02:28:41PM +0400, alexander.tsvetkov@oracle.com wrote:
> generic/204 fails on device with Advanced Format of 4096 bytes per
> physical sector and when partition starts at the 4K boundary/./In this case
> filesystem sector/block size will be of 4096 bytes size and
> _scratch_mkfs_sized
> fails because mkfs reports that 5Mb log size is not enough to create
> a filesystem,
> for example attempt to make filesystem on such partition:
> 
> mkfs.xfs -f -bsize=4096 -l size=5m -d size=109051904 /dev/sdb2"
> 
> results to:
> 
> "log size 1280 blocks too small, minimum size is 1605 blocks"
> 
> and generic/204fails with ENOSPC before it has finished creating the
> necessary files:
> 
> "QA output created by 204
> ./tests/generic/204: line 86: echo: write error: No space left on device
> ..."
> 
> Log size of 7Mbis enough to make filesytem and pass this test.

This is missing a s-o-b line. Can you add one?

> 
> diff --git a/tests/generic/204 b/tests/generic/204
> index 13a762a..4ebab26 100755
> --- a/tests/generic/204
> +++ b/tests/generic/204
> @@ -53,9 +53,9 @@ _scratch_mkfs 2> /dev/null | _filter_mkfs 2>
> $tmp.mkfs > /dev/null
> 
>  # For xfs, we need to handle the different default log sizes that
> different
>  # versions of mkfs create. All should be valid with a 5MB log, so
> use that.
> -[ $FSTYP = "xfs" ] && MKFS_OPTIONS="$MKFS_OPTIONS -l size=5m"
> +[ $FSTYP = "xfs" ] && MKFS_OPTIONS="$MKFS_OPTIONS -l size=7m"
> 
> -SIZE=`expr 104 \* 1024 \* 1024`
> +SIZE=`expr 106 \* 1024 \* 1024`
>  _scratch_mkfs_sized $SIZE $dbsize 2> /dev/null \
>                 | _filter_mkfs 2> $tmp.mkfs > /dev/null
>  _scratch_mount

The change looks fine, but your mailer is line wrapping and
whitespace mangling the patches.  Please read
Documentation/SubmittingPatches in the kernel tree for how to do
this correctly.  Also, please don't send patches in multipart emails
- turn off html sending so that git am doesn't throw a wobbly over
it...

I've fixed it up this time....

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs