wireguard: receive: reset last_under_load to zero
authorJason A. Donenfeld <Jason@zx2c4.com>
Fri, 14 Feb 2020 22:57:21 +0000 (23:57 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Feb 2020 03:21:56 +0000 (19:21 -0800)
This is a small optimization that prevents more expensive comparisons
from happening when they are no longer necessary, by clearing the
last_under_load variable whenever we wind up in a state where we were
under load but we no longer are.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Suggested-by: Matt Dunwoodie <ncon@noconroy.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/wireguard/receive.c

index 9c6bab9c981f4a41d2bf008e0de739d2bd484c02..4a153894cee259504587f88e90e6ce78bfa68ea3 100644 (file)
@@ -118,10 +118,13 @@ static void wg_receive_handshake_packet(struct wg_device *wg,
 
        under_load = skb_queue_len(&wg->incoming_handshakes) >=
                     MAX_QUEUED_INCOMING_HANDSHAKES / 8;
-       if (under_load)
+       if (under_load) {
                last_under_load = ktime_get_coarse_boottime_ns();
-       else if (last_under_load)
+       } else if (last_under_load) {
                under_load = !wg_birthdate_has_expired(last_under_load, 1);
+               if (!under_load)
+                       last_under_load = 0;
+       }
        mac_state = wg_cookie_validate_packet(&wg->cookie_checker, skb,
                                              under_load);
        if ((under_load && mac_state == VALID_MAC_WITH_COOKIE) ||