On Wed, Aug 11, 2021 at 4:08 PM Martin KaFai Lau <kafai@xxxxxx> wrote: > Some of the function names are hardly changed. This is obviously wrong for two reasons: 1. Kernel developers did change them. As a quick example, tcp_retransmit_skb() has been changed, we do have reasons to only trace __tcp_retransmit_skb() instead. 2. Even if kernel developers never did, compilers can do inline too. For example, I see nothing to stop compiler to inline tcp_transmit_skb() which is static and only calls __tcp_transmit_skb(). You explicitly mark bpf_fentry_test1() as noinline, don't you? I understand you are eager to promote ebpf, however, please keep reasonable on facts. Thanks.
