Bug 69207 - Bencher script makes it difficult to do automated performance testing
Summary: Bencher script makes it difficult to do automated performance testing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 69208
  Show dependency treegraph
 
Reported: 2011-10-01 00:40 PDT by Filip Pizlo
Modified: 2011-10-01 15:01 PDT (History)
2 users (show)

See Also:


Attachments
the patch (12.88 KB, patch)
2011-10-01 00:50 PDT, Filip Pizlo
sam: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Filip Pizlo 2011-10-01 00:40:44 PDT
The Tools/Scripts/bencher script simplifies performance testing.  Sometimes it would be useful to use bencher itself within a script, and read out the results - for example when varying some configuration parameter of a particular build of a VM.  But bencher has no way of computing a single aggregate result for all benchmark suites, and no way of printing it.  It also gets confused if a VM is modified to print additional output beyond jsc's print statement, which makes it impossible to use bencher with certain profiling features enabled.
Comment 1 Filip Pizlo 2011-10-01 00:50:40 PDT
Created attachment 109402 [details]
the patch
Comment 2 Chris Rogers 2011-10-01 10:48:52 PDT
Hi Filip, I'm not sure why this issue is associated with the "Web Audio" component?
Comment 3 Sam Weinig 2011-10-01 13:24:14 PDT
(In reply to comment #2)
> Hi Filip, I'm not sure why this issue is associated with the "Web Audio" component?

My guess is off by one from Tools/Tests.  Fixed.
Comment 4 Filip Pizlo 2011-10-01 14:52:42 PDT
(In reply to comment #3)
> (In reply to comment #2)
> > Hi Filip, I'm not sure why this issue is associated with the "Web Audio" component?
> 
> My guess is off by one from Tools/Tests.  Fixed.

Thanks!
Comment 5 Filip Pizlo 2011-10-01 15:01:59 PDT
Landed in r96462.