diff options
| author | Jesper Dangaard Brouer <brouer@redhat.com> | 2017-08-29 16:37:40 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-08-29 10:51:28 -0700 |
| commit | c31e5a4876cdfbfcc28c4fb201ad872c65671067 (patch) | |
| tree | 2796a7c5b638c813aaed663acb6a0d79b6b9855d /include/trace | |
| parent | d0fcece7704c935a9c8740bddff21575048f27ba (diff) | |
| download | op-kernel-dev-c31e5a4876cdfbfcc28c4fb201ad872c65671067.zip op-kernel-dev-c31e5a4876cdfbfcc28c4fb201ad872c65671067.tar.gz | |
xdp: remove redundant argument to trace_xdp_redirect
Supplying the action argument XDP_REDIRECT to the tracepoint xdp_redirect
is redundant as it is only called in-case this action was specified.
Remove the argument, but keep "act" member of the tracepoint struct and
populate it with XDP_REDIRECT. This makes it easier to write a common bpf_prog
processing events.
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/trace')
| -rw-r--r-- | include/trace/events/xdp.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/trace/events/xdp.h b/include/trace/events/xdp.h index 27cf2ef..f684f3b 100644 --- a/include/trace/events/xdp.h +++ b/include/trace/events/xdp.h @@ -52,10 +52,10 @@ TRACE_EVENT(xdp_exception, TRACE_EVENT(xdp_redirect, TP_PROTO(const struct net_device *dev, - const struct bpf_prog *xdp, u32 act, + const struct bpf_prog *xdp, int to_index, int err), - TP_ARGS(dev, xdp, act, to_index, err), + TP_ARGS(dev, xdp, to_index, err), TP_STRUCT__entry( __array(u8, prog_tag, 8) @@ -68,7 +68,7 @@ TRACE_EVENT(xdp_redirect, TP_fast_assign( BUILD_BUG_ON(sizeof(__entry->prog_tag) != sizeof(xdp->tag)); memcpy(__entry->prog_tag, xdp->tag, sizeof(xdp->tag)); - __entry->act = act; + __entry->act = XDP_REDIRECT; __entry->ifindex = dev->ifindex; __entry->to_index = to_index; __entry->err = err; |
