diff options
| author |   Daniel Borkmann <daniel@iogearbox.net> | 2016-09-12 23:38:43 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-09-15 19:29:47 -0400 |
| commit | f53d8c7b18faf1bd361abe91f3c4bcbb21d0c985 (patch) | |
| tree | 45d072e34316e66eca8a7e93c4d9be101347a9c5 /net/sched | |
| parent | bpf: drop unnecessary test in cls_bpf_classify and tcf_bpf (diff) | |
| download | linux-dev-f53d8c7b18faf1bd361abe91f3c4bcbb21d0c985.tar.xz linux-dev-f53d8c7b18faf1bd361abe91f3c4bcbb21d0c985.zip | |
bpf: use skb_at_tc_ingress helper in tcf_bpf
We have a small skb_at_tc_ingress() helper for testing for ingress, so
make use of it. cls_bpf already uses it and so should act_bpf.
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/sched/act_bpf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sched/act_bpf.c b/net/sched/act_bpf.c index 78400defa790..1d3960033f61 100644 --- a/net/sched/act_bpf.c +++ b/net/sched/act_bpf.c @@ -39,10 +39,10 @@ static struct tc_action_ops act_bpf_ops; static int tcf_bpf(struct sk_buff *skb, const struct tc_action *act, struct tcf_result *res) { + bool at_ingress = skb_at_tc_ingress(skb); struct tcf_bpf *prog = to_bpf(act); struct bpf_prog *filter; int action, filter_res; - bool at_ingress = G_TC_AT(skb->tc_verd) & AT_INGRESS; tcf_lastuse_update(&prog->tcf_tm); bstats_cpu_update(this_cpu_ptr(prog->common.cpu_bstats), skb); |