<div dir="ltr">Patch 4 and 5 can be dropped, I looked at Keith's code and it does the same thing already (also tested OpenTTD and it works with Keith's code).  Patch 1 is still valid.<br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Apr 23, 2014 at 5:54 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">
AFAICT these are all the relevant fixes which went into the standalone<br>
glamor tree since it was imported into the xserver tree.<br>
<br>
Patch 4 involves changes to the PutImage path which conflict with Keith's<br>
pending rework of that, but the other parts of it are required for patch<br>
5 AFAICT. Patch 5 in turn could be dropped in favour of Keith's 'Add<br>
glamor_program based copy acceleration' change in his glamor-server<br>
branch. But either that or patch 5 is necessary e.g. to make the game<br>
OpenTTD playable (at least on Radeons).<br>
<br>
I've only lightly tested these so far.<br>
<br>
[PATCH 1/5] glamor: Fix coordinates handling for composite<br>
[PATCH 2/5] glamor: Fix memory leak in _glamor_copy_n_to_n()<br>
[PATCH 3/5] glamor: Fallback to system memory when fail to allocate<br>
[PATCH 4/5] glamor: Add in support for the stride parameter when<br>
[PATCH 5/5] glamor: Optimization to copy area when the source doesn't<br>
_______________________________________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" target="_blank">http://lists.x.org/archives/xorg-devel</a><br>
Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel" target="_blank">http://lists.x.org/mailman/listinfo/xorg-devel</a><br>
</blockquote></div><br></div>