net/mlx5: fix dynamic inline hint handling
[dpdk.git] / drivers / net / mvneta / mvneta_ethdev.c
index a3dbe56..673a8e2 100644 (file)
@@ -4,6 +4,7 @@
  * All rights reserved.
  */
 
+#include <rte_string_fns.h>
 #include <rte_ethdev_driver.h>
 #include <rte_kvargs.h>
 #include <rte_bus_vdev.h>
 
 #define MVNETA_IFACE_NAME_ARG "iface"
 
-#define MVNETA_RX_OFFLOADS (DEV_RX_OFFLOAD_JUMBO_FRAME | \
-                         DEV_RX_OFFLOAD_CHECKSUM)
-
-/** Port Tx offloads capabilities */
-#define MVNETA_TX_OFFLOADS (DEV_TX_OFFLOAD_IPV4_CKSUM | \
-                         DEV_TX_OFFLOAD_UDP_CKSUM | \
-                         DEV_TX_OFFLOAD_TCP_CKSUM | \
-                         DEV_TX_OFFLOAD_MULTI_SEGS)
-
 #define MVNETA_PKT_SIZE_MAX (16382 - MV_MH_SIZE) /* 9700B */
 #define MVNETA_DEFAULT_MTU 1500
 
@@ -42,8 +34,6 @@
 /** Maximum length of a match string */
 #define MVNETA_MATCH_LEN 16
 
-int mvneta_logtype;
-
 static const char * const valid_args[] = {
        MVNETA_IFACE_NAME_ARG,
        NULL
@@ -56,6 +46,10 @@ struct mvneta_ifnames {
 
 static int mvneta_dev_num;
 
+static int mvneta_stats_reset(struct rte_eth_dev *dev);
+static int rte_pmd_mvneta_remove(struct rte_vdev_device *vdev);
+
+
 /**
  * Deinitialize packet processor.
  */
@@ -157,7 +151,7 @@ mvneta_dev_configure(struct rte_eth_dev *dev)
  * @param info
  *   Info structure output buffer.
  */
-static void
+static int
 mvneta_dev_infos_get(struct rte_eth_dev *dev __rte_unused,
                   struct rte_eth_dev_info *info)
 {
@@ -191,6 +185,8 @@ mvneta_dev_infos_get(struct rte_eth_dev *dev __rte_unused,
        info->default_txconf.offloads = 0;
 
        info->max_rx_pktlen = MVNETA_PKT_SIZE_MAX;
+
+       return 0;
 }
 
 /**
@@ -260,7 +256,7 @@ mvneta_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
                        mbuf_data_size, mtu, mru);
        }
 
-       if (mtu < ETHER_MIN_MTU || mru > MVNETA_PKT_SIZE_MAX) {
+       if (mtu < RTE_ETHER_MIN_MTU || mru > MVNETA_PKT_SIZE_MAX) {
                MVNETA_LOG(ERR, "Invalid MTU [%u] or MRU [%u]", mtu, mru);
                return -EINVAL;
        }
@@ -347,7 +343,7 @@ mvneta_dev_start(struct rte_eth_dev *dev)
        if (priv->ppio)
                return mvneta_dev_set_link_up(dev);
 
-       snprintf(match, sizeof(match), "%s", dev->data->name);
+       strlcpy(match, dev->data->name, sizeof(match));
        priv->ppio_params.match = match;
        priv->ppio_params.inqs_params.mtu = dev->data->mtu;
 
@@ -358,6 +354,8 @@ mvneta_dev_start(struct rte_eth_dev *dev)
        }
        priv->ppio_id = priv->ppio->port_id;
 
+       mvneta_stats_reset(dev);
+
        /*
         * In case there are some some stale uc/mc mac addresses flush them
         * here. It cannot be done during mvneta_dev_close() as port information
@@ -449,6 +447,14 @@ mvneta_dev_close(struct rte_eth_dev *dev)
                mvneta_tx_queue_release(dev->data->tx_queues[i]);
                dev->data->tx_queues[i] = NULL;
        }
+
+       mvneta_dev_num--;
+
+       if (mvneta_dev_num == 0) {
+               MVNETA_LOG(INFO, "Perform MUSDK deinit");
+               mvneta_neta_deinit();
+               rte_mvep_deinit(MVEP_MOD_T_NETA);
+       }
 }
 
 /**
@@ -526,25 +532,30 @@ mvneta_link_update(struct rte_eth_dev *dev, int wait_to_complete __rte_unused)
  *
  * @param dev
  *   Pointer to Ethernet device structure.
+ *
+ * @return
+ *   always 0
  */
-static void
+static int
 mvneta_promiscuous_enable(struct rte_eth_dev *dev)
 {
        struct mvneta_priv *priv = dev->data->dev_private;
        int ret, en;
 
        if (!priv->ppio)
-               return;
+               return 0;
 
        neta_ppio_get_promisc(priv->ppio, &en);
        if (en) {
                MVNETA_LOG(INFO, "Promiscuous already enabled");
-               return;
+               return 0;
        }
 
        ret = neta_ppio_set_promisc(priv->ppio, 1);
        if (ret)
                MVNETA_LOG(ERR, "Failed to enable promiscuous mode");
+
+       return 0;
 }
 
 /**
@@ -552,25 +563,97 @@ mvneta_promiscuous_enable(struct rte_eth_dev *dev)
  *
  * @param dev
  *   Pointer to Ethernet device structure.
+ *
+ * @return
+ *   always 0
  */
-static void
+static int
 mvneta_promiscuous_disable(struct rte_eth_dev *dev)
 {
        struct mvneta_priv *priv = dev->data->dev_private;
        int ret, en;
 
        if (!priv->ppio)
-               return;
+               return 0;
 
        neta_ppio_get_promisc(priv->ppio, &en);
        if (!en) {
                MVNETA_LOG(INFO, "Promiscuous already disabled");
-               return;
+               return 0;
        }
 
        ret = neta_ppio_set_promisc(priv->ppio, 0);
        if (ret)
                MVNETA_LOG(ERR, "Failed to disable promiscuous mode");
+
+       return 0;
+}
+
+/**
+ * DPDK callback to remove a MAC address.
+ *
+ * @param dev
+ *   Pointer to Ethernet device structure.
+ * @param index
+ *   MAC address index.
+ */
+static void
+mvneta_mac_addr_remove(struct rte_eth_dev *dev, uint32_t index)
+{
+       struct mvneta_priv *priv = dev->data->dev_private;
+       char buf[RTE_ETHER_ADDR_FMT_SIZE];
+       int ret;
+
+       if (!priv->ppio)
+               return;
+
+       ret = neta_ppio_remove_mac_addr(priv->ppio,
+                                      dev->data->mac_addrs[index].addr_bytes);
+       if (ret) {
+               rte_ether_format_addr(buf, sizeof(buf),
+                                 &dev->data->mac_addrs[index]);
+               MVNETA_LOG(ERR, "Failed to remove mac %s", buf);
+       }
+}
+
+/**
+ * DPDK callback to add a MAC address.
+ *
+ * @param dev
+ *   Pointer to Ethernet device structure.
+ * @param mac_addr
+ *   MAC address to register.
+ * @param index
+ *   MAC address index.
+ * @param vmdq
+ *   VMDq pool index to associate address with (unused).
+ *
+ * @return
+ *   0 on success, negative error value otherwise.
+ */
+static int
+mvneta_mac_addr_add(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr,
+                 uint32_t index, uint32_t vmdq __rte_unused)
+{
+       struct mvneta_priv *priv = dev->data->dev_private;
+       char buf[RTE_ETHER_ADDR_FMT_SIZE];
+       int ret;
+
+       if (index == 0)
+               /* For setting index 0, mrvl_mac_addr_set() should be used.*/
+               return -1;
+
+       if (!priv->ppio)
+               return 0;
+
+       ret = neta_ppio_add_mac_addr(priv->ppio, mac_addr->addr_bytes);
+       if (ret) {
+               rte_ether_format_addr(buf, sizeof(buf), mac_addr);
+               MVNETA_LOG(ERR, "Failed to add mac %s", buf);
+               return -1;
+       }
+
+       return 0;
 }
 
 /**
@@ -582,7 +665,7 @@ mvneta_promiscuous_disable(struct rte_eth_dev *dev)
  *   MAC address to register.
  */
 static int
-mvneta_mac_addr_set(struct rte_eth_dev *dev, struct ether_addr *mac_addr)
+mvneta_mac_addr_set(struct rte_eth_dev *dev, struct rte_ether_addr *mac_addr)
 {
        struct mvneta_priv *priv = dev->data->dev_private;
        int ret;
@@ -592,13 +675,86 @@ mvneta_mac_addr_set(struct rte_eth_dev *dev, struct ether_addr *mac_addr)
 
        ret = neta_ppio_set_mac_addr(priv->ppio, mac_addr->addr_bytes);
        if (ret) {
-               char buf[ETHER_ADDR_FMT_SIZE];
-               ether_format_addr(buf, sizeof(buf), mac_addr);
+               char buf[RTE_ETHER_ADDR_FMT_SIZE];
+               rte_ether_format_addr(buf, sizeof(buf), mac_addr);
                MVNETA_LOG(ERR, "Failed to set mac to %s", buf);
        }
        return 0;
 }
 
+/**
+ * DPDK callback to get device statistics.
+ *
+ * @param dev
+ *   Pointer to Ethernet device structure.
+ * @param stats
+ *   Stats structure output buffer.
+ *
+ * @return
+ *   0 on success, negative error value otherwise.
+ */
+static int
+mvneta_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
+{
+       struct mvneta_priv *priv = dev->data->dev_private;
+       struct neta_ppio_statistics ppio_stats;
+       unsigned int ret;
+
+       if (!priv->ppio)
+               return -EPERM;
+
+       ret = neta_ppio_get_statistics(priv->ppio, &ppio_stats);
+       if (unlikely(ret)) {
+               MVNETA_LOG(ERR, "Failed to update port statistics");
+               return ret;
+       }
+
+       stats->ipackets += ppio_stats.rx_packets +
+                       ppio_stats.rx_broadcast_packets +
+                       ppio_stats.rx_multicast_packets -
+                       priv->prev_stats.ipackets;
+       stats->opackets += ppio_stats.tx_packets +
+                       ppio_stats.tx_broadcast_packets +
+                       ppio_stats.tx_multicast_packets -
+                       priv->prev_stats.opackets;
+       stats->ibytes += ppio_stats.rx_bytes - priv->prev_stats.ibytes;
+       stats->obytes += ppio_stats.tx_bytes - priv->prev_stats.obytes;
+       stats->imissed += ppio_stats.rx_discard +
+                         ppio_stats.rx_overrun -
+                         priv->prev_stats.imissed;
+       stats->ierrors = ppio_stats.rx_packets_err -
+                       priv->prev_stats.ierrors;
+       stats->oerrors = ppio_stats.tx_errors - priv->prev_stats.oerrors;
+
+       return 0;
+}
+
+/**
+ * DPDK callback to clear device statistics.
+ *
+ * @param dev
+ *   Pointer to Ethernet device structure.
+ *
+ * @return
+ *   0 on success, negative error value otherwise.
+ */
+static int
+mvneta_stats_reset(struct rte_eth_dev *dev)
+{
+       struct mvneta_priv *priv = dev->data->dev_private;
+       unsigned int ret;
+
+       if (!priv->ppio)
+               return 0;
+
+       ret = mvneta_stats_get(dev, &priv->prev_stats);
+       if (unlikely(ret))
+               MVNETA_LOG(ERR, "Failed to reset port statistics");
+
+       return ret;
+}
+
+
 static const struct eth_dev_ops mvneta_ops = {
        .dev_configure = mvneta_dev_configure,
        .dev_start = mvneta_dev_start,
@@ -609,8 +765,12 @@ static const struct eth_dev_ops mvneta_ops = {
        .link_update = mvneta_link_update,
        .promiscuous_enable = mvneta_promiscuous_enable,
        .promiscuous_disable = mvneta_promiscuous_disable,
+       .mac_addr_remove = mvneta_mac_addr_remove,
+       .mac_addr_add = mvneta_mac_addr_add,
        .mac_addr_set = mvneta_mac_addr_set,
        .mtu_set = mvneta_mtu_set,
+       .stats_get = mvneta_stats_get,
+       .stats_reset = mvneta_stats_reset,
        .dev_infos_get = mvneta_dev_infos_get,
        .dev_supported_ptypes_get = mvneta_dev_supported_ptypes_get,
        .rxq_info_get = mvneta_rxq_info_get,
@@ -645,41 +805,40 @@ mvneta_eth_dev_create(struct rte_vdev_device *vdev, const char *name)
        priv = rte_zmalloc_socket(name, sizeof(*priv), 0, rte_socket_id());
        if (!priv) {
                ret = -ENOMEM;
-               goto out_free_dev;
+               goto out_free;
        }
+       eth_dev->data->dev_private = priv;
 
        eth_dev->data->mac_addrs =
                rte_zmalloc("mac_addrs",
-                           ETHER_ADDR_LEN * MVNETA_MAC_ADDRS_MAX, 0);
+                           RTE_ETHER_ADDR_LEN * MVNETA_MAC_ADDRS_MAX, 0);
        if (!eth_dev->data->mac_addrs) {
                MVNETA_LOG(ERR, "Failed to allocate space for eth addrs");
                ret = -ENOMEM;
-               goto out_free_priv;
+               goto out_free;
        }
 
        memset(&req, 0, sizeof(req));
        strcpy(req.ifr_name, name);
        ret = ioctl(fd, SIOCGIFHWADDR, &req);
        if (ret)
-               goto out_free_mac;
+               goto out_free;
 
        memcpy(eth_dev->data->mac_addrs[0].addr_bytes,
-              req.ifr_addr.sa_data, ETHER_ADDR_LEN);
+              req.ifr_addr.sa_data, RTE_ETHER_ADDR_LEN);
 
        eth_dev->data->kdrv = RTE_KDRV_NONE;
-       eth_dev->data->dev_private = priv;
        eth_dev->device = &vdev->device;
        eth_dev->rx_pkt_burst = mvneta_rx_pkt_burst;
        mvneta_set_tx_function(eth_dev);
        eth_dev->dev_ops = &mvneta_ops;
 
+       /* Flag to call rte_eth_dev_release_port() in rte_eth_dev_close(). */
+       eth_dev->data->dev_flags |= RTE_ETH_DEV_CLOSE_REMOVE;
+
        rte_eth_dev_probing_finish(eth_dev);
        return 0;
-out_free_mac:
-       rte_free(eth_dev->data->mac_addrs);
-out_free_priv:
-       rte_free(priv);
-out_free_dev:
+out_free:
        rte_eth_dev_release_port(eth_dev);
 
        return ret;
@@ -694,8 +853,6 @@ out_free_dev:
 static void
 mvneta_eth_dev_destroy(struct rte_eth_dev *eth_dev)
 {
-       rte_free(eth_dev->data->dev_private);
-       rte_free(eth_dev->data->mac_addrs);
        rte_eth_dev_release_port(eth_dev);
 }
 
@@ -777,20 +934,16 @@ init_devices:
                ret = mvneta_eth_dev_create(vdev, ifnames.names[i]);
                if (ret)
                        goto out_cleanup;
+
+               mvneta_dev_num++;
        }
-       mvneta_dev_num += ifnum;
 
        rte_kvargs_free(kvlist);
 
        return 0;
 out_cleanup:
-       for (; i > 0; i--)
-               mvneta_eth_dev_destroy_name(ifnames.names[i]);
+       rte_pmd_mvneta_remove(vdev);
 
-       if (mvneta_dev_num == 0) {
-               mvneta_neta_deinit();
-               rte_mvep_deinit(MVEP_MOD_T_NETA);
-       }
 out_free_kvlist:
        rte_kvargs_free(kvlist);
 
@@ -809,27 +962,12 @@ out_free_kvlist:
 static int
 rte_pmd_mvneta_remove(struct rte_vdev_device *vdev)
 {
-       int i;
-       const char *name;
+       uint16_t port_id;
 
-       name = rte_vdev_device_name(vdev);
-       if (!name)
-               return -EINVAL;
-
-       MVNETA_LOG(INFO, "Removing %s", name);
-
-       RTE_ETH_FOREACH_DEV(i) {
-               if (rte_eth_devices[i].device != &vdev->device)
+       RTE_ETH_FOREACH_DEV(port_id) {
+               if (rte_eth_devices[port_id].device != &vdev->device)
                        continue;
-
-               mvneta_eth_dev_destroy(&rte_eth_devices[i]);
-               mvneta_dev_num--;
-       }
-
-       if (mvneta_dev_num == 0) {
-               MVNETA_LOG(INFO, "Perform MUSDK deinit");
-               mvneta_neta_deinit();
-               rte_mvep_deinit(MVEP_MOD_T_NETA);
+               rte_eth_dev_close(port_id);
        }
 
        return 0;
@@ -842,10 +980,4 @@ static struct rte_vdev_driver pmd_mvneta_drv = {
 
 RTE_PMD_REGISTER_VDEV(net_mvneta, pmd_mvneta_drv);
 RTE_PMD_REGISTER_PARAM_STRING(net_mvneta, "iface=<ifc>");
-
-RTE_INIT(mvneta_init_log)
-{
-       mvneta_logtype = rte_log_register("pmd.net.mvneta");
-       if (mvneta_logtype >= 0)
-               rte_log_set_level(mvneta_logtype, RTE_LOG_NOTICE);
-}
+RTE_LOG_REGISTER(mvneta_logtype, pmd.net.mvneta, NOTICE);