net/ngbe: support MAC filters
[dpdk.git] / drivers / net / octeontx / octeontx_ethdev.c
index ff2e22c..4f1e368 100644 (file)
@@ -9,13 +9,14 @@
 #include <string.h>
 #include <unistd.h>
 
+#include <eventdev_pmd.h>
 #include <rte_alarm.h>
 #include <rte_branch_prediction.h>
 #include <rte_bus_vdev.h>
 #include <rte_cycles.h>
 #include <rte_debug.h>
-#include <rte_devargs.h>
 #include <rte_dev.h>
+#include <rte_devargs.h>
 #include <rte_kvargs.h>
 #include <rte_malloc.h>
 #include <rte_mbuf_pool_ops.h>
@@ -47,9 +48,9 @@ enum octeontx_link_speed {
        OCTEONTX_LINK_SPEED_RESERVE2
 };
 
-RTE_LOG_REGISTER(otx_net_logtype_mbox, pmd.net.octeontx.mbox, NOTICE);
-RTE_LOG_REGISTER(otx_net_logtype_init, pmd.net.octeontx.init, NOTICE);
-RTE_LOG_REGISTER(otx_net_logtype_driver, pmd.net.octeontx.driver, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(otx_net_logtype_mbox, mbox, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(otx_net_logtype_init, init, NOTICE);
+RTE_LOG_REGISTER_SUFFIX(otx_net_logtype_driver, driver, NOTICE);
 
 /* Parse integer from integer argument */
 static int
@@ -158,7 +159,7 @@ octeontx_link_status_print(struct rte_eth_dev *eth_dev,
                octeontx_log_info("Port %u: Link Up - speed %u Mbps - %s",
                          (eth_dev->data->port_id),
                          link->link_speed,
-                         link->link_duplex == ETH_LINK_FULL_DUPLEX ?
+                         link->link_duplex == RTE_ETH_LINK_FULL_DUPLEX ?
                          "full-duplex" : "half-duplex");
        else
                octeontx_log_info("Port %d: Link Down",
@@ -171,38 +172,38 @@ octeontx_link_status_update(struct octeontx_nic *nic,
 {
        memset(link, 0, sizeof(*link));
 
-       link->link_status = nic->link_up ? ETH_LINK_UP : ETH_LINK_DOWN;
+       link->link_status = nic->link_up ? RTE_ETH_LINK_UP : RTE_ETH_LINK_DOWN;
 
        switch (nic->speed) {
        case OCTEONTX_LINK_SPEED_SGMII:
-               link->link_speed = ETH_SPEED_NUM_1G;
+               link->link_speed = RTE_ETH_SPEED_NUM_1G;
                break;
 
        case OCTEONTX_LINK_SPEED_XAUI:
-               link->link_speed = ETH_SPEED_NUM_10G;
+               link->link_speed = RTE_ETH_SPEED_NUM_10G;
                break;
 
        case OCTEONTX_LINK_SPEED_RXAUI:
        case OCTEONTX_LINK_SPEED_10G_R:
-               link->link_speed = ETH_SPEED_NUM_10G;
+               link->link_speed = RTE_ETH_SPEED_NUM_10G;
                break;
        case OCTEONTX_LINK_SPEED_QSGMII:
-               link->link_speed = ETH_SPEED_NUM_5G;
+               link->link_speed = RTE_ETH_SPEED_NUM_5G;
                break;
        case OCTEONTX_LINK_SPEED_40G_R:
-               link->link_speed = ETH_SPEED_NUM_40G;
+               link->link_speed = RTE_ETH_SPEED_NUM_40G;
                break;
 
        case OCTEONTX_LINK_SPEED_RESERVE1:
        case OCTEONTX_LINK_SPEED_RESERVE2:
        default:
-               link->link_speed = ETH_SPEED_NUM_NONE;
+               link->link_speed = RTE_ETH_SPEED_NUM_NONE;
                octeontx_log_err("incorrect link speed %d", nic->speed);
                break;
        }
 
-       link->link_duplex = ETH_LINK_FULL_DUPLEX;
-       link->link_autoneg = ETH_LINK_AUTONEG;
+       link->link_duplex = RTE_ETH_LINK_FULL_DUPLEX;
+       link->link_autoneg = RTE_ETH_LINK_AUTONEG;
 }
 
 static void
@@ -355,20 +356,20 @@ octeontx_tx_offload_flags(struct rte_eth_dev *eth_dev)
        struct octeontx_nic *nic = octeontx_pmd_priv(eth_dev);
        uint16_t flags = 0;
 
-       if (nic->tx_offloads & DEV_TX_OFFLOAD_OUTER_IPV4_CKSUM ||
-           nic->tx_offloads & DEV_TX_OFFLOAD_OUTER_UDP_CKSUM)
+       if (nic->tx_offloads & RTE_ETH_TX_OFFLOAD_OUTER_IPV4_CKSUM ||
+           nic->tx_offloads & RTE_ETH_TX_OFFLOAD_OUTER_UDP_CKSUM)
                flags |= OCCTX_TX_OFFLOAD_OL3_OL4_CSUM_F;
 
-       if (nic->tx_offloads & DEV_TX_OFFLOAD_IPV4_CKSUM ||
-           nic->tx_offloads & DEV_TX_OFFLOAD_TCP_CKSUM ||
-           nic->tx_offloads & DEV_TX_OFFLOAD_UDP_CKSUM ||
-           nic->tx_offloads & DEV_TX_OFFLOAD_SCTP_CKSUM)
+       if (nic->tx_offloads & RTE_ETH_TX_OFFLOAD_IPV4_CKSUM ||
+           nic->tx_offloads & RTE_ETH_TX_OFFLOAD_TCP_CKSUM ||
+           nic->tx_offloads & RTE_ETH_TX_OFFLOAD_UDP_CKSUM ||
+           nic->tx_offloads & RTE_ETH_TX_OFFLOAD_SCTP_CKSUM)
                flags |= OCCTX_TX_OFFLOAD_L3_L4_CSUM_F;
 
-       if (!(nic->tx_offloads & DEV_TX_OFFLOAD_MBUF_FAST_FREE))
+       if (!(nic->tx_offloads & RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE))
                flags |= OCCTX_TX_OFFLOAD_MBUF_NOFF_F;
 
-       if (nic->tx_offloads & DEV_TX_OFFLOAD_MULTI_SEGS)
+       if (nic->tx_offloads & RTE_ETH_TX_OFFLOAD_MULTI_SEGS)
                flags |= OCCTX_TX_MULTI_SEG_F;
 
        return flags;
@@ -380,21 +381,21 @@ octeontx_rx_offload_flags(struct rte_eth_dev *eth_dev)
        struct octeontx_nic *nic = octeontx_pmd_priv(eth_dev);
        uint16_t flags = 0;
 
-       if (nic->rx_offloads & (DEV_RX_OFFLOAD_TCP_CKSUM |
-                        DEV_RX_OFFLOAD_UDP_CKSUM))
+       if (nic->rx_offloads & (RTE_ETH_RX_OFFLOAD_TCP_CKSUM |
+                        RTE_ETH_RX_OFFLOAD_UDP_CKSUM))
                flags |= OCCTX_RX_OFFLOAD_CSUM_F;
 
-       if (nic->rx_offloads & (DEV_RX_OFFLOAD_IPV4_CKSUM |
-                               DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM))
+       if (nic->rx_offloads & (RTE_ETH_RX_OFFLOAD_IPV4_CKSUM |
+                               RTE_ETH_RX_OFFLOAD_OUTER_IPV4_CKSUM))
                flags |= OCCTX_RX_OFFLOAD_CSUM_F;
 
-       if (nic->rx_offloads & DEV_RX_OFFLOAD_SCATTER) {
+       if (nic->rx_offloads & RTE_ETH_RX_OFFLOAD_SCATTER) {
                flags |= OCCTX_RX_MULTI_SEG_F;
                eth_dev->data->scattered_rx = 1;
                /* If scatter mode is enabled, TX should also be in multi
                 * seg mode, else memory leak will occur
                 */
-               nic->tx_offloads |= DEV_TX_OFFLOAD_MULTI_SEGS;
+               nic->tx_offloads |= RTE_ETH_TX_OFFLOAD_MULTI_SEGS;
        }
 
        return flags;
@@ -423,18 +424,18 @@ octeontx_dev_configure(struct rte_eth_dev *dev)
                return -EINVAL;
        }
 
-       if (rxmode->mq_mode != ETH_MQ_RX_NONE &&
-               rxmode->mq_mode != ETH_MQ_RX_RSS) {
+       if (rxmode->mq_mode != RTE_ETH_MQ_RX_NONE &&
+               rxmode->mq_mode != RTE_ETH_MQ_RX_RSS) {
                octeontx_log_err("unsupported rx qmode %d", rxmode->mq_mode);
                return -EINVAL;
        }
 
-       if (!(txmode->offloads & DEV_TX_OFFLOAD_MT_LOCKFREE)) {
+       if (!(txmode->offloads & RTE_ETH_TX_OFFLOAD_MT_LOCKFREE)) {
                PMD_INIT_LOG(NOTICE, "cant disable lockfree tx");
-               txmode->offloads |= DEV_TX_OFFLOAD_MT_LOCKFREE;
+               txmode->offloads |= RTE_ETH_TX_OFFLOAD_MT_LOCKFREE;
        }
 
-       if (conf->link_speeds & ETH_LINK_SPEED_FIXED) {
+       if (conf->link_speeds & RTE_ETH_LINK_SPEED_FIXED) {
                octeontx_log_err("setting link speed/duplex not supported");
                return -EINVAL;
        }
@@ -478,7 +479,7 @@ octeontx_dev_configure(struct rte_eth_dev *dev)
        return 0;
 }
 
-static void
+static int
 octeontx_dev_close(struct rte_eth_dev *dev)
 {
        struct octeontx_txq *txq = NULL;
@@ -487,6 +488,8 @@ octeontx_dev_close(struct rte_eth_dev *dev)
        int ret;
 
        PMD_INIT_FUNC_TRACE();
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
+               return 0;
 
        rte_event_dev_close(nic->evdev);
 
@@ -509,14 +512,9 @@ octeontx_dev_close(struct rte_eth_dev *dev)
                rte_free(txq);
        }
 
-       /* Free MAC address table */
-       rte_free(dev->data->mac_addrs);
-       dev->data->mac_addrs = NULL;
-
        octeontx_port_close(nic);
 
-       dev->tx_pkt_burst = NULL;
-       dev->rx_pkt_burst = NULL;
+       return 0;
 }
 
 static int
@@ -527,23 +525,19 @@ octeontx_dev_mtu_set(struct rte_eth_dev *eth_dev, uint16_t mtu)
        struct rte_eth_dev_data *data = eth_dev->data;
        int rc = 0;
 
-       /* Check if MTU is within the allowed range */
-       if (frame_size < OCCTX_MIN_FRS || frame_size > OCCTX_MAX_FRS)
-               return -EINVAL;
-
        buffsz = data->min_rx_buf_size - RTE_PKTMBUF_HEADROOM;
 
        /* Refuse MTU that requires the support of scattered packets
         * when this feature has not been enabled before.
         */
        if (data->dev_started && frame_size > buffsz &&
-           !(nic->rx_offloads & DEV_RX_OFFLOAD_SCATTER)) {
+           !(nic->rx_offloads & RTE_ETH_RX_OFFLOAD_SCATTER)) {
                octeontx_log_err("Scatter mode is disabled");
                return -EINVAL;
        }
 
        /* Check <seg size> * <max_seg>  >= max_frame */
-       if ((nic->rx_offloads & DEV_RX_OFFLOAD_SCATTER) &&
+       if ((nic->rx_offloads & RTE_ETH_RX_OFFLOAD_SCATTER)     &&
            (frame_size > buffsz * OCCTX_RX_NB_SEG_MAX))
                return -EINVAL;
 
@@ -555,13 +549,6 @@ octeontx_dev_mtu_set(struct rte_eth_dev *eth_dev, uint16_t mtu)
        if (rc)
                return rc;
 
-       if (frame_size > RTE_ETHER_MAX_LEN)
-               nic->rx_offloads |= DEV_RX_OFFLOAD_JUMBO_FRAME;
-       else
-               nic->rx_offloads &= ~DEV_RX_OFFLOAD_JUMBO_FRAME;
-
-       /* Update max_rx_pkt_len */
-       data->dev_conf.rxmode.max_rx_pkt_len = frame_size;
        octeontx_log_info("Received pkt beyond  maxlen %d will be dropped",
                          frame_size);
 
@@ -584,8 +571,8 @@ octeontx_recheck_rx_offloads(struct octeontx_rxq *rxq)
        buffsz = mbp_priv->mbuf_data_room_size - RTE_PKTMBUF_HEADROOM;
 
        /* Setup scatter mode if needed by jumbo */
-       if (data->dev_conf.rxmode.max_rx_pkt_len > buffsz) {
-               nic->rx_offloads |= DEV_RX_OFFLOAD_SCATTER;
+       if (data->mtu > buffsz) {
+               nic->rx_offloads |= RTE_ETH_RX_OFFLOAD_SCATTER;
                nic->rx_offload_flags |= octeontx_rx_offload_flags(eth_dev);
                nic->tx_offload_flags |= octeontx_tx_offload_flags(eth_dev);
        }
@@ -596,8 +583,8 @@ octeontx_recheck_rx_offloads(struct octeontx_rxq *rxq)
        evdev_priv->rx_offload_flags = nic->rx_offload_flags;
        evdev_priv->tx_offload_flags = nic->tx_offload_flags;
 
-       /* Setup MTU based on max_rx_pkt_len */
-       nic->mtu = data->dev_conf.rxmode.max_rx_pkt_len - OCCTX_L2_OVERHEAD;
+       /* Setup MTU */
+       nic->mtu = data->mtu;
 
        return 0;
 }
@@ -618,7 +605,7 @@ octeontx_dev_start(struct rte_eth_dev *dev)
                octeontx_recheck_rx_offloads(rxq);
        }
 
-       /* Setting up the mtu based on max_rx_pkt_len */
+       /* Setting up the mtu */
        ret = octeontx_dev_mtu_set(dev, nic->mtu);
        if (ret) {
                octeontx_log_err("Failed to set default MTU size %d", ret);
@@ -676,7 +663,7 @@ error:
        return ret;
 }
 
-static void
+static int
 octeontx_dev_stop(struct rte_eth_dev *dev)
 {
        struct octeontx_nic *nic = octeontx_pmd_priv(dev);
@@ -690,14 +677,14 @@ octeontx_dev_stop(struct rte_eth_dev *dev)
        if (ret < 0) {
                octeontx_log_err("failed to req stop port %d res=%d",
                                        nic->port_id, ret);
-               return;
+               return ret;
        }
 
        ret = octeontx_pki_port_stop(nic->port_id);
        if (ret < 0) {
                octeontx_log_err("failed to stop pki port %d res=%d",
                                        nic->port_id, ret);
-               return;
+               return ret;
        }
 
        ret = octeontx_pko_channel_stop(nic->base_ochan);
@@ -705,8 +692,10 @@ octeontx_dev_stop(struct rte_eth_dev *dev)
                octeontx_log_err("failed to stop channel %d VF%d %d %d",
                             nic->base_ochan, nic->port_id, nic->num_tx_queues,
                             ret);
-               return;
+               return ret;
        }
+
+       return 0;
 }
 
 static int
@@ -855,10 +844,10 @@ octeontx_dev_info(struct rte_eth_dev *dev,
        struct octeontx_nic *nic = octeontx_pmd_priv(dev);
 
        /* Autonegotiation may be disabled */
-       dev_info->speed_capa = ETH_LINK_SPEED_FIXED;
-       dev_info->speed_capa |= ETH_LINK_SPEED_10M | ETH_LINK_SPEED_100M |
-                       ETH_LINK_SPEED_1G | ETH_LINK_SPEED_10G |
-                       ETH_LINK_SPEED_40G;
+       dev_info->speed_capa = RTE_ETH_LINK_SPEED_FIXED;
+       dev_info->speed_capa |= RTE_ETH_LINK_SPEED_10M | RTE_ETH_LINK_SPEED_100M |
+                       RTE_ETH_LINK_SPEED_1G | RTE_ETH_LINK_SPEED_10G |
+                       RTE_ETH_LINK_SPEED_40G;
 
        /* Min/Max MTU supported */
        dev_info->min_rx_bufsize = OCCTX_MIN_FRS;
@@ -868,7 +857,6 @@ octeontx_dev_info(struct rte_eth_dev *dev,
 
        dev_info->max_mac_addrs =
                                octeontx_bgx_port_mac_entries_get(nic->port_id);
-       dev_info->max_rx_pktlen = PKI_MAX_PKTLEN;
        dev_info->max_rx_queues = 1;
        dev_info->max_tx_queues = PKO_MAX_NUM_DQ;
        dev_info->min_rx_bufsize = 0;
@@ -973,20 +961,18 @@ octeontx_dev_tx_queue_stop(struct rte_eth_dev *dev, uint16_t qidx)
 }
 
 static void
-octeontx_dev_tx_queue_release(void *tx_queue)
+octeontx_dev_tx_queue_release(struct rte_eth_dev *dev, uint16_t qid)
 {
-       struct octeontx_txq *txq = tx_queue;
        int res;
 
        PMD_INIT_FUNC_TRACE();
 
-       if (txq) {
-               res = octeontx_dev_tx_queue_stop(txq->eth_dev, txq->queue_id);
+       if (dev->data->tx_queues[qid]) {
+               res = octeontx_dev_tx_queue_stop(dev, qid);
                if (res < 0)
-                       octeontx_log_err("failed stop tx_queue(%d)\n",
-                                  txq->queue_id);
+                       octeontx_log_err("failed stop tx_queue(%d)\n", qid);
 
-               rte_free(txq);
+               rte_free(dev->data->tx_queues[qid]);
        }
 }
 
@@ -1015,7 +1001,7 @@ octeontx_dev_tx_queue_setup(struct rte_eth_dev *dev, uint16_t qidx,
        if (dev->data->tx_queues[qidx] != NULL) {
                PMD_TX_LOG(DEBUG, "freeing memory prior to re-allocation %d",
                                qidx);
-               octeontx_dev_tx_queue_release(dev->data->tx_queues[qidx]);
+               octeontx_dev_tx_queue_release(dev, qidx);
                dev->data->tx_queues[qidx] = NULL;
        }
 
@@ -1223,9 +1209,9 @@ octeontx_dev_rx_queue_setup(struct rte_eth_dev *dev, uint16_t qidx,
 }
 
 static void
-octeontx_dev_rx_queue_release(void *rxq)
+octeontx_dev_rx_queue_release(struct rte_eth_dev *dev, uint16_t qid)
 {
-       rte_free(rxq);
+       rte_free(dev->data->rx_queues[qid]);
 }
 
 static const uint32_t *
@@ -1371,7 +1357,7 @@ octeontx_create(struct rte_vdev_device *dev, int port, uint8_t evdev,
        nic->ev_ports = 1;
        nic->print_flag = -1;
 
-       data->dev_link.link_status = ETH_LINK_DOWN;
+       data->dev_link.link_status = RTE_ETH_LINK_DOWN;
        data->dev_started = 0;
        data->promiscuous = 0;
        data->all_multicast = 0;
@@ -1463,10 +1449,9 @@ octeontx_remove(struct rte_vdev_device *dev)
        for (i = 0; i < OCTEONTX_VDEV_DEFAULT_MAX_NR_PORT; i++) {
                sprintf(octtx_name, "eth_octeontx_%d", i);
 
-               /* reserve an ethdev entry */
                eth_dev = rte_eth_dev_allocated(octtx_name);
                if (eth_dev == NULL)
-                       return -ENODEV;
+                       continue; /* port already released */
 
                if (rte_eal_process_type() != RTE_PROC_PRIMARY) {
                        rte_eth_dev_release_port(eth_dev);
@@ -1476,9 +1461,8 @@ octeontx_remove(struct rte_vdev_device *dev)
                nic = octeontx_pmd_priv(eth_dev);
                rte_event_dev_stop(nic->evdev);
                PMD_INIT_LOG(INFO, "Closing octeontx device %s", octtx_name);
-
+               octeontx_dev_close(eth_dev);
                rte_eth_dev_release_port(eth_dev);
-               rte_event_dev_close(nic->evdev);
        }
 
        if (rte_eal_process_type() != RTE_PROC_PRIMARY)