On 16-07-21 05:07 AM, Jiri Pirko wrote:
Thu, Jul 21, 2016 at 11:00:33AM CEST, j...@mojatatu.com wrote:
On 16-07-21 04:19 AM, Jiri Pirko wrote:
From: Jiri Pirko <j...@mellanox.com>

[..]

infrastructure. If answer is yes, thencould we have used
a classifier like u32 here?
i.e something like:
tc filter add dev eth25 xxxx protocol all \
u32 match u32 0 0 \
action mirred ...

That could be used. But I believe it is nicer to have explicit match-all
classifier for this case. That puts nice limit to what could be matched.


Iam indifferent. If you are planning to use u32 for ACL then it would
make sense to support span + ACL with the same classifier.

Could you point to that? checkpatch.pl does not say anything and I also
don't see anything.


scripts/Lindent (but you need to be a little careful with it)
Generally the coding style guide in Documentation is sufficient.

cheers,
jamal

Reply via email to