summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorJason A. Donenfeld <Jason@zx2c4.com>2021-02-07 14:47:13 +0100
committerJason A. Donenfeld <Jason@zx2c4.com>2021-02-08 15:18:39 +0100
commit8420c794cd17d86fea5f713ce52617b4ecf0ec29 (patch)
tree3cecd73899c51cb4e98e4d1d3af87f86142e5634
parentbb0ed3f6d91f19447732f667dac077dfd7338d57 (diff)
peer: put frequently used members above cache lines
The is_dead boolean is checked for every single packet, while the internal_id member is used basically only for pr_debug messages. So it makes sense to hoist up is_dead into some space formerly unused by a struct hole, while demoting internal_api to below the lowest struct cache line. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
-rw-r--r--src/peer.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/peer.h b/src/peer.h
index 23af409..aaff8de 100644
--- a/src/peer.h
+++ b/src/peer.h
@@ -39,6 +39,7 @@ struct wg_peer {
struct crypt_queue tx_queue, rx_queue;
struct sk_buff_head staged_packet_queue;
int serial_work_cpu;
+ bool is_dead;
struct noise_keypairs keypairs;
struct endpoint endpoint;
struct dst_cache endpoint_cache;
@@ -61,9 +62,8 @@ struct wg_peer {
struct rcu_head rcu;
struct list_head peer_list;
struct list_head allowedips_list;
- u64 internal_id;
struct napi_struct napi;
- bool is_dead;
+ u64 internal_id;
};
struct wg_peer *wg_peer_create(struct wg_device *wg,