igb: Fix not adding filter elements to the list
authorVinicius Costa Gomes <vinicius.gomes@intel.com>
Tue, 10 Apr 2018 17:49:50 +0000 (10:49 -0700)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Wed, 25 Apr 2018 16:34:54 +0000 (09:34 -0700)
Because the order of the parameters passes to 'hlist_add_behind()' was
inverted, the 'parent' node was added "behind" the 'input', as input
is not in the list, this causes the 'input' node to be lost.

Fixes: 0e71def25281 ("igb: add support of RX network flow classification")
Signed-off-by: Vinicius Costa Gomes <vinicius.gomes@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/igb/igb_ethtool.c

index e77ba0d5866d0cc64aae1aae57568638dc95e751..5975d432836f8d35488358f9322e6b0020c394d0 100644 (file)
@@ -2865,7 +2865,7 @@ static int igb_update_ethtool_nfc_entry(struct igb_adapter *adapter,
 
        /* add filter to the list */
        if (parent)
-               hlist_add_behind(&parent->nfc_node, &input->nfc_node);
+               hlist_add_behind(&input->nfc_node, &parent->nfc_node);
        else
                hlist_add_head(&input->nfc_node, &adapter->nfc_filter_list);