net/i40e: fix Rx packet statistics

[dpdk.git] / drivers / net / bonding / rte_eth_bond_api.c

diff --git a/drivers/net/bonding/rte_eth_bond_api.c b/drivers/net/bonding/rte_eth_bond_api.c
index 55c8e31..5140ef1 100644 (file)
--- a/drivers/net/bonding/rte_eth_bond_api.c
+++ b/drivers/net/bonding/rte_eth_bond_api.c
@@ -6,7 +6,7 @@
 
 #include <rte_mbuf.h>
 #include <rte_malloc.h>
-#include <rte_ethdev_driver.h>
+#include <ethdev_driver.h>
 #include <rte_tcp.h>
 #include <rte_bus_vdev.h>
 #include <rte_kvargs.h>
@@ -56,19 +56,25 @@ check_for_master_bonded_ethdev(const struct rte_eth_dev *eth_dev)
 }
 
 int
-valid_slave_port_id(uint16_t port_id, uint8_t mode)
+valid_slave_port_id(struct bond_dev_private *internals, uint16_t slave_port_id)
 {
-       RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -1);
+       RTE_ETH_VALID_PORTID_OR_ERR_RET(slave_port_id, -1);
 
-       /* Verify that port_id refers to a non bonded port */
-       if (check_for_bonded_ethdev(&rte_eth_devices[port_id]) == 0 &&
-                       mode == BONDING_MODE_8023AD) {
+       /* Verify that slave_port_id refers to a non bonded port */
+       if (check_for_bonded_ethdev(&rte_eth_devices[slave_port_id]) == 0 &&
+                       internals->mode == BONDING_MODE_8023AD) {
                RTE_BOND_LOG(ERR, "Cannot add slave to bonded device in 802.3ad"
                                " mode as slave is also a bonded device, only "
                                "physical devices can be support in this mode.");
                return -1;
        }
 
+       if (internals->port_id == slave_port_id) {
+               RTE_BOND_LOG(ERR,
+                       "Cannot add the bonded device itself as its slave.");
+               return -1;
+       }
+
        return 0;
 }
 
@@ -284,6 +290,7 @@ eth_bond_slave_inherit_dev_info_rx_first(struct bond_dev_private *internals,
        struct rte_eth_rxconf *rxconf_i = &internals->default_rxconf;
 
        internals->reta_size = di->reta_size;
+       internals->rss_key_len = di->hash_key_size;
 
        /* Inherit Rx offload capabilities from the first slave device */
        internals->rx_offload_capa = di->rx_offload_capa;
@@ -379,6 +386,11 @@ eth_bond_slave_inherit_dev_info_rx_next(struct bond_dev_private *internals,
         */
        if (internals->reta_size > di->reta_size)
                internals->reta_size = di->reta_size;
+       if (internals->rss_key_len > di->hash_key_size) {
+               RTE_BOND_LOG(WARNING, "slave has different rss key size, "
+                               "configuring rss may fail");
+               internals->rss_key_len = di->hash_key_size;
+       }
 
        if (!internals->max_rx_pktlen &&
            di->max_rx_pktlen < internals->candidate_max_rx_pktlen)
@@ -456,7 +468,7 @@ __eth_bond_slave_add_lock_free(uint16_t bonded_port_id, uint16_t slave_port_id)
        bonded_eth_dev = &rte_eth_devices[bonded_port_id];
        internals = bonded_eth_dev->data->dev_private;
 
-       if (valid_slave_port_id(slave_port_id, internals->mode) != 0)
+       if (valid_slave_port_id(internals, slave_port_id) != 0)
                return -1;
 
        slave_eth_dev = &rte_eth_devices[slave_port_id];
@@ -605,13 +617,15 @@ rte_eth_bond_slave_add(uint16_t bonded_port_id, uint16_t slave_port_id)
 
        int retval;
 
-       /* Verify that port id's are valid bonded and slave ports */
        if (valid_bonded_port_id(bonded_port_id) != 0)
                return -1;
 
        bonded_eth_dev = &rte_eth_devices[bonded_port_id];
        internals = bonded_eth_dev->data->dev_private;
 
+       if (valid_slave_port_id(internals, slave_port_id) != 0)
+               return -1;
+
        rte_spinlock_lock(&internals->lock);
 
        retval = __eth_bond_slave_add_lock_free(bonded_port_id, slave_port_id);
@@ -635,7 +649,7 @@ __eth_bond_slave_remove_lock_free(uint16_t bonded_port_id,
        bonded_eth_dev = &rte_eth_devices[bonded_port_id];
        internals = bonded_eth_dev->data->dev_private;
 
-       if (valid_slave_port_id(slave_port_id, internals->mode) < 0)
+       if (valid_slave_port_id(internals, slave_port_id) < 0)
                return -1;
 
        /* first remove from active slave list */
@@ -783,7 +797,7 @@ rte_eth_bond_primary_set(uint16_t bonded_port_id, uint16_t slave_port_id)
 
        internals = rte_eth_devices[bonded_port_id].data->dev_private;
 
-       if (valid_slave_port_id(slave_port_id, internals->mode) != 0)
+       if (valid_slave_port_id(internals, slave_port_id) != 0)
                return -1;
 
        internals->user_defined_primary_port = 1;