mirror of
https://github.com/torvalds/linux.git
synced 2026-07-28 10:09:10 +02:00
Merge branch 'udp_tunnel-speed-up-udp-tunnel-device-destruction-part-ii'
Kuniyuki Iwashima says: ==================== udp_tunnel: Speed up UDP tunnel device destruction (Part II) Most of the UDP tunnel devices called synchronize_rcu() twice during destruction, for example, vxlan had 1) synchronize_rcu() in udp_tunnel_sock_release() 2) synchronize_net() in vxlan_sock_release() The former was already removed by: https://lore.kernel.org/netdev/20260502031401.3557229-1-kuniyu@google.com/ This series removes the latter. Patch 1 & 2 & 4 remove synchronize_net() placed before udp_tunnel_sock_release(). Patch 3 removes yet another unnecessary synchronize_net() in geneve_unquiesce(). Patch 5 is a follow-up patch for a sparse report by kernel test robot. ==================== Link: https://patch.msgid.link/20260518050726.318824-1-kuniyu@google.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
commit
830d8771ae
|
|
@ -285,9 +285,8 @@ static void bareudp_sock_release(struct bareudp_dev *bareudp)
|
|||
{
|
||||
struct sock *sk;
|
||||
|
||||
sk = bareudp->sk;
|
||||
sk = rtnl_dereference(bareudp->sk);
|
||||
rcu_assign_pointer(bareudp->sk, NULL);
|
||||
synchronize_net();
|
||||
udp_tunnel_sock_release(sk);
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -1032,7 +1032,6 @@ static void geneve_sock_release(struct geneve_dev *geneve)
|
|||
#endif
|
||||
|
||||
rcu_assign_pointer(geneve->sock4, NULL);
|
||||
synchronize_net();
|
||||
|
||||
__geneve_sock_release(gs4);
|
||||
#if IS_ENABLED(CONFIG_IPV6)
|
||||
|
|
@ -2207,7 +2206,6 @@ static void geneve_unquiesce(struct geneve_dev *geneve, struct geneve_sock *gs4,
|
|||
if (gs6)
|
||||
rcu_assign_sk_user_data(gs6->sk, gs6);
|
||||
#endif
|
||||
synchronize_net();
|
||||
}
|
||||
|
||||
static int geneve_changelink(struct net_device *dev, struct nlattr *tb[],
|
||||
|
|
|
|||
|
|
@ -1515,7 +1515,6 @@ static void vxlan_sock_release(struct vxlan_dev *vxlan)
|
|||
#endif
|
||||
|
||||
RCU_INIT_POINTER(vxlan->vn4_sock, NULL);
|
||||
synchronize_net();
|
||||
|
||||
if (vxlan->cfg.flags & VXLAN_F_VNIFILTER)
|
||||
vxlan_vs_del_vnigrp(vxlan);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user