Closed
Bug 920221
Opened 12 years ago
Closed 12 years ago
Add --bootstrap and --enable-privilege arguments for android x86 reftests
Categories
(Infrastructure & Operations Graveyard :: CIDuty, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gbrown, Assigned: gbrown)
References
Details
Attachments
(1 file)
|
1.25 KB,
patch
|
armenzg
:
review+
|
Details | Diff | Splinter Review |
Two command line arguments:
--bootstrap
--enable-privilege
are used for Android 2.2 reftests (jsreftests, crashtests, and plain-reftest) but not for Android 4.0 plain-reftests and not for Android x86 (all reftests).
:jmaher recommends using both arguments.
Preliminary tests suggest that running with these arguments on x86 significantly decreases plain-reftest run time and does not cause any additional failures.
dminor is going to follow-up for Android 4.0/Panda.
I am running more tests on the loaner.
| Assignee | ||
Comment 1•12 years ago
|
||
Attachment #809414 -
Flags: review?(armenzg)
Updated•12 years ago
|
Attachment #809414 -
Flags: review?(armenzg) → review+
| Assignee | ||
Comment 2•12 years ago
|
||
| Assignee | ||
Comment 3•12 years ago
|
||
Comparing tests before and after check-in at https://tbpl.mozilla.org/?tree=Cedar&rev=1f3ab05e73e0, there is some improvement in run-time, but it doesn't seem consistent, and not as significant as I saw on the loaner:
crashtest-1 0:25:11.874631 -> 0:23:31.292969
jsreftest-1 0:46:08.039488 -> 0:52:50.745696
reftest-1 1:00:52.810226 (705 / 1213) -> 1:00:51.258885 (743 / 1213)
| Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 4•12 years ago
|
||
Why --enable-privilege is needed? It is no-op since bug 913510.
Updated•7 years ago
|
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Updated•6 years ago
|
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•