<table><tr><td style="">davidedmundson updated this revision to Diff 73201.<br />davidedmundson retitled this revision from "[scenes/opengl] Set current context when resetting common vertex buffer" to "[scenes/opengl] Remove outdated hack to reset vertex buffers".<br />davidedmundson edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-zpdbr5tfy5vyvjy/">(Show Details)</a><br />davidedmundson removed a subscriber: zzag.<br />davidedmundson 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/D26556">View Revision</a></tr></table><br /><div><div><p>Just remove the whole thing</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>handlers for an error calls scheduleVboReInit <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- the comment next to it</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">the history shows it was a<br />
forerunner to the GLX_NV_robustness_video_memory_purge but resetting</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">explicitly mentions Nvidia</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">only one tiny part based on debug output</span>.<div style="padding: 8px 0;">...</div>GLVertexBuffer will be deleted but not correctly re-initialised.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
We have two very common crashes with a corrupted<br />
GLVertexBuffer::streamingBuffer() which would match up perfectly.</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">We have two very common crashes with a corrupted GLVertexBuffer::streamingBuffer()..</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Given that we now have a proper mechanism to reset the entire scene, we<br />
don't need this timer based hack and resolve that problem</span>.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">CC</span>BUG: 399499<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">CC</span>BUG: 372305</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D26556?vs=73180&id=73201">https://phabricator.kde.org/D26556?vs=73180&id=73201</a></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26556">https://phabricator.kde.org/D26556</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/scenes/opengl/scene_opengl.cpp</div></div></div><br /><div><strong>To: </strong>davidedmundson, KWin<br /><strong>Cc: </strong>kwin, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart, zzag<br /></div>