Closed Bug 1611193 Opened 5 years ago Closed 5 years ago

Limit Nvidia WebRender laptop on beta to latest windows and latest nvidia drivers

Categories

(Core :: Graphics: WebRender, enhancement, P1)

73 Branch
Desktop
Windows 10
enhancement

Tracking

()

VERIFIED FIXED
mozilla74
Tracking Status
firefox72 --- disabled
firefox73 --- verified
firefox74 --- verified

People

(Reporter: jrmuizel, Assigned: aosmond)

References

Details

Attachments

(1 file)

This will minimize our risks here

Assignee: nobody → aosmond
Status: NEW → ASSIGNED
Priority: -- → P2
Priority: P2 → P1
OS: Unspecified → Windows 10
Hardware: Unspecified → Desktop
Version: unspecified → 73 Branch

try on central: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1ed80ec483fa0a2d01e030e3fbde67940da1b102

I manually verified this on our reference machine, forcing it to believe it has a battery to ensure we got WR.

try on beta: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ee7385bf3c97b9a1b883e1e1a59d4b6f3448d24f

Pushed by aosmond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0fc48050e27a Allow WebRender on late beta/release for recent Windows 10 builds and NVIDIA drivers. r=jrmuizel

Comment on attachment 9123832 [details]
Bug 1611193 - Allow WebRender on late beta/release for recent Windows 10 builds and NVIDIA drivers.

Beta/Release Uplift Approval Request

  • User impact if declined: WebRender will not be enabled by default for users with a battery, on the latest Windows 10 build, with the latest NVIDIA drivers.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): I verified the patch on top of mozilla/central with a local build; it hasn't been verified in nightly because it hasn't made it into central yet. If the windows/driver version is too old, then it leaves WebRender disabled.

It is not risky because it is a tweak to our WebRender decision criteria. We already ran earlier in beta with the same users and now we are selecting for a smaller subset. We are already shipping WebRender to make NVIDIA users with success.

  • String changes made/needed: N/A
Attachment #9123832 - Flags: approval-mozilla-beta?
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74

I think QA should try to verify this fix if they have the right hardware and software combo to do so.

Flags: qe-verify+

Comment on attachment 9123832 [details]
Bug 1611193 - Allow WebRender on late beta/release for recent Windows 10 builds and NVIDIA drivers.

Limits the WebRender rollout to systems with newer Windows versions and nVidia drivers. Approved for 73.0RC1.

Attachment #9123832 - Flags: approval-mozilla-beta? → approval-mozilla-release+
QA Whiteboard: [qa-triaged]

Hello,

I can confirm that webrender is turned off by default for laptops with old nvidia drivers ( August 20, 2019 in this case) on 73.0RC1.

Should this bug be verified on 74 Nightly also?

Flags: needinfo?(aosmond)

Please do, thanks.

Flags: needinfo?(aosmond)

Daniel, can you also confirm that it's enabled with new Nvidia drivers on 73.0RC1?

Flags: needinfo?(daniel.cicas)

Hello,

So I can confirm that with a new driver Webrender is enabled by default on 73.0RC1.

But on Fx 74.0a1 (BuildID: 20200205055433) with an old driver, Webrender is still enabled by default at start-up.

Flags: needinfo?(daniel.cicas) → needinfo?(aosmond)

I had to look at the patch again to see what we agreed :). We still allow old drivers on nightly, as this isn't a driver bug concern, so much as a slow roll out / minimize risk concern. So as expected, thanks!

Flags: needinfo?(aosmond)

Thank you for looking in to it, marking the bug accordingly.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

Not sure if this is the right place to call this out, but the Firefox 73 release notes doesn't specify that this is Windows only - a user noticed that discrepancy here: https://www.reddit.com/r/firefox/comments/f28sec/firefox_730_see_all_new_features_updates_and_fixes/fhc77l6/

Flags: needinfo?(jbonisteel)

Ah good catch, I will see if we can edit that.

Flags: needinfo?(jbonisteel)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: