ARM port of testsuite and general testsuite fixes
David Smith
dsmith@redhat.com
Wed Jun 6 16:02:00 GMT 2007
Quentin Barnes wrote:
> The ARM version of the test suite of 20070602 just completed. I'll
> post the specifics later. With the earlier patches plus the patch
> below, here's the summary using a 2.6.21.1 kernel in case anyone's
> curious:
>
> =======
> === systemtap Summary ===
>
> # of expected passes 277
> # of unexpected failures 37
> # of unexpected successes 1
> # of expected failures 129
> # of known failures 7
> # of untested testcases 30
> # of unsupported tests 3
> =======
Of course it is amazing that you got this working at all, but 37
failures is still quite high.
> Below is the patch necessary to fix all timeout problems when running
> on an ARM processor and other porting issues related to ARM.
>
> I didn't go with a strategy that makes the hardcoded timeout values
> variable. Once I fixed various testsuite bugs, the timeout values
> didn't increase all that much after all. I felt the difference
> wasn't enough warrant switching to a new scheme for. If people feel
> otherwise, we should discuss it further.
Hmm. Looking over the timeout value changes/additions, I see several
different values:
20 (1 instance)
60 (1 instance)
120 (4 instances)
150 (17 instances)
180 (3 instances)
240 (4 instances)
360 (1 instance)
400 (1 instance)
1800 (1 instance)
It seems like we ought to standardize a bit and have 4 (or some other
number) standard timeout values (that we could customize per platform if
needed). I think you (or someone else) suggested this earlier.
(Note that I don't have a problem with you checking your patch in as is
and then we can go back later and improve things.)
> There are various bug fixes scattered throughout the patch. Please
> review them carefully, but most should be self-explanatory.
One question I have is that you changed several instances of "." to "\."
in regular expressions. Out of curiosity did you actually see a problem
here or were you just cleaning up?
--
David Smith
dsmith@redhat.com
Red Hat
http://www.redhat.com
256.217.0141 (direct)
256.837.0057 (fax)
More information about the Systemtap
mailing list