On Wed, Jan 3, 2018 at 5:00 AM, Rainer Jung <rainer.j...@kippdata.de> wrote: > > I did a check on Solaris 10 Sparc. Looks good as well:
Thanks Rainer, the PR is about Sparc so this is quite valuable return. I first thought it was some concurrency issues (in mpm_event) specific to Solaris Sparc until I found this.. > > - r1819856 (without the additional check and without the fix): no hang in > testpoll > > - r1819857 (with the additional check but still without the fix): hangs in > testpoll in pollset_wakeup, more precisely in the new for loop in iteration > 2 of 2 and there in the call to apr_pollset_wakeup(). Don't you mean that the hang was in the second call to apr_pollset_poll()? I don't really expect apr_pollset_wakeup() to block unless the pipe is full, which'd be a capacity of one byte only here... > > - r1819858 (with the additional check and with the fix): no hang in testpoll > > - r1819902 (head of trunk): no hang in testpoll > > Thanks for finding and fixing this! > > Happy new year, Best wishes to all! Regards, Yann.
