On 16-07-21 04:19 AM, Jiri Pirko wrote:
From: Jiri Pirko <[email protected]>This patchset introduces tc matchall classifier and its offload to Spectrum hardware. In combination with mirred action, defined port mirroring setup is offloaded by mlxsw/spectrum driver. The commands used for creating mirror ports: # ingress mirroring using matchall tc qdisc add dev eth25 handle ffff: ingress tc filter add dev eth25 parent ffff: \ matchall skip_sw \ action mirred egress mirror \ dev eth27 # egress mirroring using matchall tc qdisc add dev eth25 handle 1: root prio tc filter add dev eth25 parent 1: \ matchall skip_sw \ action mirred egress mirror \ dev eth27
Kudos to Mellanox for all this nice work! I am assuming the chip is capable as well of doing mirroring via the ACL infrastructure and you are adding this classifier because you are going via the SPAN 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 ... BTW: I am not a big styling lawyer on netdev (I am normally the victim) but would be useful to look at some of these patches with that coding style in in mind (I think some of the chip specific patches had some style issue in function definition). again - kudos cheers, jamal
