Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Sepherosa Ziehau
September 05, 2013 02:48AM
On Tue, Sep 3, 2013 at 10:36 PM, Maxim Dounin <mdounin@mdounin.ru> wrote:

> Hello!
>


Hi,



> > Well, the idea is to keep at least one listen socket opened. Maybe I
> could
> > find other way in kernel to make it less tricky. However, that may add
> > extra syscall or socket option.
>
> I think extra syscall/socket option will be ok as long as it'll
> save us from the hassle of opening sockets. Not sure what to do
> with Linux compatibility though.
>


Yeah, this is also my concern.



>
> Another aproach which may be slightly better than the code is your
> last patch is to reopen sockets before spawning each worker
> process: this way, master may keep listen sockets open (listen
> queue is shared with the same socket as inherited by a worker
> process then, right?) and worker processes are equal and don't
> need to open sockets themself. It needs careful handling on dead
> process respawn codepath though.
>


This may be doable and could better than my approach. I will take a look
at the code and try implementing it.



> > > (We've also discussed this here in office serveral times, and it
> > > seems that general consensus is that SO_REUSEPORT for TCP balancing
> > > isn't really good interface. It would be much easier for everyone
> > > if normal workflow with inherited listen socket descriptors just
> > > worked. Especially given the fact that in nginx case it's mostly
> > > about benchmarking, since in real life load distribution between
> > > worker processes is good enough.)
> >
> >
> > In DragonFly, SO_REUSEPORT is more than load balance: it makes the
> accepted
> > sockets network processing completely CPU localized (from user land to
> > kernel land on both RX and TX path). This level of network processing
> CPU
> > localization could not be achieved by the old listen socket inheritance
> > usage model (even if I could divide listen socket's completion queue to
> > each CPU base on RX hash, the level of CPU localization achieved by
> > SO_REUSEPORT still could not be achieved easily).
>
> Could you please point out how it's achieved?
>
>

I have just put something up, which may help understanding what I have
described above. Here it is:
http://leaf.dragonflybsd.org/~sephe/netisr_so_reuseport.txt



> We here tend to think that proper interface from an application
> point of view would be to implement a socket option which
> basically creates separate listen queues for inherited sockets.
> But if this isn't going to work, it's probably better to focus on
> SO_REUSEPORT.
>


Well, I think I am going to stick w/ SO_REUSEPORT, mainly because the
implementation is simple, straightforward, less invasive and the result is
good. Besides, user space applications only need small changes to the
listen socket related code (most of the time, it is quite simple), which
means easy adoption. And in addition to TCP listen socket, SO_REUSEPORT
also helps UDP socket reception load distribution and processing CPU
localization.



>
> BTW, are you going to be on the upcoming EuroBSDcon? I'm not, but
> Igor and Gleb Smirnoff (glebius@freebsd.org) will be there, and it
> will be cool if you'll meet and discuss the SO_REUSEPORT usage for
> balancing.
>
>

Sorry, I am not going to attend EuroBSDcon. However, it will be cool if we
could discuss (through email) about SO_REUSEPORT or something that you
folks are planning.

Best Regards,
sephe

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

[PATCH] SO_REUSEPORT support for listen sockets (round 3)

Sepherosa Ziehau 1702 August 02, 2013 01:18AM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Sepherosa Ziehau 666 August 29, 2013 09:26AM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Maxim Dounin 628 August 29, 2013 03:52PM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Sepherosa Ziehau 597 August 30, 2013 04:20AM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Maxim Dounin 543 September 02, 2013 10:50AM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Sepherosa Ziehau 841 September 02, 2013 10:34PM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Maxim Dounin 621 September 03, 2013 10:38AM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Sepherosa Ziehau 573 September 05, 2013 02:48AM

Re: [PATCH] SO_REUSEPORT support for listen sockets (round 3)

Maxim Dounin 1385 September 05, 2013 01:30PM

accept_mutex

Myla John-B22173 598 October 22, 2013 01:08PM

Re: accept_mutex

Maxim Dounin 605 October 22, 2013 06:06PM



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

Online Users

Guests: 281
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