Software WebRender results are mixing with regular WebRender results
Categories
(Testing :: Talos, defect, P2)
Tracking
(firefox-esr78 unaffected, firefox81 unaffected, firefox82 disabled, firefox83 disabled, firefox84 fixed)
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox81 | --- | unaffected |
firefox82 | --- | disabled |
firefox83 | --- | disabled |
firefox84 | --- | fixed |
People
(Reporter: davehunt, Assigned: sparky)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
Bug 1657769 started running a number of Talos tests with software WebRender enabled, however there's no way in Perfherder to distinguish these results from the 'regular' WebRender results.
In Raptor we're adding 'webrender' to extraOptions as a way to distinguish these results (in addition to the -qr platform suffix). Perhaps we should also include this in Talos, with a variant of 'webrender-sw'? I'm open to other suggestions, but I am keen to avoid overloading the test name with these characteristics.
Example results in Perfherder: https://treeherder.mozilla.org/perf.html#/graphs?highlightAlerts=1&series=autoland,1925804,1,1&timerange=5184000
Reporter | ||
Comment 1•4 years ago
|
||
It looks like adding a 'webrender' extra option should be simple based on what we're doing for Fission:
# differentiate fission vs non-fission results in perfherder
if browser_config.get('enable_fission', False):
talos_results.add_extra_option('fission')
We should be able to use enable_webrender
for 'regular webrender, but I'm less sure how we can detect software enabled WebRender. :ahal do you have any suggestions? Can we check the gfx.webrender.software
pref, or the presence of this on the command line?
Reporter | ||
Updated•4 years ago
|
Comment 2•4 years ago
|
||
Yes, here's the webrender-sw
definition:
https://searchfox.org/mozilla-central/source/taskcluster/taskgraph/transforms/tests.py#256
So either that pref, or you could check for -swr
in the label, or we could add a tag or something to the tags attribute. Personally I feel like checking for the pref is the safest because then even if a developer manually uses --setpref gfx.webrender.software=1
on the command line, then webrender-sw will still be detected.
Comment 3•4 years ago
|
||
Note, we do similar things in the mochitest harness:
https://searchfox.org/mozilla-central/source/testing/mochitest/runtests.py#2671
Comment 4•4 years ago
|
||
I believe T-gli is affected by this issue too.
Reporter | ||
Comment 5•4 years ago
|
||
(In reply to Matt Woodrow (:mattwoodrow) from comment #4)
I believe T-gli is affected by this issue too.
As far as I can tell, the tests in T-gli are unique to that group so I don't think we're mixing these results with others.
Comment 6•4 years ago
|
||
(In reply to Dave Hunt [:davehunt] [he/him] ⌚BST from comment #5)
(In reply to Matt Woodrow (:mattwoodrow) from comment #4)
I believe T-gli is affected by this issue too.
As far as I can tell, the tests in T-gli are unique to that group so I don't think we're mixing these results with others.
OSX 10.14 Shippable opt - T(webgl) and T-gli(webgl) had combined results on my try run. The numbers are pretty similar though, so it's not obviously bimodal.
Reporter | ||
Comment 7•4 years ago
|
||
(In reply to Matt Woodrow (:mattwoodrow) from comment #6)
(In reply to Dave Hunt [:davehunt] [he/him] ⌚BST from comment #5)
(In reply to Matt Woodrow (:mattwoodrow) from comment #4)
I believe T-gli is affected by this issue too.
As far as I can tell, the tests in T-gli are unique to that group so I don't think we're mixing these results with others.
OSX 10.14 Shippable opt - T(webgl) and T-gli(webgl) had combined results on my try run. The numbers are pretty similar though, so it's not obviously bimodal.
You're correct, here's a mix for T(webgl) and T-gli(webgl). I'll clone this bug, and we should add an extra option to distinguish webgl-ipc.
Updated•4 years ago
|
Comment 8•4 years ago
|
||
Set release status flags based on info from the regressing bug 1657769
Updated•4 years ago
|
Assignee | ||
Comment 9•4 years ago
|
||
This patch adds extra options to the perfherder data for webrender, webrender-sw (webrender-software), and webgl-ipc (to distinguish from webgl results).
Updated•4 years ago
|
Comment 10•4 years ago
|
||
Comment 11•4 years ago
|
||
bugherder |
Description
•