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

Pekka Paalanen ppaalanen at gmail.com
Tue Aug 19 04:37:25 PDT 2014


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.

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.

I believe your patch should fix it, so I pushed them. I try to remember
to leave a 'make check' loop running over night.


Thanks,
pq


More information about the wayland-devel mailing list