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

2017-07-25 Thread Yann Ylavic
On Tue, Jul 25, 2017 at 7:49 PM, Ruediger Pluem wrote: > > > On 07/25/2017 03:32 PM, Yann Ylavic wrote: >> On Tue, Jul 25, 2017 at 9:54 AM, Ruediger Pluem wrote: >>> >>> On 07/25/2017 01:37 AM, Yann Ylavic wrote: This change may be disputable...

Re: [Bug 60956] Event MPM listener thread may get blocked by SSL shutdowns

2017-07-25 Thread Stefan Eissing
I am waiting to hear back from the peeps that opened the github issue. From how I read their logs, the patch should help them. Will report what they say. -Stefan > Am 25.07.2017 um 15:40 schrieb Stefan Eissing : > > Well, if the customer could reproduce this at a

Re: [Bug 60956] Event MPM listener thread may get blocked by SSL shutdowns

2017-07-25 Thread Stefan Eissing
Well, if the customer could reproduce this at a LogLevel http2:trace2 that would help. > Am 25.07.2017 um 15:38 schrieb Stefan Priebe - Profihost AG > : > > Hello Stefan, > > thanks for the patch. No it does not solve the problem our customer is > seeing. > > What

Re: [Bug 60956] Event MPM listener thread may get blocked by SSL shutdowns

2017-07-25 Thread Stefan Priebe - Profihost AG
Hello Stefan, thanks for the patch. No it does not solve the problem our customer is seeing. What kind of details / logs you need? Greets, Stefan Am 25.07.2017 um 11:59 schrieb Stefan Eissing: > The issue was opened here: https://github.com/icing/mod_h2/issues/143 > > I made a patch that i

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

2017-07-25 Thread Yann Ylavic
On Tue, Jul 25, 2017 at 9:54 AM, Ruediger Pluem wrote: > > On 07/25/2017 01:37 AM, Yann Ylavic wrote: >> >> This change may be disputable... >> >> I wanted to: >> 1. avoid blocking in any case (we can't where this is called), >> 2. avoid issueing a nonblocking "normal" close()

Re: [Bug 60956] Event MPM listener thread may get blocked by SSL shutdowns

2017-07-25 Thread Stefan Eissing
The issue was opened here: https://github.com/icing/mod_h2/issues/143 I made a patch that i hope addresses the problem. The 2.4.x version I attach to this mail. Thanks! Stefan h2_stream_stall_2.4.x-v0.diff Description: Binary data > Am 25.07.2017 um 08:13 schrieb Stefan Priebe - Profihost

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

2017-07-25 Thread Ruediger Pluem
On 07/25/2017 01:37 AM, Yann Ylavic wrote: > On Tue, Jul 25, 2017 at 1:19 AM, wrote: >> >> New abort_socket_nonblocking() allows to reset connections when nonblocking >> is >> required and we can't do much about the connection anymore, nor want the >> system >> to linger

Re: [Bug 60956] Event MPM listener thread may get blocked by SSL shutdowns

2017-07-25 Thread Stefan Priebe - Profihost AG
Am 24.07.2017 um 23:06 schrieb Stefan Eissing: > I have another report of request getting stuck, resulting in the error you > noticed. Will look tomorrow and report back here what I find. Thanks, if you need any logs. Pleae ask. Stefan > >> Am 24.07.2017 um 22:20 schrieb Stefan Priebe -