ixgbe/base: fix setting flow director flag twice
authorWenzhuo Lu <wenzhuo.lu@intel.com>
Sun, 14 Feb 2016 08:55:02 +0000 (16:55 +0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Wed, 16 Mar 2016 16:08:23 +0000 (17:08 +0100)
commite2f368a4bfde2a3e2da54ca5021208d9261c3183
tree473d0612cbe44b51d0c185459c9a15c291caaabd
parent0c14952f173e0830dc37a8dbc30e3e7c7be0087b
ixgbe/base: fix setting flow director flag twice

Do not set FDIRCTRL.DROP_NO_MATCH in ixgbe_init_fdir_perfect_82599(),
this bit is already set in ixgbe_set_fdir_drop_queue_82599() which
makes more sense for drivers that call that function.

This resolves an issue where packets were being dropped when switching
to perfect filters mode.

Setting this bit makes no sense in perfect filters mode for the
driver as we do not want to route all packets that don't match an FDIR
rule to a single queue and instead fall back to RSS.
Drivers that need this bit set can call ixgbe_set_fdir_drop_queue_82599()
and the ones that don't, can preserve the old behavior.

Fixes: 2241ce281646 ("ixgbe/base: add flow director drop queue")
Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
doc/guides/rel_notes/release_16_04.rst
drivers/net/ixgbe/base/ixgbe_82599.c