[PATCH 3/3] tests: add one more test for event-loop signal source

[Marek Chalupa]

On 19 August 2014 13:37, Pekka Paalanen <ppaalanen at gmail.com> wrote:

> On Tue, 19 Aug 2014 12:37:53 +0200
> Marek Chalupa <mchqwerty at gmail.com> wrote:
>
> > On 18 August 2014 12:23, Marek Chalupa <mchqwerty at 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20140819/28af42de/attachment-0001.html>