diff options
author | Eric Dumazet <edumazet@google.com> | 2018-11-27 18:03:21 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-11-30 16:28:03 -0500 |
commit | 6015c71e656bb6895b416c31a8b7db457e45cecf (patch) | |
tree | ba31e8ae21f52048e475daa33f6bb86df5bfc3b5 /net/ipv4/tcp_output.c | |
parent | 2f69555315ad7dc1ac37366b2ac2429e2d24d444 (diff) |
tcp: md5: add tcp_md5_needed jump label
Most linux hosts never setup TCP MD5 keys. We can avoid a
cache line miss (accessing tp->md5ig_info) on RX and TX
using a jump label.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_output.c')
-rw-r--r-- | net/ipv4/tcp_output.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index e4c1e51b18c1..d3b691f3a9e8 100644 --- a/net/ipv4/tcp_output.c +++ b/net/ipv4/tcp_output.c | |||
@@ -594,7 +594,8 @@ static unsigned int tcp_syn_options(struct sock *sk, struct sk_buff *skb, | |||
594 | 594 | ||
595 | *md5 = NULL; | 595 | *md5 = NULL; |
596 | #ifdef CONFIG_TCP_MD5SIG | 596 | #ifdef CONFIG_TCP_MD5SIG |
597 | if (unlikely(rcu_access_pointer(tp->md5sig_info))) { | 597 | if (static_key_false(&tcp_md5_needed) && |
598 | rcu_access_pointer(tp->md5sig_info)) { | ||
598 | *md5 = tp->af_specific->md5_lookup(sk, sk); | 599 | *md5 = tp->af_specific->md5_lookup(sk, sk); |
599 | if (*md5) { | 600 | if (*md5) { |
600 | opts->options |= OPTION_MD5; | 601 | opts->options |= OPTION_MD5; |
@@ -730,7 +731,8 @@ static unsigned int tcp_established_options(struct sock *sk, struct sk_buff *skb | |||
730 | 731 | ||
731 | *md5 = NULL; | 732 | *md5 = NULL; |
732 | #ifdef CONFIG_TCP_MD5SIG | 733 | #ifdef CONFIG_TCP_MD5SIG |
733 | if (unlikely(rcu_access_pointer(tp->md5sig_info))) { | 734 | if (static_key_false(&tcp_md5_needed) && |
735 | rcu_access_pointer(tp->md5sig_info)) { | ||
734 | *md5 = tp->af_specific->md5_lookup(sk, sk); | 736 | *md5 = tp->af_specific->md5_lookup(sk, sk); |
735 | if (*md5) { | 737 | if (*md5) { |
736 | opts->options |= OPTION_MD5; | 738 | opts->options |= OPTION_MD5; |