Commit 2a8a4df3 authored by Jason A. Donenfeld's avatar Jason A. Donenfeld Committed by David S. Miller

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: default avatarJason A. Donenfeld <Jason@zx2c4.com>
Suggested-by: default avatarMatt Dunwoodie <ncon@noconroy.net>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 04ddf120
...@@ -118,10 +118,13 @@ static void wg_receive_handshake_packet(struct wg_device *wg, ...@@ -118,10 +118,13 @@ static void wg_receive_handshake_packet(struct wg_device *wg,
under_load = skb_queue_len(&wg->incoming_handshakes) >= under_load = skb_queue_len(&wg->incoming_handshakes) >=
MAX_QUEUED_INCOMING_HANDSHAKES / 8; MAX_QUEUED_INCOMING_HANDSHAKES / 8;
if (under_load) if (under_load) {
last_under_load = ktime_get_coarse_boottime_ns(); 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); 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, mac_state = wg_cookie_validate_packet(&wg->cookie_checker, skb,
under_load); under_load);
if ((under_load && mac_state == VALID_MAC_WITH_COOKIE) || if ((under_load && mac_state == VALID_MAC_WITH_COOKIE) ||
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment