httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lu, Yingqi" <yingqi...@intel.com>
Subject RE: svn commit: r1599531 - in /httpd/httpd/trunk: CHANGES include/ap_listen.h server/listen.c server/mpm/event/event.c server/mpm/prefork/prefork.c server/mpm/worker/worker.c server/mpm_unix.c
Date Sun, 05 Oct 2014 18:38:28 GMT
Here (attachment) is the most recent version of the fix. It fixes a small issue for event mpm
in the version I sent out yesterday. Please use this one as the final fix. I have already
updated the Bugzilla database for "Bug 55897 - [PATCH]patch with SO_REUSEPORT support"

Jim/Jeff, can you please help review it and add it into the trunk?

Thanks very much for your help,
Yingqi

-----Original Message-----
From: Lu, Yingqi [mailto:yingqi.lu@intel.com] 
Sent: Saturday, October 04, 2014 10:54 PM
To: dev@httpd.apache.org
Subject: RE: svn commit: r1599531 - in /httpd/httpd/trunk: CHANGES include/ap_listen.h server/listen.c
server/mpm/event/event.c server/mpm/prefork/prefork.c server/mpm/worker/worker.c server/mpm_unix.c

Hi Kaspar,

Thanks very much for testing the fixes and I am glad it works. 

I tested it on prefork/worker/event as well. It was a universal issue to all the mpms. It
should all work now. Really appreciated your help.

I will updated the Bugzilla database for "Bug 55897 - [PATCH]patch with SO_REUSEPORT support".


Thanks!
Yingqi

-----Original Message-----
From: Kaspar Brand [mailto:httpd-dev.2014@velox.ch]
Sent: Saturday, October 04, 2014 10:48 PM
To: dev@httpd.apache.org
Subject: Re: svn commit: r1599531 - in /httpd/httpd/trunk: CHANGES include/ap_listen.h server/listen.c
server/mpm/event/event.c server/mpm/prefork/prefork.c server/mpm/worker/worker.c server/mpm_unix.c

On 05.10.2014 02:27, Lu, Yingqi wrote:
> Kaspar, can you please test the patch and let us know if that resolves 
> your issue?

Yes, makes the restart issues disappear for me (only tested with the worker MPM, and not very
extensively). Thanks.

> At the meantime, can some please review the patch and help add it into 
> trunk?

I'll defer to Jeff or Jim (I'm definitely not familiar enough with anything below server/mpm/...).

Kaspar
Mime
View raw message