On 19 August 2014 13:37, Pekka Paalanen <ppaala...@gmail.com> wrote: > On Tue, 19 Aug 2014 12:37:53 +0200 > Marek Chalupa <mchqwe...@gmail.com> wrote: > > > On 18 August 2014 12:23, Marek Chalupa <mchqwe...@gmail.com> wrote: > > > > > Hi, > > > > > > thanks for reviewing. > > > > > > The failure you got is not introduced by this patch, I reported it some > > > time ago here: > > > https://bugs.freedesktop.org/show_bug.cgi?id=80594 > > > > > > > I put a patch for this bug into bugzilla: > > > > https://bugs.freedesktop.org/show_bug.cgi?id=80594 > > Hi, > > ok, I picked them up from bugzilla, but I would really prefer email to > the list, because I can't review bugzilla attachments on the mailing > list, and it is much much easier for me to apply patches from email. >
Ok :) > So the essence of the test is to get two timers fire so, that they will > both be dispatched on the same wl_event_loop_dispatch() call. What it > actually tests is that the timerfd handler won't get stuck because the > handler that runs first modifies the second. > Yes. > I believe your patch should fix it, so I pushed them. I try to remember > to leave a 'make check' loop running over night. > > I had it running in a loop for some time and I hope it fixed it. It was not a whole night, though :) > > Thanks, > pq > Thanks, Marek
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel