Run acceptance suite with -Dgcthreshold and -load to stress the GC

VERIFIED FIXED in Q3 11 - Serrano

Status

Tamarin
Build Config
P4
normal
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: Brent Baker, Assigned: Brent Baker)

Tracking

unspecified
Q3 11 - Serrano
Bug Flags:
flashplayer-qrb +
flashplayer-triage +

Details

(Whiteboard: buildbot)

Attachments

(3 attachments)

(Assignee)

Description

7 years ago
Add a run of the acceptance suite in the deep phase of the build on a release and debug-debugger shells using the threshold and load switches for the GC to increase the number of collection cycles that happen.

Using the normal default values for collection threshold and gc load factors, there are only a couple hundred full collections happening when running the entire acceptance suite. However using the new gcthreshold switch, to lower the threshold before collections start, and changing the load factor to grow the heap slowly, we can dramatically increase the number of collection cycles happening.

Switch combinations:
[1] -Dgcthreshold 128
[2] -Dgcthreshold 128 -load 1.05,1,1.05,5,1.05,20

Release build (mac64-intel):
[1] gccount =>  2939
[2] gccount => 18280

Debug-Debugger build (mac64-intel):
[1] gccount =>  5181
[2] gccount => 86597
Flags: flashplayer-triage+
Flags: flashplayer-qrb?
(Assignee)

Comment 1

7 years ago
Created attachment 489146 [details]
Runner script

This is the simple script I was using the gather the gc collection cycles
(Assignee)

Comment 2

7 years ago
This is gc collection tracking of the 32bit shell, which normally does not allocate enough memory during an acceptance run in order to cross over the gcthreshold.

Switch combinations:
[0] default values, no switches
[1] -Dgcthreshold 128
[2] -Dgcthreshold 128 -load 1.05,1,1.05,5,1.05,20

mac32-intel release:
[0] gccount ==>   397
[1] gccount ==>  2781
[2] gccount ==> 14762

mac32-intel debug-debugger:
[0] gccount ==>  1730
[1] gccount ==>  3234
[2] gccount ==> 85491
(Assignee)

Comment 3

7 years ago
Created attachment 495814 [details] [diff] [review]
run aggressive gc collection in deep phase

Run acceptance suite with release and debug-debugger builds in the deep phase with aggressive gc load policy and a much smaller collection threshold. This will cause the gc to start running collections much sooner and will cause many more collections to happen as the heap is expanded very slowly.
Attachment #495814 - Flags: review?(dschaffe)
(Assignee)

Comment 4

7 years ago
Created attachment 495815 [details] [diff] [review]
Add arm-linux buildslave for more deep testing

Add an idle beagle-board to the deep phase machines and run acceptance on the slaves with aggressive gc settings.
Attachment #495815 - Flags: review?(dschaffe)
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

7 years ago
Flags: flashplayer-qrb?

Updated

7 years ago
Attachment #495814 - Flags: review?(dschaffe) → review+

Updated

7 years ago
Attachment #495815 - Flags: review?(dschaffe) → review+

Comment 5

7 years ago
changeset: 5701:743c95daf6ef
user:      Brent Baker <brbaker@adobe.com>
summary:   Bug 610656: add gcthreshold and gcload testing into the deep phase of the build (r+dschaffe)

http://hg.mozilla.org/tamarin-redux/rev/743c95daf6ef

Updated

7 years ago
Assignee: nobody → brbaker
Flags: flashplayer-qrb+
Priority: -- → P4
Target Milestone: --- → flash10.x-Serrano
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.