<table><tr><td style="">subdiff retitled this revision from "[DrmBackend] dic EGL surface buffer release after page flip" to "[DrmBackend] Straighten out EGL surface buffer release logic on page flip".<br />subdiff edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-myeuulsoamy3ehn/" rel="noreferrer">(Show Details)</a>
</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/D5060" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">~~</span>Currently the EGL surface buffer is released while the buffer is still being scanned out to the display. This is not the correct behavior, since it should be only released after the buffer is not scanned out anymore  (i.e.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> the next buffer was page flipped).~~<br />
I just realized the above isn't true, since we always ran into the first branch of pageFlippedBufferRemover, if the buffer uses a surface.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the next buffer was page flipped)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Still imo the patched code makes more sense structurally and I'm wondering how the display removal has worked before</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5060" rel="noreferrer">https://phabricator.kde.org/D5060</a></div></div><br /><div><strong>To: </strong>subdiff, KWin<br /><strong>Cc: </strong>kwin, plasma-devel, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol<br /></div>