From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from ipmail04.adl6.internode.on.net ([150.101.137.141]:36977 "EHLO ipmail04.adl6.internode.on.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932696AbcJYFuU (ORCPT ); Tue, 25 Oct 2016 01:50:20 -0400 Date: Tue, 25 Oct 2016 16:50:16 +1100 From: Dave Chinner Subject: Re: [PATCH] xfs/054: only filter ROOT_INO at the beginning Message-ID: <20161025055016.GB14023@dastard> References: <1477114796-19019-1-git-send-email-eguan@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1477114796-19019-1-git-send-email-eguan@redhat.com> Sender: fstests-owner@vger.kernel.org To: Eryu Guan Cc: fstests@vger.kernel.org List-ID: On Sat, Oct 22, 2016 at 01:39:56PM +0800, Eryu Guan wrote: > In the test ROOT_INO is filtered out or replaced, but if ROOT_INO is > also 32, more "32"s are filtered or replaced than expected. This > happens to me when testing 512B block size XFS and 1k block size CRC > enabled XFS. > > Also the ROOT_INO should be the inode number of TEST_DIR not > SCRATCH_MNT. > > Signed-off-by: Eryu Guan > --- > tests/xfs/054 | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/tests/xfs/054 b/tests/xfs/054 > index 7d08f8a..578b752 100755 > --- a/tests/xfs/054 > +++ b/tests/xfs/054 > @@ -52,13 +52,13 @@ _require_test > _require_xfs_io_command "inode" > > # We know the root inode is there > -ROOT_INO=`ls -id $SCRATCH_MNT | awk '{print $1}'` > +ROOT_INO=`ls -id $TEST_DIR | awk '{print $1}'` > > touch $TEST_DIR/file > > $XFS_IO_PROG -c "inode" $TEST_DIR/file > $XFS_IO_PROG -c "inode -v" $TEST_DIR/file | \ > - grep -vw $ROOT_INO | sed -e s/.*:/LAST:/g > + grep -vw "^$ROOT_INO" | sed -e s/.*:/LAST:/g So all of these sed expressions have the modifier "/g" on them, which means "replace every matching occurrence on the line". If all we want to do is replace the first match, then we should be dropping the "g" modifier from the sed expressions... Cheers, Dave. -- Dave Chinner david@fromorbit.com