<div dir="ltr">Hi Dave<br><div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jun 29, 2016 at 11:00 AM Dave Airlie <<a href="mailto:airlied@gmail.com">airlied@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Rob (and list),<br>
<br>
I finally found some time to look at the OA performance under PTS, and<br>
tracked it down to the demo level PTS uses having some flares in it<br>
and the OA flare code doing 8 individual 1x1 readpixels on the depth<br>
buffer. This is clearly not a great idea, and with virgl is an extra<br>
bad idea.<br>
<br>
I've created a mesa branch:<br>
<a href="https://cgit.freedesktop.org/~airlied/mesa/log/?h=virgl-readpixels-sync" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~airlied/mesa/log/?h=virgl-readpixels-sync</a><br>
<br>
That attempts to fix this, the branch does better tracking of resource<br>
reading/writing, and then when it sees a transfer for a 2D resource<br>
that is out of sync with the host copy, it does a complete surface<br>
sync and then doesn't readback from the host for any subsequent<br>
transfers until the surface is dirtied again.<br>
<br>
I haven't ran this through a complete piglit run and I'm not sure it<br>
won't destroy some other apps speed, but it would be good if anyone<br>
has time to give it a spin with some other apps/games.<br></blockquote><div><br></div>When running piglit, I noticed half the tests failed, crashing. I fixed your patches (correcting offset, and fixing vtest),I ran piglit:<br><br>with: [52799/52799] skip: 34638, pass: 17661, warn: 2, fail: 465, crash: 33 without: [52799/52799] skip: 34638, pass: 17646, warn: 2, fail: 479, crash: 34<br><br>Also did a naive measurement on the piglit run time takes ~25% less with your patch.<br><br>My fixed branch is:<br><a href="https://github.com/elmarco/mesa">https://github.com/elmarco/mesa</a> airlied<br><div><br></div><div><br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Dave.<br>
_______________________________________________<br>
virglrenderer-devel mailing list<br>
<a href="mailto:virglrenderer-devel@lists.freedesktop.org" target="_blank">virglrenderer-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/virglrenderer-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/virglrenderer-devel</a><br>
</blockquote></div></div></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr">Marc-André Lureau<br></div></div>