https://issues.apache.org/bugzilla/show_bug.cgi?id=55897
--- Comment #8 from [email protected] --- In this newer version of the patch, we removed dependency of APR change by using apr_os_sock_get() and setsockopt() directly from listen.c file. Now, you only need this attached httpd patch file to test the patch. Jeff, thanks very much for your suggestion! After last update, we also spent some time testing this prefork patch and we observed over 2X performance improvements on Intel modern dual socket platforms. We are still actively working on extending this patch to worker and event mpms. Meanwhile, we would like to gather your feedback and comments on the current prefork patch. Please take some time test it and let us know how it works in your environment. Our configurations are: Configuration: <IfModule prefork.c> ListenBacklog 105384 ServerLimit 105000 MaxClients 1024 MaxRequestsPerChild 0 StartServers 64 MinSpareServers 8 MaxSpareServers 16 </IfModule> 1. Software and workloads used in performance tests may have been optimized for performance only on Intel microprocessors. Performance tests, such as SYSmark and MobileMark, are measured using specific computer systems, components, software, operations and functions. Any change to any of those factors may cause the results to vary. You should consult other information and performance tests to assist you in fully evaluating your contemplated purchases, including the performance of that product when combined with other products. Thanks, Yingqi (In reply to Yingqi.Lu from comment #6) > Hi Jeff, > > Thanks very much for your response! > > Yes, we chose to use prefork mpm due to the use of libphp5.so (non-zts). We > tested the zts version as well, but it showed some performance issues > before. then, we decided to try the patch on prefork mpm first. > > We can surely extend this patch to worker and event mpm. We will use fcgi > instead of libphp5.so for testing. Also, We will follow your suggestion to > call apr_os_sock_get() in the follow up version of the patch. > > We will update this thread soon. > > Thanks! > > Yingqi > > > > (In reply to Jeff Trawick from comment #5) > > Was this change tested with event or worker MPMs? Is the prefork MPM > > desirable in this scenario due to the use of mod_php or some other > > third-party module, or is there an issue with bundled modules that > > necessitates the use of prefork, or is there some other reason for prefork? > > > > FWIW, you don't actually need an APR change to make the desired setsockopt > > call. Call apr_os_sock_get() to get the file descriptor and call setsockopt > > directly. That would make it easier for others to use the patch with > > existing builds of APR. -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
