Commit b06337df authored by Jesper Dangaard Brouer's avatar Jesper Dangaard Brouer Committed by David S. Miller

xdp: make xdp tracepoints report bpf prog id instead of prog_tag

Given previous patch expose the map_id, it seems natural to also
report the bpf prog id.
Signed-off-by: default avatarJesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8d3b778f
...@@ -31,20 +31,19 @@ TRACE_EVENT(xdp_exception, ...@@ -31,20 +31,19 @@ TRACE_EVENT(xdp_exception,
TP_ARGS(dev, xdp, act), TP_ARGS(dev, xdp, act),
TP_STRUCT__entry( TP_STRUCT__entry(
__array(u8, prog_tag, 8) __field(int, prog_id)
__field(u32, act) __field(u32, act)
__field(int, ifindex) __field(int, ifindex)
), ),
TP_fast_assign( TP_fast_assign(
BUILD_BUG_ON(sizeof(__entry->prog_tag) != sizeof(xdp->tag)); __entry->prog_id = xdp->aux->id;
memcpy(__entry->prog_tag, xdp->tag, sizeof(xdp->tag));
__entry->act = act; __entry->act = act;
__entry->ifindex = dev->ifindex; __entry->ifindex = dev->ifindex;
), ),
TP_printk("prog=%s action=%s ifindex=%d", TP_printk("prog_id=%d action=%s ifindex=%d",
__print_hex_str(__entry->prog_tag, 8), __entry->prog_id,
__print_symbolic(__entry->act, __XDP_ACT_SYM_TAB), __print_symbolic(__entry->act, __XDP_ACT_SYM_TAB),
__entry->ifindex) __entry->ifindex)
); );
...@@ -59,7 +58,7 @@ DECLARE_EVENT_CLASS(xdp_redirect_template, ...@@ -59,7 +58,7 @@ DECLARE_EVENT_CLASS(xdp_redirect_template,
TP_ARGS(dev, xdp, to_ifindex, err, map, map_index), TP_ARGS(dev, xdp, to_ifindex, err, map, map_index),
TP_STRUCT__entry( TP_STRUCT__entry(
__array(u8, prog_tag, 8) __field(int, prog_id)
__field(u32, act) __field(u32, act)
__field(int, ifindex) __field(int, ifindex)
__field(int, err) __field(int, err)
...@@ -69,8 +68,7 @@ DECLARE_EVENT_CLASS(xdp_redirect_template, ...@@ -69,8 +68,7 @@ DECLARE_EVENT_CLASS(xdp_redirect_template,
), ),
TP_fast_assign( TP_fast_assign(
BUILD_BUG_ON(sizeof(__entry->prog_tag) != sizeof(xdp->tag)); __entry->prog_id = xdp->aux->id;
memcpy(__entry->prog_tag, xdp->tag, sizeof(xdp->tag));
__entry->act = XDP_REDIRECT; __entry->act = XDP_REDIRECT;
__entry->ifindex = dev->ifindex; __entry->ifindex = dev->ifindex;
__entry->err = err; __entry->err = err;
...@@ -79,9 +77,9 @@ DECLARE_EVENT_CLASS(xdp_redirect_template, ...@@ -79,9 +77,9 @@ DECLARE_EVENT_CLASS(xdp_redirect_template,
__entry->map_index = map_index; __entry->map_index = map_index;
), ),
TP_printk("prog=%s action=%s ifindex=%d to_ifindex=%d err=%d" TP_printk("prog_id=%d action=%s ifindex=%d to_ifindex=%d err=%d"
" map_id=%d map_index=%d", " map_id=%d map_index=%d",
__print_hex_str(__entry->prog_tag, 8), __entry->prog_id,
__print_symbolic(__entry->act, __XDP_ACT_SYM_TAB), __print_symbolic(__entry->act, __XDP_ACT_SYM_TAB),
__entry->ifindex, __entry->to_ifindex, __entry->ifindex, __entry->to_ifindex,
__entry->err, __entry->err,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment