Re: [PATCH 1/1] net: Add SO_REUSEPORT_LISTEN_OFF socket option as drain mode
From: Tom Herbert <hidden>
Date: 2015-11-11 17:05:46
From: Tom Herbert <hidden>
Date: 2015-11-11 17:05:46
On Tue, Nov 10, 2015 at 10:19 PM, Eric Dumazet [off-list ref] wrote:
On Tue, 2015-11-10 at 21:41 -0800, Tom Herbert wrote:quoted
Tolga, are you still planning to respin this patch (when tree opens?)I was planning to add an union on skc_tx_queue_mapping and sk_max_ack_backlog, so that adding a check on sk_max_ack_backlog in listener lookup would not add an additional cache line miss. This would remove false sharing because sk_ack_backlog is often dirtied when a socket is added into accept queue.
That's sounds like good fixes, but my question was more about the problem originally described by Tolga where we are transitioning processing for a listener port from one process to another. I think the conclusion in this thread was to modify the code so that listen(fd, 0) would stop new connections from being assigned to a socket (as opposed to explicit SO_REUSEPORT_LISTEN_OFF option). Does this still seem reasonable? Tom