EGL/GLES compiz builds do a eglWaitGL (synchronous wait) on every frame, likely to slow things down
Bug #1086779 reported by
Daniel van Vugt
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Compiz |
Fix Released
|
Medium
|
Daniel van Vugt | ||
0.9.8 |
Won't Fix
|
Medium
|
Unassigned | ||
ubuntu-nexus7 |
Fix Committed
|
High
|
Daniel van Vugt | ||
compiz (Ubuntu) |
Fix Released
|
Medium
|
Daniel van Vugt |
Bug Description
I just noticed that compiz' EGL code calls eglWaitGL on every frame which appears to be pointless. We don't need and certainly don't want to synchronize on every frame. That prevents the CPU and GPU from operating in parallel, which could have a significant performance impact on some systems.
Related branches
lp:~vanvugt/compiz/fix-1086779
- Sam Spilsbury: Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 22 lines (+0/-5)1 file modifiedplugins/opengl/src/screen.cpp (+0/-5)
Changed in compiz (Ubuntu): | |
status: | New → In Progress |
importance: | Undecided → Medium |
assignee: | nobody → Daniel van Vugt (vanvugt) |
milestone: | none → ubuntu-13.04-alpha-1 |
summary: |
- EGL/GLES builds do a eglWaitGL (synchronous wait) on every frame, likely - to slow things down + EGL/GLES compiz builds do a eglWaitGL (synchronous wait) on every frame, + likely to slow things down |
tags: | added: nexus7 |
Changed in ubuntu-nexus7: | |
status: | New → In Progress |
assignee: | nobody → Daniel van Vugt (vanvugt) |
tags: | added: mobile |
Changed in compiz: | |
status: | In Progress → Fix Committed |
Changed in compiz (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in ubuntu-nexus7: | |
status: | In Progress → Fix Committed |
Changed in compiz (Ubuntu): | |
milestone: | ubuntu-13.04-alpha-1 → ubuntu-13.04-month-2 |
Changed in ubuntu-nexus7: | |
importance: | Undecided → High |
Changed in compiz: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
This bug was fixed in the package compiz - 1:0.9.9~ daily13. 01.14-0ubuntu1
--------------- 9~daily13. 01.14-0ubuntu1) raring; urgency=low
compiz (1:0.9.
[ sampo555 ] :applyDodgeTran sform() (LP:
* compiz crashed with SIGSEGV in DodgeAnim:
#1048840)
* compiz crashing if window un-/minimize animation is "Random" (LP:
#1098185)
[ Daniel van Vugt ] :getProgram( ) from GLWindowAutoPro gram::getProgra m() (LP:
* Several leaks in new GLProgram from compileProgram() from
GLScreen:
#1097644)
[ Sam Spilsbury ] ttingListAppend () ... from egrationBackend GetIntegratedSe tting() from readSetting c:375) (LP: #1097661)
* Several leaks in ccsIntegratedSe
ccsGNOMEInt
(gsettings.
[ MC Return ]
* Thumbnail Window Previews: Flickering of background/glow and window
title text (LP: #1098758)
[ Automatic PS uploader ]
* Automatic snapshot from revision 3561
-- Automatic PS uploader <email address hidden> Mon, 14 Jan 2013 04:03:09 +0000