net/i40e: fix double flow mark action check
authorMesut Ali Ergin <mesut.a.ergin@intel.com>
Thu, 25 Jul 2019 21:50:49 +0000 (14:50 -0700)
committerFerruh Yigit <ferruh.yigit@intel.com>
Fri, 26 Jul 2019 11:24:32 +0000 (13:24 +0200)
commit3d6666ac97d790660067218c900a1c642c55fd26
tree0c2d2aee3a873873b0db6153f63e0c02dcd219f2
parent43b29221c1f13189f41d5200027a22f11149dfee
net/i40e: fix double flow mark action check

This commit fixes an issue with the error checking in flow
MARK action. Previously, (ANY + MARK) would fail, as the
(mark_spec == 0) condition would cause an early error return,
however really it is (mark_spec != 0) that should cause the
early error return.

Flipping the binary comparison corrects the behaviour, and
(ANY + MARK) now succeeds, while (MARK + MARK) fails.

Fixes: 0bbcfc706a2b ("net/i40e: support MARK and RSS flow action")

Suggested-by: Harry van Haaren <harry.van.haaren@intel.com>
Signed-off-by: Mesut Ali Ergin <mesut.a.ergin@intel.com>
Acked-by: Qi Zhang <qi.z.zhang@intel.com>
drivers/net/i40e/i40e_flow.c