On Fri, 18 Feb 2022 16:31:25 +0800 menglong8.dong@xxxxxxxxx wrote: > +static inline void tcp_drop(struct sock *sk, struct sk_buff *skb) The patches are marked as changes requested in patchwork. I presume Dave also thinks this static inline is best avoided. Is this function really not getting inlined? Otherwise please repost with the inline keyword removed. > +{ > + tcp_drop_reason(sk, skb, SKB_DROP_REASON_NOT_SPECIFIED); > }