net: dev: use kfree_skb_reason() for do_xdp_generic()

Replace kfree_skb() used in do_xdp_generic() with kfree_skb_reason().
The drop reason SKB_DROP_REASON_XDP is introduced for this case.

Signed-off-by: Menglong Dong <imagedong@tencent.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h
index 44ecbff..7a38d0f 100644 (file)
--- a/include/linux/skbuff.h
+++ b/include/linux/skbuff.h
@@ -405,6 +405,7 @@ enum skb_drop_reason {
                                         * full (see netdev_max_backlog in
                                         * net.rst) or RPS flow limit
                                         */
+       SKB_DROP_REASON_XDP,            /* dropped by XDP in input path */
        SKB_DROP_REASON_MAX,
 };
 

diff --git a/include/trace/events/skb.h b/include/trace/events/skb.h
index 3bb90ca..8c4c343 100644 (file)
--- a/include/trace/events/skb.h
+++ b/include/trace/events/skb.h
@@ -48,6 +48,7 @@
        EM(SKB_DROP_REASON_TC_EGRESS, TC_EGRESS)                \
        EM(SKB_DROP_REASON_QDISC_DROP, QDISC_DROP)              \
        EM(SKB_DROP_REASON_CPU_BACKLOG, CPU_BACKLOG)            \
+       EM(SKB_DROP_REASON_XDP, XDP)                            \
        EMe(SKB_DROP_REASON_MAX, MAX)
 
 #undef EM

diff --git a/net/core/dev.c b/net/core/dev.c
index 460ac94..f449834 100644 (file)
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -4825,7 +4825,7 @@ int do_xdp_generic(struct bpf_prog *xdp_prog, struct sk_buff *skb)
        }
        return XDP_PASS;
 out_redir:
-       kfree_skb(skb);
+       kfree_skb_reason(skb, SKB_DROP_REASON_XDP);
        return XDP_DROP;
 }
 EXPORT_SYMBOL_GPL(do_xdp_generic);