On 4/3/15 4:48 PM, Daniel Borkmann wrote:
On 04/04/2015 01:26 AM, Daniel Borkmann wrote:
On 04/04/2015 01:11 AM, Alexei Starovoitov wrote:
...
nope. will take it back.
that doesn't work, since this check cannot be done in ingress_enqueue(),
because it sees the pointer to first filter only, so both TCQ_F_INGRESS
flag and CLS_REQUIRES_L2 flag need to be checked inside
So on a quick glance, we're calling into cls_bpf_classify() in
tp->classify()
(net/sched/cls_api.c +265), so all remaining filters in that list we're
traversing in cls_bpf_classify() are all BPF filters, no?
I see, you mean the classifier chain, not the chain of filters within
the cls_bpf classifier, ok.
yes. the chain of classifiers can have different types, so we
cannot check it once in ingress_enqueue().
As you said we can refactor it later.
--
To unsubscribe from this list: send the line "unsubscribe linux-api" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
