aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/src/device.c (follow)
Commit message (Collapse)AuthorAgeFilesLines
* global: prefix all functions with wg_Jason A. Donenfeld2018-10-021-36/+37
| | | | | | | | | | | | | I understand why this must be done, though I'm not so happy about having to do it. In some places, it puts us over 80 chars and we have to break lines up in further ugly ways. And in general, I think this makes things harder to read. Yet another thing we must do to please upstream. Maybe this can be replaced in the future by some kind of automatic module namespacing logic in the linker, or even combined with LTO and aggressive symbol stripping. Suggested-by: Andrew Lunn <andrew@lunn.ch>
* global: put SPDX identifier on its own lineJason A. Donenfeld2018-09-201-2/+2
| | | | | The kernel has very specific rules correlating file type with comment type, and also SPDX identifiers can't be merged with other comments.
* global: prefer sizeof(*pointer) when possibleJason A. Donenfeld2018-09-041-3/+2
| | | | Suggested-by: Sultan Alsawaf <sultanxda@gmail.com>
* global: satisfy check_patch.pl errorsJason A. Donenfeld2018-09-031-2/+3
|
* global: run through clang-formatJason A. Donenfeld2018-08-281-37/+58
| | | | | | | This is the worst commit in the whole repo, making the code much less readable, but so it goes with upstream maintainers. We are now woefully wrapped at 80 columns.
* send: switch handshake stamp to an atomicJason A. Donenfeld2018-08-041-1/+1
| | | | | | | | | | | Rather than abusing the handshake lock, we're much better off just using a boring atomic64 for this. It's simpler and performs better. Also, while we're at it, we set the handshake stamp both before and after the calculations, in case the calculations block for a really long time waiting for the RNG to initialize. Otherwise it's possible that when the RNG finally initializes, two handshakes are sent back to back, which isn't sensible.
* device: adjust commentJason A. Donenfeld2018-07-311-1/+1
| | | | Suggested-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
* device: destroy workqueue before freeing queueJason A. Donenfeld2018-07-181-1/+1
|
* device: print daddr not saddr in missing peer errorJason A. Donenfeld2018-07-041-1/+4
|
* global: use fast boottime instead of normal boottimeJason A. Donenfeld2018-06-231-1/+1
| | | | Generally if we're inaccurate by a few nanoseconds, it doesn't matter.
* global: use ktime boottime instead of jiffiesJason A. Donenfeld2018-06-231-1/+1
| | | | | | | | Since this is a network protocol, expirations need to be accounted for, even across system suspend. On real systems, this isn't a problem, since we're clearing all keys before suspend. But on Android, where we don't do that, this is something of a problem. So, we switch to using boottime instead of jiffies.
* device: do not assume dst is always validJason A. Donenfeld2018-05-311-1/+1
| | | | | | The new flow offloading feature at the moment does not set the dst. We have a patch pending to fix this upstream, but in the meantime, work around it here.
* device: remove allowedips before individual peersJason A. Donenfeld2018-05-181-1/+1
| | | | This avoids an O(n^2) traversal in favor of an O(n) one.
* send: account for route-based MTUJason A. Donenfeld2018-04-151-0/+5
| | | | | | | | | | It might be that a particular route has a different MTU than the interface, via `ip route add ... dev wg0 mtu 1281`, for example. In this case, it's important that we don't accidently pad beyond the end of the MTU. We accomplish that in this patch by carrying forward the MTU from the dst if it exists. We also add a unit test for this issue. Reported-by: Roman Mamedov <rm.wg@romanrm.net>
* device: let udev know what kind of device we areJason A. Donenfeld2018-01-301-0/+6
|
* global: year bumpJason A. Donenfeld2018-01-031-1/+1
|
* global: add SPDX tags to all filesGreg Kroah-Hartman2017-12-091-1/+4
| | | | | | | | | | | | | It's good to have SPDX identifiers in all files as the Linux kernel developers are working to add these identifiers to all files. Update all files with the correct SPDX license identifier based on the license text of the project or based on the license in the file itself. The SPDX identifier is a legally binding shorthand, which can be used instead of the full boiler plate text. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Modified-by: Jason A. Donenfeld <Jason@zx2c4.com>
* device: clear last handshake timer on ifdownJason A. Donenfeld2017-11-291-0/+1
| | | | | | | Otherwise new handshakes might not occur immediately when the interface goes up and down. Also initialize peers to having a proper zeroed handshake jiffies.
* device: do not clear keys during sleep on AndroidJason A. Donenfeld2017-11-281-4/+4
| | | | | | This makes sense for the security model of laptops, but not for clicking phones on and off, where we actually want to be able to handle incoming packets.
* device: uninitialize socket first in destructionJason A. Donenfeld2017-11-171-2/+2
|
* socket: only free socket after successful creation of newJason A. Donenfeld2017-11-171-3/+3
| | | | | | | | | | | | | | When an interface is down, the socket port can change freely. A socket will be allocated when the interface comes up, and if a socket can't be allocated, the interface doesn't come up. However, a socket port can change while the interface is up. In this case, if a new socket with a new port cannot be allocated, it's important to keep the interface in a consistent state. The choices are either to bring down the interface or to preserve the old socket. This patch implements the latter. Reported-by: Marc-Antoine Perennou <keruspe@exherbo.org>
* allowedips: rename from routingtableJason A. Donenfeld2017-11-101-3/+3
| | | | Makes it more clear that this _not_ a routing table replacement.
* device: wait for all peers to be freed before destroyingJason A. Donenfeld2017-11-101-0/+1
|
* device: please lockdepJason A. Donenfeld2017-11-031-1/+1
|
* device: expand scope of destruct lockJason A. Donenfeld2017-10-311-2/+2
|
* device: only take reference if netns is differentJason A. Donenfeld2017-10-311-9/+50
| | | | | | If we take two references, the namespace and the device are never freed in the usual manner. We should thus only take a reference to another namespace when it is a different namespace from our own.
* global: style nitsJason A. Donenfeld2017-10-311-4/+8
|
* global: infuriating kernel iterator styleJason A. Donenfeld2017-10-311-4/+4
| | | | | | | | | | | | | | | | | One types: for (i = 0 ... So one should also type: for_each_obj (obj ... But the upstream kernel style guidelines are insane, and so we must instead do: for_each_obj(obj ... Ugly, but one must choose his battles wisely.
* peer: get rid of peer_for_each magicJason A. Donenfeld2017-10-311-6/+12
| | | | | | | | | Since the peer list is protected by the device_update_lock, and since items are removed from the peer list before putting their final reference, we don't actually need to take a reference when iterating. This allows us to simplify the macro considerably. Suggested-by: Johannes Berg <johannes@sipsolutions.net>
* global: accept decent check_patch.pl suggestionsJason A. Donenfeld2017-10-311-1/+2
|
* device: no need to take lock for integer comparisonJason A. Donenfeld2017-10-171-4/+3
|
* device: our use of queues means this check is worthlessJason A. Donenfeld2017-10-171-6/+0
|
* compat: move version logic to compat.h and out of main .cJason A. Donenfeld2017-10-111-6/+6
|
* routingtable: only use device's mutex, not a special rt oneJason A. Donenfeld2017-10-091-1/+1
|
* queueing: use ptr_ring instead of linked listsJason A. Donenfeld2017-10-051-6/+6
|
* global: add space around variable declarationsJason A. Donenfeld2017-10-031-1/+2
|
* netlink: switch from ioctl to netlink for configurationJason A. Donenfeld2017-10-021-21/+3
|
* timers: convert to use netif_runningJason A. Donenfeld2017-09-241-2/+1
|
* queue: entirely rework parallel systemJason A. Donenfeld2017-09-181-47/+37
| | | | | | | | | | This removes our dependency on padata and moves to a different mode of multiprocessing that is more efficient. This began as Samuel Holland's GSoC project and was gradually reworked/redesigned/rebased into this present commit, which is a combination of his initial contribution and my subsequent rewriting and redesigning.
* device: IFF_NO_QUEUE is a private flag, not a public oneJason A. Donenfeld2017-09-111-1/+1
|
* timers: rename confusingly named functions and variablesJason A. Donenfeld2017-08-041-2/+2
| | | | Suggested-by: Mathias Hall-Andersen <mathias@hall-andersen.dk>
* global: use pointer to net_deviceJason A. Donenfeld2017-07-201-0/+1
| | | | | | DaveM prefers it to be this way per [1]. [1] http://www.spinics.net/lists/netdev/msg443992.html
* device: support 4.13's extact newlink paramJason A. Donenfeld2017-07-201-1/+1
|
* device: cleanup register_netdev logicJason A. Donenfeld2017-07-061-15/+7
|
* compat: priv_destructor got backportedJason A. Donenfeld2017-07-061-5/+7
|
* global: cleanup IP header checkingJason A. Donenfeld2017-06-261-1/+7
| | | | This way is more correct and ensures we're within the skb head.
* device: remove icmp conntrack hacksJason A. Donenfeld2017-06-261-33/+6
| | | | This logic belongs upstream.
* device: avoid double icmp send on routing loopJason A. Donenfeld2017-06-261-1/+0
|
* ratelimiter: rewrite from scratchJason A. Donenfeld2017-06-261-3/+5
| | | | | | | This not only removes the depenency on x_tables, but it also gives us much better performance and memory usage. Now, systems are able to have millions of WireGuard interfaces, without having to worry about a thundering herd of garbage collection.
* device: only use one sleep notifierJason A. Donenfeld2017-06-241-19/+26
| | | | | This greatly improves performance when adding and removing interfaces, since the power registration function does a linear search each time.