ixgbe: fix parsing of TC actions for HW offload
authorOndřej Hlavatý <ohlavaty@redhat.com>
Thu, 31 May 2018 21:21:04 +0000 (23:21 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 1 Jun 2018 03:01:00 +0000 (23:01 -0400)
commit16e6653c8259ca5383126e1c6bcf7fe062af87fc
tree8452041e189fb54607e7942e00d0a04214b4f691
parent8005b09d99fac78e6f5fb9da30b5ae94840af03b
ixgbe: fix parsing of TC actions for HW offload

The previous code was optimistic, accepting the offload of whole action
chain when there was a single known action (drop/redirect). This results
in offloading a rule which should not be offloaded, because its behavior
cannot be reproduced in the hardware.

For example:

$ tc filter add dev eno1 parent ffff: protocol ip \
    u32 ht 800: order 1 match tcp src 42 FFFF \
    action mirred egress mirror dev enp1s16 pipe \
    drop

The controller is unable to mirror the packet to a VF, but still
offloads the rule by dropping the packet.

Change the approach of the function to a pessimistic one, rejecting the
chain when an unknown action is found. This is better suited for future
extensions.

Note that both recognized actions always return TC_ACT_SHOT, therefore
it is safe to ignore actions behind them.

Signed-off-by: Ondřej Hlavatý <ohlavaty@redhat.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c