<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 27, 2014 at 1:21 PM, Alexander Monakov <span dir="ltr"><<a href="mailto:amonakov@ispras.ru" target="_blank">amonakov@ispras.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class=""><br>
<br>
On Tue, 27 May 2014, <a href="mailto:io.github.apitrace@io7m.com">io.github.apitrace@io7m.com</a> wrote:<br>
</div><div class="">> > Actually I see why the behavior changed for you; you've probably updated from<br>
> > apitrace 4.0 to 5.0; 4.0 did not supply the application-requested GL profile<br>
> > version to the driver: <a href="https://github.com/apitrace/apitrace/issues/176" target="_blank">https://github.com/apitrace/apitrace/issues/176</a><br>
> ><br>
> > So, disregard what I said about Mesa/Xorg before; the issue and the behavior<br>
> > change is entirely on apitrace side.  The fix, I believe, would be to make X<br>
> > protocol errors not fatal in the replayer.<br>
><br>
> Ok, thanks. So, is it just a case of waiting for a fix now? I don't<br>
> think I know enough about apitrace's implementation to correctly fix<br>
> this myself.<br>
<br>
</div>You should be able to fix it yourself if you want; in retrace/glws_glx.cpp,<br>
into init(void) add a call to XSetErrorHandler with a handler function (that<br>
you'll need to write).  The handler function may be empty, as the<br>
corresponding waffle patch does:<br>
<a href="http://lists.freedesktop.org/archives/waffle/2012-November/000125.html" target="_blank">http://lists.freedesktop.org/archives/waffle/2012-November/000125.html</a><br>
<br>
(but you can also print error messages with XGetErrorText).<br>
<br>
See "man XSetErrorHandler" and "man XErrorEvent" for details.<br>
<br>
Alexandre</blockquote><div><br></div><div></div><div>Although I believe the right fix is to not attempt replaying failed calls (fix pushed as <a href="https://github.com/apitrace/apitrace/commit/7cdaf0210385b3189512d8956c9931e2eabdc858">https://github.com/apitrace/apitrace/commit/7cdaf0210385b3189512d8956c9931e2eabdc858</a> ) , I also think that having our own X error handler is a good idea too.<br>
</div></div><br></div><div class="gmail_extra">Jose<br></div></div>