Message ID | 20111214112209.1002.61FB500B@jp.fujitsu.com |
---|---|
State | Not Applicable, archived |
Headers | show |
On Wed, Dec 14, 2011 at 11:22:10AM +0900, Masayoshi MIZUMA wrote: > > test 068 will catch problems like the one your specific test > > catches, but maybe not every time. Test 068 will catch problems your > > test case won't, though - it's a trade-off between having lots of > > tests that are similar but slightly different (difficult to > > maintain, increases runtime, etc), and having one test that > > exercises the functionality in a simple manner likely to detect > > problems. > > Thank you for your explanation about the policy and I understand it. > I tried to reproduce the problem not dropping the sleep, but the problem was > not reproduced... Therefore, I dropped it and the problem was reproduced. > > However, as you mentioned, the problem is a timing proglem, so the > my reproduction might be just by chance. Dropping sleep may increase > the possibility of the reproduction, but not every time, so the change > is not good. That is same for the arguments of fsstress which I changed. > > OK, I update 068 just to run other filesystem, ext3, ext4 and btrfs which > I confirmed xfs_freeze works on. > (xfs_freeze may work on the other filesystems which have freeze_fs/unfreeze_fs > super_operations but I don't confirm...) > > The patch is below. Given that MIZUMAs patch reproduces a real life issue I think adding his original patch in addition to this change would be a good idea. Dave, do you have a strong opinion against that? -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Thanks, applied. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" 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/068 b/068 index 5d0053d..6f08f18 100755 --- a/068 +++ b/068 @@ -51,7 +51,7 @@ trap "_cleanup" 0 1 2 3 15 . ./common.filter # real QA test starts here -_supported_fs xfs +_supported_fs btrfs ext3 ext4 xfs _supported_os Linux IRIX _require_scratch @@ -62,7 +62,7 @@ rm -f $seq.full umount $SCRATCH_DEV >/dev/null 2>&1 echo "*** MKFS ***" >>$seq.full echo "" >>$seq.full -_scratch_mkfs_xfs >>$seq.full 2>&1 \ +_scratch_mkfs >>$seq.full 2>&1 \ || _fail "mkfs failed" _scratch_mount >>$seq.full 2>&1 \ || _fail "mount failed"