1544648 - Add remaining GC parameters to gcparam()

Status: RESOLVED FIXED

(Core :: JavaScript: GC, task, P3)

Description

[Paul Bone [:pbone]]

The gcparam() testing function is missing a few of the GC parameters. Add the remaining ones.

Comment 1

[Paul Bone [:pbone]]

Attachment: Bug 1544648 - Add remaining GC parameters r?jonco

Depends on D27642

Comment 2

[Paul Bone [:pbone]]

Hi Fuzzers,

I'm adding new parameters to gcparam(), Please be gentle / feel free to NI me on anything you find, as you did last time I did this.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2f61be416873
Add remaining GC parameters r=jonco

Comment 4

[Narcis Beleuzu [:NarcisB]]

Backed out for bustages on Nursery.cpp

Backout link: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&selectedJob=240959510&revision=2f61be416873860b79506382e4bfc2e42fe87a97
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=240959510&repo=autoland&lineNumber=6308

Comment 5

[Paul Bone [:pbone]]

I'm attempting to re-land this without other patches in its stack. The other patches caused the bustage.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a51a461f3df3
Add remaining GC parameters r=jonco

Comment 7

[Paul Bone [:pbone]]

NI for myself to create some examples for how to use these parameters, what there "suitable" ranges are and what behaviours they change.

Comment 8

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/a51a461f3df3

Comment 9

[Paul Bone [:pbone]]

I've updated the documentation for these parameters at: https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey/JSAPI_reference/JS_GetGCParameter