yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

kwin: Don't call glFlush() unnecessarily in endRenderingFrame()

Browse source 
We should only call glFlush() here if we're deferring the posting
of the back buffer to the next prepareRenderingFrame() call.
This commit is contained in:
Fredrik Höglund 2013-11-21 10:38:16 +01:00
parent d5e439aae0
commit 12446c9b4c
2 changed files with 16 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
eglonxbackend.cpp
View file

@ -338,9 +338,15 @@ void EglOnXBackend::prepareRenderingFrame()
void EglOnXBackend::endRenderingFrame(const QRegion &damage)
{
setLastDamage(damage);
glFlush();
if (!blocksForRetrace()) {
present(); // this sets lastDamage emtpy and prevents execution from prepareRenderingFrame()
// This also sets lastDamage to empty which prevents the frame from
// being posted again when prepareRenderingFrame() is called.
present();
} else {
// Make sure that the GPU begins processing the command stream
// now and not the next time prepareRenderingFrame() is called.
glFlush();
}
if (overlayWindow()->window()) // show the window only after the first pass,

10
glxbackend.cpp
View file

@ -511,9 +511,15 @@ void GlxBackend::prepareRenderingFrame()
void GlxBackend::endRenderingFrame(const QRegion &damage)
{
setLastDamage(damage);
glFlush();
if (!blocksForRetrace()) {
present(); // this sets lastDamage emtpy and prevents execution from prepareRenderingFrame()
// This also sets lastDamage to empty which prevents the frame from
// being posted again when prepareRenderingFrame() is called.
present();
} else {
// Make sure that the GPU begins processing the command stream
// now and not the next time prepareRenderingFrame() is called.
glFlush();
}
if (overlayWindow()->window()) // show the window only after the first pass,