diff options
| author |   Antonio Quartulli <antonio@openvpn.net> | 2025-04-30 02:35:18 +0200 |
|---|---|---|
| committer | Antonio Quartulli <antonio@openvpn.net> | 2025-05-15 13:09:36 +0200 |
| commit | adcdaac57d3ccb38f2f1b0a8da31b5c1403385f0 (patch) | |
| tree | 25cc36e7f32cc80b9cb6241ec365acd454bb0837 | |
| parent | selftest/net/ovpn: extend coverage with more test cases (diff) | |
| download | wireguard-linux-adcdaac57d3ccb38f2f1b0a8da31b5c1403385f0.tar.xz wireguard-linux-adcdaac57d3ccb38f2f1b0a8da31b5c1403385f0.zip | |
ovpn: drop useless reg_state check in keepalive worker
The keepalive worker is cancelled before calling
unregister_netdevice_queue(), therefore it will never
hit a situation where the reg_state can be different
than NETDEV_REGISTERED.
For this reason, checking reg_state is useless and the
condition can be removed.
Signed-off-by: Antonio Quartulli <antonio@openvpn.net>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/ovpn/peer.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ovpn/peer.c b/drivers/net/ovpn/peer.c index a37f89fffb02..24eb9d81429e 100644 --- a/drivers/net/ovpn/peer.c +++ b/drivers/net/ovpn/peer.c @@ -1353,8 +1353,7 @@ void ovpn_peer_keepalive_work(struct work_struct *work) } /* prevent rearming if the interface is being destroyed */ - if (next_run > 0 && - READ_ONCE(ovpn->dev->reg_state) == NETREG_REGISTERED) { + if (next_run > 0) { netdev_dbg(ovpn->dev, "scheduling keepalive work: now=%llu next_run=%llu delta=%llu\n", next_run, now, next_run - now); |