<table><tr><td style="">zzag added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D23514">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D23514#553048" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D23514#553048</a>, <a href="https://phabricator.kde.org/p/romangg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@romangg</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>You know that the show fps effect is not a benchmark, right? I mean it says so just below it.</p></div>
</blockquote>

<p>I know that, however it should not display such low numbers when there is only one static [1] client open.</p>

<p>This patch intends to move <tt style="background: #ebebeb; font-size: 13px;">present()</tt> from <tt style="background: #ebebeb; font-size: 13px;">prepareRenderingFrame()</tt> to <tt style="background: #ebebeb; font-size: 13px;">endRenderingFrame()</tt>, but it also introduces a few undesired side-effects, which should be addressed either here or in the follow-up patches. The frame rate drop is a pretty serious issue. Waiting the whole vblank interval will cause the time span during which kwin records rendering commands to drift, which may introduce undesired stuttering. The other problem is that rendering is no longer synchronized to vblank, it basically means that our timings are off.</p>

<p>[1] Meaning it's not frequently updated</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23514">https://phabricator.kde.org/D23514</a></div></div><br /><div><strong>To: </strong>romangg, KWin<br /><strong>Cc: </strong>zzag, alexeymin, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>