* [PATCH V2] xfs/096: strip log stripe warnings from mkfs.xfs
@ 2014-11-20 0:08 Dave Chinner
2014-12-01 5:08 ` Dave Chinner
2014-12-01 15:12 ` Brian Foster
0 siblings, 2 replies; 3+ messages in thread
From: Dave Chinner @ 2014-11-20 0:08 UTC (permalink / raw)
To: fstests
From: Dave Chinner <dchinner@redhat.com>
The warnings have been removed when stripe unit/width are too big
for the log stripe unit to be set. Hence add them to the mkfs.xfs
filter and strip them from golden output file so the test succeeds
on both new and old mfks binaries.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
---
tests/xfs/096 | 4 +++-
tests/xfs/096.external | 2 --
tests/xfs/096.internal | 4 ----
3 files changed, 3 insertions(+), 7 deletions(-)
diff --git a/tests/xfs/096 b/tests/xfs/096
index b2719bc..0ecf88f 100755
--- a/tests/xfs/096
+++ b/tests/xfs/096
@@ -88,6 +88,8 @@ _mkfs_filter()
-e '/.*crc=/d' \
-e 's/ *$//' \
-e 's/ ftype=[01]//' \
+ -e '/^log stripe unit.*too large/d' \
+ -e '/^log stripe unit adjusted/d' \
| grep -v parent
}
@@ -143,7 +145,7 @@ do
fi
echo "--- mkfs=$mkfs ---"
export MKFS_OPTIONS="$mkfs"
- _scratch_mkfs_xfs | _mkfs_filter
+ _scratch_mkfs_xfs 2>&1 | _mkfs_filter
echo ""
echo ""
done
diff --git a/tests/xfs/096.external b/tests/xfs/096.external
index 9175904..95833c8 100644
--- a/tests/xfs/096.external
+++ b/tests/xfs/096.external
@@ -7,8 +7,6 @@ log stripe unit (262656) must be a multiple of the block size (4096)
# test log stripe greater than LR size
--- mkfs=-l version=2,su=266240 ---
-log stripe unit (266240 bytes) is too large (maximum is 256KiB)
-log stripe unit adjusted to 32KiB
meta-data=DEV isize=256 agcount=N, agsize=N blks
data = bsize=4096 blocks=N, imaxpct=N
= sunit=0 swidth=0 blks, unwritten=1
diff --git a/tests/xfs/096.internal b/tests/xfs/096.internal
index cf75037..7bf848c 100644
--- a/tests/xfs/096.internal
+++ b/tests/xfs/096.internal
@@ -7,8 +7,6 @@ log stripe unit (262656) must be a multiple of the block size (4096)
# test log stripe greater than LR size
--- mkfs=-l version=2,su=266240 ---
-log stripe unit (266240 bytes) is too large (maximum is 256KiB)
-log stripe unit adjusted to 32KiB
meta-data=DEV isize=256 agcount=N, agsize=N blks
data = bsize=4096 blocks=N, imaxpct=N
= sunit=0 swidth=0 blks, unwritten=1
@@ -20,8 +18,6 @@ realtime =REALTIME extsz=N, blocks=N, rtextents=N
# same test but get log stripe from data stripe
--- mkfs=-l version=2 -d su=266240,sw=1 ---
-log stripe unit (266240 bytes) is too large (maximum is 256KiB)
-log stripe unit adjusted to 32KiB
meta-data=DEV isize=256 agcount=N, agsize=N blks
data = bsize=4096 blocks=N, imaxpct=N
= sunit=65 swidth=65 blks, unwritten=1
--
2.0.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH V2] xfs/096: strip log stripe warnings from mkfs.xfs
2014-11-20 0:08 [PATCH V2] xfs/096: strip log stripe warnings from mkfs.xfs Dave Chinner
@ 2014-12-01 5:08 ` Dave Chinner
2014-12-01 15:12 ` Brian Foster
1 sibling, 0 replies; 3+ messages in thread
From: Dave Chinner @ 2014-12-01 5:08 UTC (permalink / raw)
To: fstests
ping?
On Thu, Nov 20, 2014 at 11:08:26AM +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> The warnings have been removed when stripe unit/width are too big
> for the log stripe unit to be set. Hence add them to the mkfs.xfs
> filter and strip them from golden output file so the test succeeds
> on both new and old mfks binaries.
>
> Signed-off-by: Dave Chinner <dchinner@redhat.com>
> ---
> tests/xfs/096 | 4 +++-
> tests/xfs/096.external | 2 --
> tests/xfs/096.internal | 4 ----
> 3 files changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/tests/xfs/096 b/tests/xfs/096
> index b2719bc..0ecf88f 100755
> --- a/tests/xfs/096
> +++ b/tests/xfs/096
> @@ -88,6 +88,8 @@ _mkfs_filter()
> -e '/.*crc=/d' \
> -e 's/ *$//' \
> -e 's/ ftype=[01]//' \
> + -e '/^log stripe unit.*too large/d' \
> + -e '/^log stripe unit adjusted/d' \
> | grep -v parent
> }
>
> @@ -143,7 +145,7 @@ do
> fi
> echo "--- mkfs=$mkfs ---"
> export MKFS_OPTIONS="$mkfs"
> - _scratch_mkfs_xfs | _mkfs_filter
> + _scratch_mkfs_xfs 2>&1 | _mkfs_filter
> echo ""
> echo ""
> done
> diff --git a/tests/xfs/096.external b/tests/xfs/096.external
> index 9175904..95833c8 100644
> --- a/tests/xfs/096.external
> +++ b/tests/xfs/096.external
> @@ -7,8 +7,6 @@ log stripe unit (262656) must be a multiple of the block size (4096)
>
> # test log stripe greater than LR size
> --- mkfs=-l version=2,su=266240 ---
> -log stripe unit (266240 bytes) is too large (maximum is 256KiB)
> -log stripe unit adjusted to 32KiB
> meta-data=DEV isize=256 agcount=N, agsize=N blks
> data = bsize=4096 blocks=N, imaxpct=N
> = sunit=0 swidth=0 blks, unwritten=1
> diff --git a/tests/xfs/096.internal b/tests/xfs/096.internal
> index cf75037..7bf848c 100644
> --- a/tests/xfs/096.internal
> +++ b/tests/xfs/096.internal
> @@ -7,8 +7,6 @@ log stripe unit (262656) must be a multiple of the block size (4096)
>
> # test log stripe greater than LR size
> --- mkfs=-l version=2,su=266240 ---
> -log stripe unit (266240 bytes) is too large (maximum is 256KiB)
> -log stripe unit adjusted to 32KiB
> meta-data=DEV isize=256 agcount=N, agsize=N blks
> data = bsize=4096 blocks=N, imaxpct=N
> = sunit=0 swidth=0 blks, unwritten=1
> @@ -20,8 +18,6 @@ realtime =REALTIME extsz=N, blocks=N, rtextents=N
>
> # same test but get log stripe from data stripe
> --- mkfs=-l version=2 -d su=266240,sw=1 ---
> -log stripe unit (266240 bytes) is too large (maximum is 256KiB)
> -log stripe unit adjusted to 32KiB
> meta-data=DEV isize=256 agcount=N, agsize=N blks
> data = bsize=4096 blocks=N, imaxpct=N
> = sunit=65 swidth=65 blks, unwritten=1
> --
> 2.0.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Dave Chinner
david@fromorbit.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V2] xfs/096: strip log stripe warnings from mkfs.xfs
2014-11-20 0:08 [PATCH V2] xfs/096: strip log stripe warnings from mkfs.xfs Dave Chinner
2014-12-01 5:08 ` Dave Chinner
@ 2014-12-01 15:12 ` Brian Foster
1 sibling, 0 replies; 3+ messages in thread
From: Brian Foster @ 2014-12-01 15:12 UTC (permalink / raw)
To: Dave Chinner; +Cc: fstests
On Thu, Nov 20, 2014 at 11:08:26AM +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> The warnings have been removed when stripe unit/width are too big
> for the log stripe unit to be set. Hence add them to the mkfs.xfs
> filter and strip them from golden output file so the test succeeds
> on both new and old mfks binaries.
>
> Signed-off-by: Dave Chinner <dchinner@redhat.com>
> ---
Looks fine and makes xfs/096 pass again...
Reviewed-by: Brian Foster <bfoster@redhat.com>
> tests/xfs/096 | 4 +++-
> tests/xfs/096.external | 2 --
> tests/xfs/096.internal | 4 ----
> 3 files changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/tests/xfs/096 b/tests/xfs/096
> index b2719bc..0ecf88f 100755
> --- a/tests/xfs/096
> +++ b/tests/xfs/096
> @@ -88,6 +88,8 @@ _mkfs_filter()
> -e '/.*crc=/d' \
> -e 's/ *$//' \
> -e 's/ ftype=[01]//' \
> + -e '/^log stripe unit.*too large/d' \
> + -e '/^log stripe unit adjusted/d' \
> | grep -v parent
> }
>
> @@ -143,7 +145,7 @@ do
> fi
> echo "--- mkfs=$mkfs ---"
> export MKFS_OPTIONS="$mkfs"
> - _scratch_mkfs_xfs | _mkfs_filter
> + _scratch_mkfs_xfs 2>&1 | _mkfs_filter
> echo ""
> echo ""
> done
> diff --git a/tests/xfs/096.external b/tests/xfs/096.external
> index 9175904..95833c8 100644
> --- a/tests/xfs/096.external
> +++ b/tests/xfs/096.external
> @@ -7,8 +7,6 @@ log stripe unit (262656) must be a multiple of the block size (4096)
>
> # test log stripe greater than LR size
> --- mkfs=-l version=2,su=266240 ---
> -log stripe unit (266240 bytes) is too large (maximum is 256KiB)
> -log stripe unit adjusted to 32KiB
> meta-data=DEV isize=256 agcount=N, agsize=N blks
> data = bsize=4096 blocks=N, imaxpct=N
> = sunit=0 swidth=0 blks, unwritten=1
> diff --git a/tests/xfs/096.internal b/tests/xfs/096.internal
> index cf75037..7bf848c 100644
> --- a/tests/xfs/096.internal
> +++ b/tests/xfs/096.internal
> @@ -7,8 +7,6 @@ log stripe unit (262656) must be a multiple of the block size (4096)
>
> # test log stripe greater than LR size
> --- mkfs=-l version=2,su=266240 ---
> -log stripe unit (266240 bytes) is too large (maximum is 256KiB)
> -log stripe unit adjusted to 32KiB
> meta-data=DEV isize=256 agcount=N, agsize=N blks
> data = bsize=4096 blocks=N, imaxpct=N
> = sunit=0 swidth=0 blks, unwritten=1
> @@ -20,8 +18,6 @@ realtime =REALTIME extsz=N, blocks=N, rtextents=N
>
> # same test but get log stripe from data stripe
> --- mkfs=-l version=2 -d su=266240,sw=1 ---
> -log stripe unit (266240 bytes) is too large (maximum is 256KiB)
> -log stripe unit adjusted to 32KiB
> meta-data=DEV isize=256 agcount=N, agsize=N blks
> data = bsize=4096 blocks=N, imaxpct=N
> = sunit=65 swidth=65 blks, unwritten=1
> --
> 2.0.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-12-01 15:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-20 0:08 [PATCH V2] xfs/096: strip log stripe warnings from mkfs.xfs Dave Chinner
2014-12-01 5:08 ` Dave Chinner
2014-12-01 15:12 ` Brian Foster
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox