2.4.35 in Sept?

2018-08-29 Thread Jim Jagielski
Looking to maybe do an interim release in Sept... Comments? Feedback?

Re: svn commit: r1839535 - /httpd/httpd/branches/2.4.x/STATUS

2018-08-29 Thread Jim Jagielski
We have before and, I think, we have provided no guarantee on how the status page "looks"... it is for that reason we provide the XML format alternative. > On Aug 28, 2018, at 11:41 PM, rj...@apache.org wrote: > > Author: rjung > Date: Wed Aug 29 03:41:51 2018 > New Revision: 1839535 > > URL:

Re: Fwd: [Bug 62590] performance drop after moving from apache 2.2 to apache 2.4

2018-08-29 Thread Ruediger Pluem
On 08/29/2018 10:49 AM, Christophe Jaillet wrote: > Ok, it is trickier than I thought. > > On 2018/08/29 07:55:04, Ruediger Pluem wrote: >> >> 2. As the error handling is a stack I think it is a valid use case for >> another consumer of openssl to check for the >>error later and not

Re: Fwd: [Bug 62590] performance drop after moving from apache 2.2 to apache 2.4

2018-08-29 Thread Christophe Jaillet
Ok, it is trickier than I thought. On 2018/08/29 07:55:04, Ruediger Pluem wrote: > > 2. As the error handling is a stack I think it is a valid use case for > another consumer of openssl to check for the >error later and not directly like e.g. with errno. > but in this case, haven't we

Re: [Bug 62590] performance drop after moving from apache 2.2 to apache 2.4

2018-08-29 Thread Stefan Eissing
Just for interest ran my h2 load tests on trunk/2.4.x unpatched and with commenting ERR_clear_error(); in ssl_engine_io.c On my Mac I see no differences, really. trunk used openssl 1.0.2o and 2.4.x had 1.0.2l: > h2load -i gen/load-urls-1.txt -n 20 -t 4 -m 100 -c trunk raw, 128

Re: Fwd: [Bug 62590] performance drop after moving from apache 2.2 to apache 2.4

2018-08-29 Thread Ruediger Pluem
On 08/29/2018 09:37 AM, Christophe Jaillet wrote: > Pure speculation: > > Actually we ERR_clear_error unconditionally so that in case of error, we can > safely call SSL_get_error. > > Doc says: > << >> > ERR_get_error() returns the earliest error code from the thread's error

Re: Fwd: [Bug 62590] performance drop after moving from apache 2.2 to apache 2.4

2018-08-29 Thread Christophe Jaillet
Pure speculation: Actually we ERR_clear_error unconditionally so that in case of error, we can safely call SSL_get_error. Doc says: << >> ERR_get_error() returns the earliest error code from the thread's error queue and removes the entry. This function can be called repeatedly

Re: Fwd: [Bug 62590] performance drop after moving from apache 2.2 to apache 2.4

2018-08-29 Thread Ruediger Pluem
Having a brief look at ERR_clear_error in openssl it looks like that various locking operations are performed to get the error state. I assume that this causes a lot of contention in the load case. The question is whether there is another solution to BZ62590 but to call ERR_clear_error before

Re: Fwd: [Bug 62590] performance drop after moving from apache 2.2 to apache 2.4

2018-08-29 Thread Ruediger Pluem
I guess it would be helpful to know which openssl version was used for the test. There was a valid use case for BZ62590 and the question is why ERR_clear_error is that expensive. But the expensiveness might depend on the used openssl version. Regards RĂ¼diger On 08/28/2018 04:54 PM, William A