Message ID | 1476759584-4150-1-git-send-email-yangx.jy@cn.fujitsu.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Tue, Oct 18, 2016 at 10:59:44AM +0800, Xiao Yang wrote: > We need to remove unused ${SCRATCH_DEV} when calling __populate_check_xfs_dir > because __populate_check_xfs_dir function has been rewritten by Darrick. What a jerk, that guy. ;) Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> (There'll be a lot more going on soon with common/populate once I get going on the "fuzz every field" tests again.) > Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com> > --- > tests/xfs/099 | 2 +- > tests/xfs/100 | 2 +- > tests/xfs/101 | 2 +- > tests/xfs/102 | 2 +- > tests/xfs/105 | 2 +- > tests/xfs/112 | 2 +- > tests/xfs/113 | 2 +- > 7 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/tests/xfs/099 b/tests/xfs/099 > index ebc02dd..85193ab 100755 > --- a/tests/xfs/099 > +++ b/tests/xfs/099 > @@ -75,7 +75,7 @@ echo "+ check fs" > _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" > > echo "+ check dir" > -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" block > +__populate_check_xfs_dir "${inode}" block > > echo "+ corrupt dir" > _scratch_xfs_db -x -c "inode ${inode}" -c 'dblock 0' -c "stack" -c "blocktrash -x 32 -y $((blksz * 8)) -z ${FUZZ_ARGS}" >> $seqres.full > diff --git a/tests/xfs/100 b/tests/xfs/100 > index 2412c81..3bfafce 100755 > --- a/tests/xfs/100 > +++ b/tests/xfs/100 > @@ -75,7 +75,7 @@ echo "+ check fs" > _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" > > echo "+ check dir" > -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" leaf > +__populate_check_xfs_dir "${inode}" leaf > > echo "+ corrupt dir" > loff=0 > diff --git a/tests/xfs/101 b/tests/xfs/101 > index 679940f..d3abd19 100755 > --- a/tests/xfs/101 > +++ b/tests/xfs/101 > @@ -75,7 +75,7 @@ echo "+ check fs" > _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" > > echo "+ check dir" > -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" leaf > +__populate_check_xfs_dir "${inode}" leaf > > echo "+ corrupt dir" > _scratch_xfs_db -x -c "inode ${inode}" -c "dblock ${leaf_lblk}" -c "stack" -c "blocktrash -x 32 -y $((blksz * 8)) -z ${FUZZ_ARGS}" >> $seqres.full > diff --git a/tests/xfs/102 b/tests/xfs/102 > index 8018a9e..cf0d7a6 100755 > --- a/tests/xfs/102 > +++ b/tests/xfs/102 > @@ -75,7 +75,7 @@ echo "+ check fs" > _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" > > echo "+ check dir" > -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" node > +__populate_check_xfs_dir "${inode}" node > > echo "+ corrupt dir" > loff=0 > diff --git a/tests/xfs/105 b/tests/xfs/105 > index 27ee4b3..07ccf00 100755 > --- a/tests/xfs/105 > +++ b/tests/xfs/105 > @@ -75,7 +75,7 @@ echo "+ check fs" > _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" > > echo "+ check dir" > -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" node > +__populate_check_xfs_dir "${inode}" node > > echo "+ corrupt dir" > loff="${leaf_lblk}" > diff --git a/tests/xfs/112 b/tests/xfs/112 > index c237f03..84f1f1d 100755 > --- a/tests/xfs/112 > +++ b/tests/xfs/112 > @@ -75,7 +75,7 @@ echo "+ check fs" > _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" > > echo "+ check dir" > -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" node > +__populate_check_xfs_dir "${inode}" node > > echo "+ corrupt dir" > loff="${node_lblk}" > diff --git a/tests/xfs/113 b/tests/xfs/113 > index 6d6e6fd..ec328bc 100755 > --- a/tests/xfs/113 > +++ b/tests/xfs/113 > @@ -75,7 +75,7 @@ echo "+ check fs" > _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" > > echo "+ check dir" > -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" btree > +__populate_check_xfs_dir "${inode}" btree > > echo "+ corrupt dir" > loff=0 > -- > 1.8.3.1 > > > > -- > 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 -- 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
On Tue, Oct 18, 2016 at 10:59:44AM +0800, Xiao Yang wrote: > We need to remove unused ${SCRATCH_DEV} when calling __populate_check_xfs_dir > because __populate_check_xfs_dir function has been rewritten by Darrick. Can you reference the commit that the relevant change was contained in, please? Darrick has made lots of changes - saying "fix something after Darrick changed something else" isn't very meaningful, especially when someone looks at the commit in a few years time. Instead, try something like this: ---- [PATCH] xfstests: remove the device parameter from __populate_check_xfs_dir In commit abcdef1234 ("rewrite the frobnozzle"), the need for callers to pass the device to populate into __populate_check_xfs_dir() was removed. We can now clean up all the callers by removing the device parameter." ---- Cheers, Dave.
On 2016/10/19 12:42, Dave Chinner wrote: > On Tue, Oct 18, 2016 at 10:59:44AM +0800, Xiao Yang wrote: >> We need to remove unused ${SCRATCH_DEV} when calling __populate_check_xfs_dir >> because __populate_check_xfs_dir function has been rewritten by Darrick. > Can you reference the commit that the relevant change was contained > in, please? Darrick has made lots of changes - saying "fix > something after Darrick changed something else" isn't very > meaningful, especially when someone looks at the commit in a few > years time. Instead, try something like this: > > ---- > [PATCH] xfstests: remove the device parameter from __populate_check_xfs_dir > > In commit abcdef1234 ("rewrite the frobnozzle"), the need for > callers to pass the device to populate into > __populate_check_xfs_dir() was removed. We can now clean up all the > callers by removing the device parameter." > ---- > > Cheers, > > Dave. Hi Darrick, Dave Sorry, it's my fault, I will rewrite this commit. Thanks, Xiao Yang -- 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
diff --git a/tests/xfs/099 b/tests/xfs/099 index ebc02dd..85193ab 100755 --- a/tests/xfs/099 +++ b/tests/xfs/099 @@ -75,7 +75,7 @@ echo "+ check fs" _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" echo "+ check dir" -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" block +__populate_check_xfs_dir "${inode}" block echo "+ corrupt dir" _scratch_xfs_db -x -c "inode ${inode}" -c 'dblock 0' -c "stack" -c "blocktrash -x 32 -y $((blksz * 8)) -z ${FUZZ_ARGS}" >> $seqres.full diff --git a/tests/xfs/100 b/tests/xfs/100 index 2412c81..3bfafce 100755 --- a/tests/xfs/100 +++ b/tests/xfs/100 @@ -75,7 +75,7 @@ echo "+ check fs" _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" echo "+ check dir" -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" leaf +__populate_check_xfs_dir "${inode}" leaf echo "+ corrupt dir" loff=0 diff --git a/tests/xfs/101 b/tests/xfs/101 index 679940f..d3abd19 100755 --- a/tests/xfs/101 +++ b/tests/xfs/101 @@ -75,7 +75,7 @@ echo "+ check fs" _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" echo "+ check dir" -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" leaf +__populate_check_xfs_dir "${inode}" leaf echo "+ corrupt dir" _scratch_xfs_db -x -c "inode ${inode}" -c "dblock ${leaf_lblk}" -c "stack" -c "blocktrash -x 32 -y $((blksz * 8)) -z ${FUZZ_ARGS}" >> $seqres.full diff --git a/tests/xfs/102 b/tests/xfs/102 index 8018a9e..cf0d7a6 100755 --- a/tests/xfs/102 +++ b/tests/xfs/102 @@ -75,7 +75,7 @@ echo "+ check fs" _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" echo "+ check dir" -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" node +__populate_check_xfs_dir "${inode}" node echo "+ corrupt dir" loff=0 diff --git a/tests/xfs/105 b/tests/xfs/105 index 27ee4b3..07ccf00 100755 --- a/tests/xfs/105 +++ b/tests/xfs/105 @@ -75,7 +75,7 @@ echo "+ check fs" _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" echo "+ check dir" -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" node +__populate_check_xfs_dir "${inode}" node echo "+ corrupt dir" loff="${leaf_lblk}" diff --git a/tests/xfs/112 b/tests/xfs/112 index c237f03..84f1f1d 100755 --- a/tests/xfs/112 +++ b/tests/xfs/112 @@ -75,7 +75,7 @@ echo "+ check fs" _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" echo "+ check dir" -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" node +__populate_check_xfs_dir "${inode}" node echo "+ corrupt dir" loff="${node_lblk}" diff --git a/tests/xfs/113 b/tests/xfs/113 index 6d6e6fd..ec328bc 100755 --- a/tests/xfs/113 +++ b/tests/xfs/113 @@ -75,7 +75,7 @@ echo "+ check fs" _scratch_xfs_repair -n >> $seqres.full 2>&1 || _fail "xfs_repair should not fail" echo "+ check dir" -__populate_check_xfs_dir "${SCRATCH_DEV}" "${inode}" btree +__populate_check_xfs_dir "${inode}" btree echo "+ corrupt dir" loff=0
We need to remove unused ${SCRATCH_DEV} when calling __populate_check_xfs_dir because __populate_check_xfs_dir function has been rewritten by Darrick. Signed-off-by: Xiao Yang <yangx.jy@cn.fujitsu.com> --- tests/xfs/099 | 2 +- tests/xfs/100 | 2 +- tests/xfs/101 | 2 +- tests/xfs/102 | 2 +- tests/xfs/105 | 2 +- tests/xfs/112 | 2 +- tests/xfs/113 | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-)