diff options
author | Yafang Shao <laoar.shao@gmail.com> | 2018-05-29 23:27:31 +0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-05-31 13:20:47 -0400 |
commit | 3d97d88e8091f3501e016f6b4ce45a32c4b8f2f6 (patch) | |
tree | d5337f3bf77ad4677098f20351e91af3af325adf /include/trace | |
parent | 0c3a4cf84fb9ab17a4c9d3b34cfd6098887dd998 (diff) | |
download | linux-stable-3d97d88e8091f3501e016f6b4ce45a32c4b8f2f6.tar.gz linux-stable-3d97d88e8091f3501e016f6b4ce45a32c4b8f2f6.tar.bz2 linux-stable-3d97d88e8091f3501e016f6b4ce45a32c4b8f2f6.zip |
tcp: minor optimization around tcp_hdr() usage in receive path
This is additional to the
commit ea1627c20c34 ("tcp: minor optimizations around tcp_hdr() usage").
At this point, skb->data is same with tcp_hdr() as tcp header has not
been pulled yet. So use the less expensive one to get the tcp header.
Remove the third parameter of tcp_rcv_established() and put it into
the function body.
Furthermore, the local variables are listed as a reverse christmas tree :)
Cc: Eric Dumazet <edumazet@google.com>
Signed-off-by: Yafang Shao <laoar.shao@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/trace')
-rw-r--r-- | include/trace/events/tcp.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/include/trace/events/tcp.h b/include/trace/events/tcp.h index 703abb6e11fa..ac55b328d61b 100644 --- a/include/trace/events/tcp.h +++ b/include/trace/events/tcp.h @@ -248,8 +248,9 @@ TRACE_EVENT(tcp_probe, ), TP_fast_assign( - const struct tcp_sock *tp = tcp_sk(sk); + const struct tcphdr *th = (const struct tcphdr *)skb->data; const struct inet_sock *inet = inet_sk(sk); + const struct tcp_sock *tp = tcp_sk(sk); memset(__entry->saddr, 0, sizeof(struct sockaddr_in6)); memset(__entry->daddr, 0, sizeof(struct sockaddr_in6)); @@ -261,7 +262,7 @@ TRACE_EVENT(tcp_probe, __entry->dport = ntohs(inet->inet_dport); __entry->mark = skb->mark; - __entry->data_len = skb->len - tcp_hdrlen(skb); + __entry->data_len = skb->len - __tcp_hdrlen(th); __entry->snd_nxt = tp->snd_nxt; __entry->snd_una = tp->snd_una; __entry->snd_cwnd = tp->snd_cwnd; |