lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 29 Aug 2022 09:19:15 -0700 From: Kuniyuki Iwashima <kuniyu@...zon.com> To: "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com> CC: Kuniyuki Iwashima <kuniyu@...zon.com>, Kuniyuki Iwashima <kuni1840@...il.com>, <netdev@...r.kernel.org> Subject: [PATCH v2 net-next 0/5] tcp: Introduce optional per-netns ehash. The more sockets we have in the hash table, the longer we spend looking up the socket. While running a number of small workloads on the same host, they penalise each other and cause performance degradation. The root cause might be a single workload that consumes much more resources than the others. It often happens on a cloud service where different workloads share the same computing resource. On EC2 c5.24xlarge instance (196 GiB memory and 524288 (1Mi / 2) ehash entries), after running iperf3 in different netns, creating 24Mi sockets without data transfer in the root netns causes about 10% performance regression for the iperf3's connection. thash_entries sockets length Gbps 524288 1 1 50.7 24Mi 48 45.1 It is basically related to the length of the list of each hash bucket. For testing purposes to see how performance drops along the length, I set 131072 (1Mi / 8) to thash_entries, and here's the result. thash_entries sockets length Gbps 131072 1 1 50.7 1Mi 8 49.9 2Mi 16 48.9 4Mi 32 47.3 8Mi 64 44.6 16Mi 128 40.6 24Mi 192 36.3 32Mi 256 32.5 40Mi 320 27.0 48Mi 384 25.0 To resolve the socket lookup degradation, we introduce an optional per-netns hash table for TCP, but it's just ehash, and we still share the global bhash, bhash2 and lhash2. With a smaller ehash, we can look up non-listener sockets faster and isolate such noisy neighbours. Also, we can reduce lock contention. For details, please see the last patch. patch 1 - 3: prep for per-netns ehash patch 4: small optimisation for netns dismantle without TIME_WAIT sockets patch 5: add per-netns ehash Changes: v2: * Drop flock() and UDP stuff * Patch 2 * Rename inet_get_hashinfo() to tcp_or_dccp_get_hashinfo() (Eric Dumazet) * Patch 4 * Remove unnecessary inet_twsk_purge() calls for unshare() * Factorise inet_twsk_purge() calls (Eric Dumazet) * Patch 5 * Change max buckets size as 16Mi * Use unsigned int for ehash size (Eric Dumazet) * Use GFP_KERNEL_ACCOUNT for the per-netns ehash allocation (Eric Dumazet) * Use current->nsproxy->net_ns for parent netns (Eric Dumazet) v1: https://lore.kernel.org/netdev/20220826000445.46552-1-kuniyu@amazon.com/ Kuniyuki Iwashima (5): tcp: Clean up some functions. tcp: Set NULL to sk->sk_prot->h.hashinfo. tcp: Access &tcp_hashinfo via net. tcp: Save unnecessary inet_twsk_purge() calls. tcp: Introduce optional per-netns ehash. Documentation/networking/ip-sysctl.rst | 22 ++++ .../chelsio/inline_crypto/chtls/chtls_cm.c | 5 +- .../mellanox/mlx5/core/en_accel/ktls_rx.c | 5 +- .../net/ethernet/netronome/nfp/crypto/tls.c | 5 +- include/net/inet_hashtables.h | 16 +++ include/net/netns/ipv4.h | 1 + include/net/tcp.h | 1 + net/core/filter.c | 5 +- net/dccp/proto.c | 2 + net/ipv4/af_inet.c | 2 +- net/ipv4/esp4.c | 3 +- net/ipv4/inet_connection_sock.c | 22 ++-- net/ipv4/inet_hashtables.c | 102 ++++++++++++---- net/ipv4/inet_timewait_sock.c | 4 +- net/ipv4/netfilter/nf_socket_ipv4.c | 2 +- net/ipv4/netfilter/nf_tproxy_ipv4.c | 17 ++- net/ipv4/sysctl_net_ipv4.c | 58 +++++++++ net/ipv4/tcp.c | 1 + net/ipv4/tcp_diag.c | 18 ++- net/ipv4/tcp_ipv4.c | 113 ++++++++++++------ net/ipv4/tcp_minisocks.c | 31 ++++- net/ipv6/esp6.c | 3 +- net/ipv6/inet6_hashtables.c | 4 +- net/ipv6/netfilter/nf_socket_ipv6.c | 2 +- net/ipv6/netfilter/nf_tproxy_ipv6.c | 5 +- net/ipv6/tcp_ipv6.c | 20 ++-- net/mptcp/mptcp_diag.c | 7 +- 27 files changed, 362 insertions(+), 114 deletions(-) -- 2.30.2
Powered by blists - more mailing lists