target/linux/generic/backport-5.4/080-wireguard-0090-wireguard-socket-remove-extra-call-to-synchronize_ne.patch

   1 From ce6c6fa0d2dd4ca9c500e6240e4f22c48018a0ae Mon Sep 17 00:00:00 2001
   2 From: "Jason A. Donenfeld" <Jason@zx2c4.com>
   3 Date: Fri, 14 Feb 2020 23:57:23 +0100
   4 Subject: [PATCH 090/124] wireguard: socket: remove extra call to
   5  synchronize_net
   6
   7 commit 1fbc33b0a7feb6ca72bf7dc8a05d81485ee8ee2e upstream.
   8
   9 synchronize_net() is a wrapper around synchronize_rcu(), so there's no
  10 point in having synchronize_net and synchronize_rcu back to back,
  11 despite the documentation comment suggesting maybe it's somewhat useful,
  12 "Wait for packets currently being received to be done." This commit
  13 removes the extra call.
  14
  15 Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
  16 Suggested-by: Eric Dumazet <eric.dumazet@gmail.com>
  17 Reviewed-by: Eric Dumazet <edumazet@google.com>
  18 Signed-off-by: David S. Miller <davem@davemloft.net>
  19 Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
  20 ---
  21  drivers/net/wireguard/socket.c | 1 -
  22  1 file changed, 1 deletion(-)
  23
  24 --- a/drivers/net/wireguard/socket.c
  25 +++ b/drivers/net/wireguard/socket.c
  26 @@ -432,7 +432,6 @@ void wg_socket_reinit(struct wg_device *
  27                 wg->incoming_port = ntohs(inet_sk(new4)->inet_sport);
  28         mutex_unlock(&wg->socket_update_lock);
  29         synchronize_rcu();
  30 -       synchronize_net();
  31         sock_free(old4);
  32         sock_free(old6);
  33  }