Maxim Dounin
October 08, 2014 09:00AM
Hello!

On Tue, Oct 07, 2014 at 07:32:08PM +0000, Lu, Yingqi wrote:

> Dear All,
>
> It has been quiet for a while on this patch. I am checking to
> see if there is any questions/feedbacks/concerns we need to
> address?
>
> Please let me know. Thanks very much for your help!

Apart from style/coding issues, I disagree with the whole
approach.

As far as I understand the patch idea, it tries to introduce
multiple listening sockets to avoid in-kernel lock contention.
This is something that can be done completely in kernel though,
and I see no reason to introduce any changes to nginx here.

The approach previously discussed with Sepherosa Ziehau looks much
more interesting.

--
Maxim Dounin
http://nginx.org/

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[Patch] SO_REUSEPORT support from master process Attachments

Lu, Yingqi 1456 August 22, 2014 12:56PM

RE: [Patch] SO_REUSEPORT support from master process

Lu, Yingqi 671 August 22, 2014 01:00PM

Re: [Patch] SO_REUSEPORT support from master process

Sepherosa Ziehau 707 August 27, 2014 05:10AM

RE: [Patch] SO_REUSEPORT support from master process

luy 1524 August 27, 2014 12:26PM

Re: [Patch] SO_REUSEPORT support from master process

Sepherosa Ziehau 717 August 28, 2014 05:28AM

RE: [Patch] SO_REUSEPORT support from master process

luy 646 August 27, 2014 01:36PM

RE: [Patch] SO_REUSEPORT support from master process

luy 1501 September 18, 2014 04:08PM

Re: [Patch] SO_REUSEPORT support from master process

Valentin V. Bartenev 596 September 19, 2014 04:38AM

RE: [Patch] SO_REUSEPORT support from master process

luy 668 September 19, 2014 11:54AM

Re: [Patch] SO_REUSEPORT support from master process

Valentin V. Bartenev 608 September 19, 2014 12:50PM

RE: [Patch] SO_REUSEPORT support from master process

luy 698 September 19, 2014 01:34PM

RE: [Patch] SO_REUSEPORT support from master process

luy 607 September 20, 2014 06:24PM

RE: [Patch] SO_REUSEPORT support from master process

luy 601 September 20, 2014 07:02PM

RE: [Patch] SO_REUSEPORT support from master process

luy 706 September 20, 2014 07:46PM

RE: [Patch] SO_REUSEPORT support from master process

luy 612 September 23, 2014 11:38AM

RE: [Patch] SO_REUSEPORT support from master process

luy 661 September 23, 2014 11:48AM

RE: [Patch] SO_REUSEPORT support from master process

luy 717 September 29, 2014 03:46PM

RE: [Patch] SO_REUSEPORT support from master process

luy 612 October 07, 2014 03:34PM

Re: [Patch] SO_REUSEPORT support from master process

Maxim Dounin 637 October 08, 2014 09:00AM

RE: [Patch] SO_REUSEPORT support from master process

luy 601 October 08, 2014 01:08PM

RE: [Patch] SO_REUSEPORT support from master process

luy 590 October 08, 2014 02:26PM

RE: [Patch] SO_REUSEPORT support from master process

luy 575 October 30, 2014 06:26PM

Re: [Patch] SO_REUSEPORT support from master process

Sepherosa Ziehau 647 November 16, 2014 04:08AM

Re: [Patch] SO_REUSEPORT support from master process

Maxim Dounin 841 November 16, 2014 06:52AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 203
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready