Hi Kartikeya, I recently started experimenting with the new tc-bpf API (which is great, many thanks!) and I wanted to share a potential problem I found. I'm using this "Fixes for TC-BPF series" thread to write about it, but it is not directly related to this patch set. According to the API summary given in https://lore.kernel.org/bpf/20210512103451.989420-3-memxor@xxxxxxxxx/, "It is advised that if the qdisc is operated on by many programs, then the program at least check that there are no other existing filters before deleting the clsact qdisc." In the example given, one should: /* set opts as NULL, as we're not really interested in * getting any info for a particular filter, but just * detecting its presence. */ r = bpf_tc_query(&hook, NULL); However, following in this summary, where bpf_tc_query is described, it is written that the opts argument cannot be NULL. And indeed, when I tried to use the example above, an error (EINVAL) was returned (as expected?) Am I missing something? Yaniv
