Merge lp://qastaging/~vanvugt/unity/fix-861061-trunk into lp://qastaging/unity
Proposed by
Daniel van Vugt
Status: | Merged |
---|---|
Merged at revision: | 1816 |
Proposed branch: | lp://qastaging/~vanvugt/unity/fix-861061-trunk |
Merge into: | lp://qastaging/unity |
Diff against target: |
60 lines (+13/-4) 4 files modified
plugins/unityshell/src/BackgroundEffectHelper.cpp (+9/-0) plugins/unityshell/src/BackgroundEffectHelper.h (+1/-1) plugins/unityshell/src/PanelView.cpp (+1/-2) plugins/unityshell/src/unityshell.cpp (+2/-1) |
To merge this branch: | bzr merge lp://qastaging/~vanvugt/unity/fix-861061-trunk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sam Spilsbury (community) | Approve | ||
Unity Team | Pending | ||
Review via email:
|
Description of the change
Fix major performance regressions due to unnecessary UnityFBO binding (LP: #861061) (LP: #880707)
UnityFBO was being bound even when not required. This caused major lag in glPaintOutput, which slowed down all rendering. This was seen in reduced framerates in apps (LP: #861061) and significantly worse screen tearing with Unity 4.x compared to 3.x (LP: #880707).
To post a comment you must log in.
(please excuse me, I don't know this code very well and I only looked
at the diff)
Does the ref handling not break now that bind() is conditionalized? It
would seem that you always unbind, but only conditionally bind the
fbo.