Closed Bug 1784453 Opened 2 years ago Closed 2 years ago

Prefer GLES by default on ARM

Categories

(Core :: Graphics: WebRender, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
105 Branch
Tracking Status
firefox105 --- fixed

People

(Reporter: rmader, Assigned: rmader)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

On most ARM devices the integrated GPUs and their drivers are usually better optimized for GLES than desktop GL. Thus lets prefer GLES on ARM builds, potentially reducing bug and improving performance.

In case of v3d this is also needed to turn on HW-WR by default as we disable it due to the first default GL context not supporting GL/GLES >= 3.0.

The Bugbug bot thinks this bug should belong to the 'Core::Graphics: WebRender' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: General → Graphics: WebRender

On most ARM devices the integrated GPUs and their drivers are
usually better optimized for GLES than desktop GL. Thus lets
prefer GLES on ARM builds, potentially reducing bug and improving
performance.

In case of v3d this is also needed to turn on HW-WR by default
as we disable it due to the first default GL context not supporting
GL >= 3.0.

Notes:

  • Unfortunately we can't straightly use StaticPrefs in glxtest
  • We need to ask for a 2.x context as GLES defaults to 1.x. Asking
    for 2.x should, however, should give us a 3.x/4.x one if supported.
Assignee: nobody → robert.mader
Status: NEW → ASSIGNED
Pushed by robert.mader@posteo.de:
https://hg.mozilla.org/integration/autoland/rev/bb15320dee97
Prefer GLES by default on Linux-ARM, r=gfx-reviewers,nical
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 105 Branch

This didn't work as planned - _ARM64_ is apparently Windows only 🤦

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

_ARM64_ appears to be a Windows-only thing.

Fixes D154506

Pushed by robert.mader@posteo.de:
https://hg.mozilla.org/integration/autoland/rev/51722c5665a0
Use correct compiler macro for ARM builds,
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: