Closed
Bug 1120753
Opened 11 years ago
Closed 10 years ago
Fallback to software vsync if hardware vsync fails to initialize on b2g
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: mchang, Assigned: mchang)
References
Details
Attachments
(1 file, 2 obsolete files)
3.71 KB,
patch
|
mchang
:
review+
bajaj
:
approval-mozilla-b2g37+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #1119850 +++
If hardware vsync fails to initialize, fallback and use software vsync. This is required for the b2g-emulators which have no HwcComposer but still use gfxAndroidPlatform. In the cases where vsync + compositor aligned vsync are enabled but the HwcComposer fails to initialize, we won't be able to composite anything.
Assignee | ||
Comment 1•11 years ago
|
||
Requesting blocking-b2g 2.2 as this is required to enable silk by default, bug 987532.
blocking-b2g: --- → 2.2?
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8548321 -
Flags: review?(bugmail.mozilla)
Assignee | ||
Updated•11 years ago
|
Summary: Fallback to software vsync is hardware vsync fails to initialize on b2g → Fallback to software vsync if hardware vsync fails to initialize on b2g
Updated•11 years ago
|
Attachment #8548321 -
Flags: review?(bugmail.mozilla) → review+
Assignee | ||
Updated•11 years ago
|
blocking-b2g: 2.2? → ---
Assignee | ||
Updated•11 years ago
|
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → ARM
Assignee | ||
Comment 3•11 years ago
|
||
Rebased on master. Also found a bug on a flame device where we initialize the gfxPlatform before we initialize the HwcComposer vsync callbacks. Hardware vsync wouldn't be enabled yet and so we'd fallback to software vsync. This version initializes the hardware vsync callback when we create the HwcComposer instead of when we initialize the HwcComposer. Initialization of the HwcComposer occurs once we have a glcontext, which is later in the startup process.
Attachment #8548321 -
Attachment is obsolete: true
Attachment #8549034 -
Flags: review?(bugmail.mozilla)
Comment 4•11 years ago
|
||
Comment on attachment 8549034 [details] [diff] [review]
Fallback to Software Vsync. v2
Review of attachment 8549034 [details] [diff] [review]:
-----------------------------------------------------------------
r? mwu for HwcComposer2D changes.
::: gfx/thebes/VsyncSource.h
@@ -45,5 @@
> // Large parts of Gecko assume TimeStamps should not be in the future such as animations
> virtual void NotifyVsync(TimeStamp aVsyncTimestamp);
>
> nsRefPtr<RefreshTimerVsyncDispatcher> GetRefreshTimerVsyncDispatcher();
> -
nit: whitespace deletion
@@ -62,5 @@
> };
>
> void AddCompositorVsyncDispatcher(CompositorVsyncDispatcher* aCompositorVsyncDispatcher);
> void RemoveCompositorVsyncDispatcher(CompositorVsyncDispatcher* aCompositorVsyncDispatcher);
> -
nit: whitespace deletion
Attachment #8549034 -
Flags: review?(mwu)
Attachment #8549034 -
Flags: review?(bugmail.mozilla)
Attachment #8549034 -
Flags: review+
Updated•11 years ago
|
Attachment #8549034 -
Flags: review?(mwu) → review+
Assignee | ||
Comment 5•11 years ago
|
||
Carrying r+, updated without whitespace deletions.
Attachment #8549034 -
Attachment is obsolete: true
Attachment #8549141 -
Flags: review+
Assignee | ||
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Assignee | ||
Comment 8•10 years ago
|
||
Comment on attachment 8549141 [details] [diff] [review]
Fallback to Software Vsync. v3
Approval Request Comment
[Feature/regressing bug #]: Required for silk, bug 987532
[User impact if declined]: Once silk is enabled by default, mochitests will fail to execute.
[Describe test coverage new/current, TBPL]: Without this patch, all tests will fail with silk enabled by default since hardware vsync does not work on the emulators.
[Risks and why]: Low - Currently only used if silk is enabled, which is currently prefed off.
[String/UUID change made/needed]: None
Attachment #8549141 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8549141 [details] [diff] [review]
Fallback to Software Vsync. v3
From https://bugzilla.mozilla.org/show_bug.cgi?id=1121065#c8, asking for b2g37 approval instead. Approval request in comment 8.
Attachment #8549141 -
Flags: approval-mozilla-aurora? → approval-mozilla-b2g37?
Updated•10 years ago
|
Attachment #8549141 -
Flags: approval-mozilla-b2g37? → approval-mozilla-b2g37+
Comment 10•10 years ago
|
||
status-b2g-v2.2:
--- → fixed
status-b2g-master:
--- → fixed
status-firefox36:
--- → wontfix
status-firefox37:
--- → wontfix
status-firefox38:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•