Re: svn commit: r1818960 - /httpd/httpd/trunk/server/mpm/event/event.c

2017-12-21 Thread Yann Ylavic
On Thu, Dec 21, 2017 at 7:04 PM, Yann Ylavic wrote: > On Thu, Dec 21, 2017 at 6:59 PM, Yann Ylavic wrote: >> On Thu, Dec 21, 2017 at 6:49 PM, Stefan Eissing wrote: >>> Oops, thought it was OK. That causes the hangers? >> >> Not

Re: svn commit: r1818960 - /httpd/httpd/trunk/server/mpm/event/event.c

2017-12-21 Thread Stefan Eissing
Oops, thought it was OK. That causes the hangers? > Am 21.12.2017 um 18:45 schrieb yla...@apache.org: > > Author: ylavic > Date: Thu Dec 21 17:45:08 2017 > New Revision: 1818960 > > URL: http://svn.apache.org/viewvc?rev=1818960=rev > Log: > mpm_event: follow up to r1818804. > > Allow DONE as a

Re: svn commit: r1818960 - /httpd/httpd/trunk/server/mpm/event/event.c

2017-12-21 Thread Yann Ylavic
On Thu, Dec 21, 2017 at 6:49 PM, Stefan Eissing wrote: > Oops, thought it was OK. That causes the hangers? Not anymore :) But h2_conn_run() seems to possibly return CONN_STATE_HANDLER, it looks stange to me, and for once causes hanger ;) What's the meaning of it?

Re: svn commit: r1818960 - /httpd/httpd/trunk/server/mpm/event/event.c

2017-12-21 Thread Yann Ylavic
On Thu, Dec 21, 2017 at 6:59 PM, Yann Ylavic wrote: > On Thu, Dec 21, 2017 at 6:49 PM, Stefan Eissing wrote: >> Oops, thought it was OK. That causes the hangers? > > Not anymore :) > > But h2_conn_run() seems to possibly return CONN_STATE_HANDLER, it >

Re: svn commit: r1818804 - in /httpd/httpd/trunk: CHANGES server/mpm/event/event.c

2017-12-21 Thread Yann Ylavic
On Wed, Dec 20, 2017 at 6:03 PM, Eric Covener wrote: > On Wed, Dec 20, 2017 at 11:28 AM, Yann Ylavic wrote: >> On Wed, Dec 20, 2017 at 4:51 PM, Eric Covener wrote: >>> >>> is the "rc" check just covering further bases? Should we trace