]> git.baikalelectronics.ru Git - kernel.git/commit
net: mscc: ocelot: restrict tc-trap actions to VCAP IS2 lookup 0
authorVladimir Oltean <vladimir.oltean@nxp.com>
Wed, 4 May 2022 23:55:02 +0000 (02:55 +0300)
committerJakub Kicinski <kuba@kernel.org>
Fri, 6 May 2022 02:15:16 +0000 (19:15 -0700)
commit3a28009d805830055e184c287324907e5b515963
tree08b77dd2e3178df36d2ffdc4752b1f60512d764c
parent31041f067cbaec0b3bb662214fddc824483993b7
net: mscc: ocelot: restrict tc-trap actions to VCAP IS2 lookup 0

Once the CPU port was added to the destination port mask of a packet, it
can never be cleared, so even packets marked as dropped by the MASK_MODE
of a VCAP IS2 filter will still reach it. This is why we need the
OCELOT_POLICER_DISCARD to "kill dropped packets dead" and make software
stop seeing them.

We disallow policer rules from being put on any other chain than the one
for the first lookup, but we don't do this for "drop" rules, although we
should. This change is merely ascertaining that the rules dont't
(completely) work and letting the user know.

The blamed commit is the one that introduced the multi-chain architecture
in ocelot. Prior to that, we should have always offloaded the filters to
VCAP IS2 lookup 0, where they did work.

Fixes: c67a410d668d ("net: mscc: ocelot: create TCAM skeleton from tc filter chains")
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/mscc/ocelot_flower.c