diff options
author | Nir Dotan <nird@mellanox.com> | 2018-08-03 15:57:44 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-08-03 12:28:01 -0700 |
commit | caebd1b389708bf3d0465be829480fc706a68720 (patch) | |
tree | efe603173134e9aa9dda6043038d0d22edf6b454 /drivers/net/wan | |
parent | 7cc6169493990dec488eda0a3f6612729ca25e81 (diff) | |
download | lwn-caebd1b389708bf3d0465be829480fc706a68720.tar.gz lwn-caebd1b389708bf3d0465be829480fc706a68720.zip |
mlxsw: core_acl_flex_actions: Remove redundant mirror resource destruction
In previous patch mlxsw_afa_resource_del() was added to avoid a duplicate
resource detruction scenario.
For mirror actions, such duplicate destruction leads to a crash as in:
# tc qdisc add dev swp49 ingress
# tc filter add dev swp49 parent ffff: \
protocol ip chain 100 pref 10 \
flower skip_sw dst_ip 192.168.101.1 action drop
# tc filter add dev swp49 parent ffff: \
protocol ip pref 10 \
flower skip_sw dst_ip 192.168.101.1 action goto chain 100 \
action mirred egress mirror dev swp4
Therefore add a call to mlxsw_afa_resource_del() in
mlxsw_afa_mirror_destroy() in order to clear that resource
from rule's resources.
Fixes: d0d13c1858a1 ("mlxsw: spectrum_acl: Add support for mirror action")
Signed-off-by: Nir Dotan <nird@mellanox.com>
Reviewed-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wan')
0 files changed, 0 insertions, 0 deletions