diff options
| author | Jason A. Donenfeld <Jason@zx2c4.com> | 2020-02-14 23:57:21 +0100 |
|---|---|---|
| committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2022-07-07 13:26:41 +0200 |
| commit | cb46e5bca583fabeb25e3b1e637970611b1aa91c (patch) | |
| tree | ff451cfa070c8bf14c04488d0cc44222bbc1481f | |
| parent | aa4e44aeabe4036eb66b31ee5c996f2e6db8e952 (diff) | |
| download | wireguard-linux-trimmed-cb46e5bca583fabeb25e3b1e637970611b1aa91c.tar.gz wireguard-linux-trimmed-cb46e5bca583fabeb25e3b1e637970611b1aa91c.zip | |
wireguard: receive: reset last_under_load to zero
commit 11717797814ecc227f5bc4759d3f39f3cc22ddcf upstream.
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>
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
| -rw-r--r-- | drivers/net/wireguard/receive.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/wireguard/receive.c b/drivers/net/wireguard/receive.c index 9c6bab9..4a15389 100644 --- a/drivers/net/wireguard/receive.c +++ b/drivers/net/wireguard/receive.c @@ -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) || |