From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:35822 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753376AbaLAPMi (ORCPT ); Mon, 1 Dec 2014 10:12:38 -0500 Date: Mon, 1 Dec 2014 10:12:33 -0500 From: Brian Foster Subject: Re: [PATCH V2] xfs/096: strip log stripe warnings from mkfs.xfs Message-ID: <20141201151232.GA23055@bfoster.bfoster> References: <1416442106-14140-1-git-send-email-david@fromorbit.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1416442106-14140-1-git-send-email-david@fromorbit.com> Sender: fstests-owner@vger.kernel.org To: Dave Chinner Cc: fstests@vger.kernel.org List-ID: On Thu, Nov 20, 2014 at 11:08:26AM +1100, Dave Chinner wrote: > From: Dave Chinner > > 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 > --- Looks fine and makes xfs/096 pass again... Reviewed-by: Brian Foster > 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