Remove Hardware Composer on Gonk

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
2 years ago
2 years ago

People

(Reporter: gerard-majax, Assigned: gerard-majax)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
This is a fragile bit, very device specific, and hard to maintain. Let's take some burden off gfx team :)
(Assignee)

Comment 1

2 years ago
Okay, digging into this I could see that on Gonk we have Hwc also handling VSync. Should we also kill this ?
Flags: needinfo?(sotaro.ikeda.g)
(Assignee)

Comment 3

2 years ago
Ok, this patch builds but effectively kills any gfx rendering after the boot animation. I have no idea why :)
On recent android, hwc always exist as final stage of composition. We can not remove HwcComposer2D. If we want to remove "full hwc composition" for "GL-hwc-mixed" composition, pref "layers.composer2d.enabled" just does it. Or we could remove HwcComposer2D::TryRenderWithHwc().

Even when composition is fully done by OpenGL(Compositor), its buffer is finally handled by hwc on android framework. In this case, HwcComposer2D::Render() is used to do it.

The following is related diagram.
 https://github.com/sotaroikeda/firefox-diagrams/blob/master/widget/widget_GonkDisplayJB_FirefoxOS_3_0.pdf
Flags: needinfo?(sotaro.ikeda.g)
(In reply to Alexandre LISSY :gerard-majax from comment #1)
> Okay, digging into this I could see that on Gonk we have Hwc also handling
> VSync. Should we also kill this ?

We can not kill vsync handling. It is important for smooth rendering. It is used by VsyncSource. The following is a related diagram.
 https://github.com/sotaroikeda/firefox-diagrams/blob/master/gfx/gfx_VsyncSource_FirefoxOS_2_5.pdf
Sounds like getting rid of HWC would be more painful than keeping it around. Could we maybe just simplify it a bit?
(Assignee)

Updated

2 years ago
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.