<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 15, 2018 at 8:31 AM, Michel Dänzer <span dir="ltr"><<a href="mailto:michel@daenzer.net" target="_blank">michel@daenzer.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2018-06-15 05:25 PM, Jason Ekstrand wrote:<br>
> On June 15, 2018 01:14:24 Michel Dänzer <<a href="mailto:michel@daenzer.net">michel@daenzer.net</a>> wrote:<br>
>> On 2018-06-15 07:31 AM, Jason Ekstrand wrote:<br>
>>><br>
</span><span class="">>>> I did some testing and x11perf -copywinwin500 is... exactly the same<br>
>>> with<br>
>>> or without my patches.  If anything they might improve it by just a<br>
>>> hair.<br>
>><br>
>> Possible explanations I can think of:<br>
>><br>
>> 1. Your glamor still has its own FBO cache. Which version of xserver are<br>
>> you testing with?<br>
>><br>
</span>> 1.19 I think<br>
<br>
Okay, that doesn't have the glamor FBO cache anymore.<br>
<span class=""><br>
<br>
>> 2. The i965 driver cache isn't hit even before these changes.<br>
> <br>
> It's definitely getting hit in both cases, it just may require a<br>
> slightly larger cache of we aren't recycling BOs until they're idle.<br>
<br>
</span>It might be more than just slightly, -copywinwin500 can queue many<br>
overlapping copies between flushes. Can you compare the maximum total<br>
cache size with and without this series?<br></blockquote><div><br></div><div>That should be possible.  I'll see what I can do.  It's always a bit annoying getting stuff out of X because printf gets weird but I should be able to come up with something. <br></div></div></div></div>