[PATCH] xfstests: add background noise to test 276 (btrfs backref resolving)
Jan Schmidt
list.btrfs at jan-o-sch.net
Tue Mar 19 14:17:57 CDT 2013
On 19.03.2013 18:09, Eric Sandeen wrote:
>> Furthermore, this increases two constants which make the test simply cycle a
>> few seconds longer, increasing the chance to hit on something suspicious in
>> case we broke something.
>
> Normally we don't change existing tests lest new failures look like regressions
> when they aren't, but hey, "btrfs is an experimental filesystem" so maybe it's
> ok in this case. ;) At some point when things are settled down, we wouldn't
> want to make a change like this. But for now it doesn't bother me.
(justification) I thought about adding this modification as a separate
test - and I have no strict objections against doing so. It's just that
I hate duplicating code and I couldn't think of a good way to share all
that code between two individual tests. Plus: We do need the new
version, because it shows backref walking in fact is currently broken,
while the old version would never fail where the new one wouldn't.
Thanks for taking a look!
-Jan
More information about the xfs
mailing list