Alexei Starovoitov <alexei.starovoi...@gmail.com> writes:
> On Mon, Oct 09, 2017 at 10:35:47PM -0700, Cong Wang wrote:
[...]
>> + trace_tcp_retransmit_skb(sk, skb, segs);
>
> I'm happy to see new tracepoints being added to tcp stack, but I'm concerned
> with practical usability of them.
> Like the above tracepoint definition makes it not very useful from bpf point
> of view,
> since 'sk' pointer is not recored by as part of the tracepoint.
> In bpf/tracing world we prefer tracepoints to have raw pointers recorded
> in TP_STRUCT__entry() and _not_ printed in TP_printk()
> (since pointers are useless for userspace).
Ack.
Also could the TP_printk also use the socket cookies so they can get
associated with netlink dumps and as such also be associated to user
space processes? It could help against races while trying to associate
the socket with a process. ss already supports dumping those cookies
with -e.
The corresponding commit would be:
commit 33cf7c90fe2f97afb1cadaa0cfb782cb9d1b9ee2
Author: Eric Dumazet <eduma...@google.com>
Date: Wed Mar 11 18:53:14 2015 -0700
net: add real socket cookies
Right now they only get set when needed but as Eric already mentioned in
his commit log, this could be refined.
[...]
