From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-db5eur01on0090.outbound.protection.outlook.com ([104.47.2.90]:30282 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752926AbeCVJpC (ORCPT ); Thu, 22 Mar 2018 05:45:02 -0400 Subject: [PATCH net-next v3 0/5] Rework ip_ra_chain protection From: Kirill Tkhai To: davem@davemloft.net, yoshfuji@linux-ipv6.org, edumazet@google.com, yanhaishuang@cmss.chinamobile.com, nikolay@cumulusnetworks.com, yotamg@mellanox.com, soheil@google.com, avagin@virtuozzo.com, nicolas.dichtel@6wind.com, ebiederm@xmission.com, fw@strlen.de, roman.kapl@sysgo.com, netdev@vger.kernel.org, xiyou.wangcong@gmail.com, dvyukov@google.com, andreyknvl@google.com, lkp@intel.com, ktkhai@virtuozzo.com Date: Thu, 22 Mar 2018 12:44:46 +0300 Message-ID: <152171176936.18202.11912079579606814167.stgit@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: netdev-owner@vger.kernel.org List-ID: Commit 1215e51edad1 "ipv4: fix a deadlock in ip_ra_control" made rtnl_lock() be used in raw_close(). This function is called on every RAW socket destruction, so that rtnl_mutex is taken every time. This scales very sadly. I observe cleanup_net() spending a lot of time in rtnl_lock() and raw_close() is one of the biggest rtnl user (since we have percpu net->ipv4.icmp_sk). This patchset reworks the locking: reverts the problem commit and its descendant, and introduces rtnl-independent locking. This may have a continuation, and someone may work on killing rtnl_lock() in mrtsock_destruct() in the future. Thanks, Kirill --- v3: Change patches order: [2/5] and [3/5]. v2: Fix sparse warning [4/5], as reported by kbuild test robot. --- Kirill Tkhai (5): net: Revert "ipv4: get rid of ip_ra_lock" net: Move IP_ROUTER_ALERT out of lock_sock(sk) net: Revert "ipv4: fix a deadlock in ip_ra_control" net: Make ip_ra_chain per struct net net: Replace ip_ra_lock with per-net mutex include/net/ip.h | 13 +++++++++++-- include/net/netns/ipv4.h | 2 ++ net/core/net_namespace.c | 1 + net/ipv4/ip_input.c | 5 ++--- net/ipv4/ip_sockglue.c | 34 +++++++++++++--------------------- net/ipv4/ipmr.c | 11 +++++++++-- net/ipv4/raw.c | 2 -- 7 files changed, 38 insertions(+), 30 deletions(-) -- Signed-off-by: Kirill Tkhai