Closed
Bug 1388833
Opened 7 years ago
Closed 7 years ago
Make sure all linux64-qr tests are running with HWA enabled
Categories
(Core :: Graphics: WebRender, enhancement, P3)
Tracking
()
RESOLVED
FIXED
mozilla57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
(Whiteboard: [gfx-noted])
Attachments
(2 files, 1 obsolete file)
As of bug 1387764, WebRender will be disabled if HWA is disabled. Since Firefox on Linux has HWA disabled by default, that means WebRender is going to be disabled by default. And that in turn means all the linux64-qr test suites that don't have HWA enabled will not actually be running webrender!
We should fix this, otherwise we will basically only be left with reftest coverage (thanks to bug 1388726) and all the other suites will be running without webrender.
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Comment 1•7 years ago
|
||
We can enable HWA either by setting layers.acceleration.force-enabled to true, or setting MOZ_ACCELERATED=1. I think the latter is easier since we can just plop it wherever we are setting MOZ_WEBRENDER=1.
Assignee | ||
Comment 2•7 years ago
|
||
Assignee | ||
Comment 3•7 years ago
|
||
And actually we can stop running the "reftest-no-accel" on linux64-qr as well, because they are pointless. Either webrender will be enabled, and it'll be the same as the accelerated reftests, or webrender won't be enabled and it will be the same as the non-qr linux64 reftest-no-accel.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•7 years ago
|
||
@ahal: I'd have flagged jmaher for review on the first patch but he's on PTO and not accept review requests. It's a pretty simple change though.
Comment 7•7 years ago
|
||
mozreview-review |
Comment on attachment 8895515 [details]
Bug 1388833 - Force-enable hardware acceleration for test suites that enable WebRender, because HWA is now a WR requirement.
https://reviewboard.mozilla.org/r/166720/#review171904
Lgtm!
Attachment #8895515 -
Flags: review?(ahalberstadt) → review+
Assignee | ||
Comment 8•7 years ago
|
||
In the interest of not leaving WR testing broken for too long I'm going to land the first patch now while waiting for review on the second patch.
Assignee | ||
Updated•7 years ago
|
Attachment #8895516 -
Attachment is obsolete: true
Attachment #8895516 -
Flags: review?(dustin)
Assignee | ||
Updated•7 years ago
|
Keywords: leave-open
Version: Other Branch → 57 Branch
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/96172895b826
Force-enable hardware acceleration for test suites that enable WebRender, because HWA is now a WR requirement. r=ahal
Assignee | ||
Comment 10•7 years ago
|
||
Attachment #8895800 -
Flags: review?(dustin)
Assignee | ||
Updated•7 years ago
|
Attachment #8895515 -
Flags: checkin+
Updated•7 years ago
|
Attachment #8895800 -
Flags: review?(dustin) → review+
Comment 11•7 years ago
|
||
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4e1d0438c31b
Stop running reftest-no-accel on linux64-qr builds because they are redundant. r=dustin
Comment 12•7 years ago
|
||
bugherder |
Assignee | ||
Updated•7 years ago
|
Keywords: leave-open
Comment 13•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in
before you can comment on or make changes to this bug.
Description
•