Closed Bug 1501584 Opened Last year Closed Last year
Run jittests for Gecko
View on Bitbar and Packet x86-64
We don't currently run jittests on Android because of test failures in bug 1454918 and bug 1475648. We'd like to run them on our new GeckoView platforms where possible: Bitbar ARM32 (Moto G5) Bitbar ARM64 (Pixel 2) Packet x86-64 We don't need to test Android x86 if we are testing Android x86-64.
I think it is time to enable the jittests on mozilla-central for android-hw-p2. I believe we had already discussed that the pixel2s would be sufficient without the need for running on moto g5 or packet.net emulators.
Assignee: nobody → bob
Status: NEW → ASSIGNED
Attachment #9033795 - Flags: review?(jmaher)
Comment on attachment 9033795 [details] [diff] [review] bug-1501584-enable-jittests-on-mc.patch Review of attachment 9033795 [details] [diff] [review]: ----------------------------------------------------------------- nice
Attachment #9033795 - Flags: review?(jmaher) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/389460801c5c Run jittests for opt js shell for arm7, arm64 on android-hw-p2, r=jmaher.
This didn't actually result in the jit tests running. jmaher: Do you know off hand what I missed here?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I see the jit tests running on m-c: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&searchStr=android%2Cjit&revision=fda9de78ff7f1836278a7bd4b2581940883afb84
Thanks Joel. I guess skip-unless-schedules jittest in the taskgraph along with https://searchfox.org/mozilla-central/source/taskcluster/docs/optimization-schedules.rst and https://searchfox.org/mozilla-central/source/js/moz.build means only run when js/src has changed which will cut the load significantly. jmaher, cpeterson: Do you think we should go ahead and try to run on inbound and autoland now that the runs are green and we will only run when js/src changes?
oh I forgot about that, so these will only run if js/* is modified which is smart. I think we should have this on built-projects (i.e. the default rule) and that will help find regressions faster. It sounds like the js team is already aware of the skipped tests on arm64 which might be concerning.
Attachment #9033995 - Flags: review?(jmaher) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c58927f9b3c7 Run jittests for opt js shell for android-hw-p2 on built-projects,r=jmaher.
(In reply to Bob Clary [:bc:] from comment #7) > jmaher, cpeterson: Do you think we should go ahead and try to run on inbound > and autoland now that the runs are green and we will only run when js/src > changes? Since we didn't see any big problems running the subset of jittests that are green on try and central, also running them on inbound and autoland SGTM.
You need to log in before you can comment on or make changes to this bug.