<p dir="ltr"><br>
Hi Jonas,</p>
<p dir="ltr">the reason I am asking this is that I'm trying to push the patch into the Mageia xlib package - mainly because it is the candidate for putting realtime applications  the next coming 6/7 years. The requirements are quite strong: it needs to run 24/7 and is heavy on graphics (one of them would crash after less than 24 hrs).<br>

Now, even though the application is entirely under my control, the compiler is not, so I'm stuck with a deprecated 32-bit compiler for this iteration (before migrating to another compiler).<br>
Now, I had already traced it down to xlib and was traversing the library code when I found that you had already went down that road. I reviewed your patch and - provided there are no other hidden dragons - found that it worked as advertised (using XNoOp as proof of concept and by running my app). Thanks for the good work btw.</p>

<p dir="ltr">Regarding the seriousness I completely agree. It is an important bug. Other applications might crash after weeks or months, in which case users will have a hard time understanding why and might conclude the "OS" is not stable or anything.</p>

<p dir="ltr">Anyway, for my apps I have no problem - we have custom installs anyways - but other people might...<br>
 </p>
<p dir="ltr">So, the only question I have for Keith is: is the patch good enough to be put into an official linux distribution, while the next xlib release has not yet been released?</p>
<p dir="ltr">best regards,<br>
Jan</p>
<div class="gmail_quote">On Aug 6, 2014 10:49 PM, "Jonas Petersen" <<a href="mailto:jnsptrsn1@gmail.com">jnsptrsn1@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div>Hi Jan,<br>
      <br>
      thanks for pushing this. <br>
      <br>
      I spent really a lot of time (weeks) tracking this down and
      finding a solution. Digging down the depths of the operating
      system, while actually writting application software. The result
      is the mentioned patch. I then posted it here. I think there is
      approval that the fix actually does work. Then there was starting
      some discussion about implementation details, optimization and
      possible further problems at other locations.<br>
      <br>
      At some point I had to take a break, since this had cost me
      already so much time. Sorry about that. It's to bad this is still
      pending.<br>
      <br>
      If nothing happens I might be willing to spend another small
      amount of time to help completing this. But my time is limited. I
      can not promise anything.<br>
      <br>
      I think this bug is quite serious. It suddenly kills programs
      without asking out of nowhere. And it's patient.<br>
      <br>
      By the way, my software now runs on 64-bit, so luckily I'm not
      affected anymore (hopefully). But there's probably still plenty of
      32-bit systems
      
      endangered by this.<br>
      <br>
      Have you seen? Keith posted a program to reproduce the bug (or
      confirm that the patch works) as fast as possible:<br>
      <br>
      <pre>/* cc -o nop nop.c `pkg-config --cflags --libs x11` */
#include    <stdio.h>
#include    <stdint.h>
#include    <X11/Xlib.h>

int
main (int argc, char **argv)
{
        uint64_t        i = 0;
        Display *dpy = XOpenDisplay(NULL);

        for (;;) {
                ++i;
                if ((i & 0xfffffff) == 0) {
                        XFlush(dpy);
                        printf ("0x%llx\n", i);
                }
                XNoOp(dpy);
        }
}
</pre>
      <br>
      Regards<br>
      Jonas<br>
      <br>
      <br>
      <br>
      <br>
      Am 29.07.2014 um 18:56 schrieb Jan Smout:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr">Hi all,<br>
        <div><br>
          I recently stumbled into an application that crashed because
          of this:<br>
          <a href="https://bugs.freedesktop.org/show_bug.cgi?id=71338" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=71338</a><br>
          <br>
          and quickly found the following patch:<br>
          <a href="http://patchwork.freedesktop.org/patch/16753/" target="_blank">http://patchwork.freedesktop.org/patch/16753/</a><br>
          <br>
        </div>
        <div>which seems to work fine (the application used to crash in
          less than 24 hrs. Has been running for 5 days straight with
          the patch).<br>
        </div>
        <div><br>
        </div>
        <div><br>
          Now my question: what is the status of this patch? Are there
          still details to be clarified before it can be put into the
          main tree?<br>
          <br>
          <br clear="all">
        </div>
        <div>best regards,<br>
        </div>
        <div>
          Jan<br>
        </div>
        <div>-- <br>
          Life is complex, it has a real part and an imaginary part.
        </div>
      </div>
    </blockquote>
    <br>
  </div>

</blockquote></div>