summaryrefslogtreecommitdiffstats
path: root/drivers/net
diff options
context:
space:
mode:
authorJason A. Donenfeld <Jason@zx2c4.com>2020-02-14 23:57:21 +0100
committerDavid S. Miller <davem@davemloft.net>2020-02-16 19:21:56 -0800
commit2a8a4df36462aa85b0db87b7c5ea145ba67e34a8 (patch)
tree67eb1dfc230baa59667ae23e79af77f231d0c13e /drivers/net
parent04ddf1208f03e1dbc39a4619c40eba640051b950 (diff)
downloadlinux-2a8a4df36462aa85b0db87b7c5ea145ba67e34a8.tar.gz
linux-2a8a4df36462aa85b0db87b7c5ea145ba67e34a8.tar.bz2
linux-2a8a4df36462aa85b0db87b7c5ea145ba67e34a8.zip
wireguard: receive: reset last_under_load to zero
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>
Diffstat (limited to 'drivers/net')
-rw-r--r--drivers/net/wireguard/receive.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/wireguard/receive.c b/drivers/net/wireguard/receive.c
index 9c6bab9c981f..4a153894cee2 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) ||