ethdev: add return code to stats reset function
[dpdk.git] / drivers / net / liquidio / lio_ethdev.c
index df91659..a3c8e67 100644 (file)
  */
 
 #include <rte_ethdev.h>
+#include <rte_ethdev_pci.h>
 #include <rte_cycles.h>
 #include <rte_malloc.h>
 #include <rte_alarm.h>
+#include <rte_ether.h>
 
 #include "lio_logs.h"
 #include "lio_23xx_vf.h"
@@ -393,6 +395,25 @@ lio_dev_info_get(struct rte_eth_dev *eth_dev,
                 struct rte_eth_dev_info *devinfo)
 {
        struct lio_device *lio_dev = LIO_DEV(eth_dev);
+       struct rte_pci_device *pci_dev = RTE_ETH_DEV_TO_PCI(eth_dev);
+
+       devinfo->pci_dev = pci_dev;
+
+       switch (pci_dev->id.subsystem_device_id) {
+       /* CN23xx 10G cards */
+       case PCI_SUBSYS_DEV_ID_CN2350_210:
+       case PCI_SUBSYS_DEV_ID_CN2360_210:
+               devinfo->speed_capa = ETH_LINK_SPEED_10G;
+               break;
+       /* CN23xx 25G cards */
+       case PCI_SUBSYS_DEV_ID_CN2350_225:
+       case PCI_SUBSYS_DEV_ID_CN2360_225:
+               devinfo->speed_capa = ETH_LINK_SPEED_25G;
+               break;
+       default:
+               lio_dev_err(lio_dev,
+                           "Unknown CN23XX subsystem device id. Not setting speed capability.\n");
+       }
 
        devinfo->max_rx_queues = lio_dev->max_rx_queues;
        devinfo->max_tx_queues = lio_dev->max_tx_queues;
@@ -425,29 +446,64 @@ lio_dev_info_get(struct rte_eth_dev *eth_dev,
 }
 
 static int
-lio_dev_validate_vf_mtu(struct rte_eth_dev *eth_dev, uint16_t new_mtu)
+lio_dev_mtu_set(struct rte_eth_dev *eth_dev, uint16_t mtu)
 {
        struct lio_device *lio_dev = LIO_DEV(eth_dev);
+       uint16_t pf_mtu = lio_dev->linfo.link.s.mtu;
+       uint32_t frame_len = mtu + ETHER_HDR_LEN + ETHER_CRC_LEN;
+       struct lio_dev_ctrl_cmd ctrl_cmd;
+       struct lio_ctrl_pkt ctrl_pkt;
 
        PMD_INIT_FUNC_TRACE();
 
        if (!lio_dev->intf_open) {
-               lio_dev_err(lio_dev, "Port %d down, can't check MTU\n",
+               lio_dev_err(lio_dev, "Port %d down, can't set MTU\n",
                            lio_dev->port_id);
                return -EINVAL;
        }
 
-       /* Limit the MTU to make sure the ethernet packets are between
-        * ETHER_MIN_MTU bytes and PF's MTU
+       /* check if VF MTU is within allowed range.
+        * New value should not exceed PF MTU.
         */
-       if ((new_mtu < ETHER_MIN_MTU) ||
-                       (new_mtu > lio_dev->linfo.link.s.mtu)) {
-               lio_dev_err(lio_dev, "Invalid MTU: %d\n", new_mtu);
-               lio_dev_err(lio_dev, "Valid range %d and %d\n",
-                           ETHER_MIN_MTU, lio_dev->linfo.link.s.mtu);
+       if ((mtu < ETHER_MIN_MTU) || (mtu > pf_mtu)) {
+               lio_dev_err(lio_dev, "VF MTU should be >= %d and <= %d\n",
+                           ETHER_MIN_MTU, pf_mtu);
                return -EINVAL;
        }
 
+       /* flush added to prevent cmd failure
+        * incase the queue is full
+        */
+       lio_flush_iq(lio_dev, lio_dev->instr_queue[0]);
+
+       memset(&ctrl_pkt, 0, sizeof(struct lio_ctrl_pkt));
+       memset(&ctrl_cmd, 0, sizeof(struct lio_dev_ctrl_cmd));
+
+       ctrl_cmd.eth_dev = eth_dev;
+       ctrl_cmd.cond = 0;
+
+       ctrl_pkt.ncmd.s.cmd = LIO_CMD_CHANGE_MTU;
+       ctrl_pkt.ncmd.s.param1 = mtu;
+       ctrl_pkt.ctrl_cmd = &ctrl_cmd;
+
+       if (lio_send_ctrl_pkt(lio_dev, &ctrl_pkt)) {
+               lio_dev_err(lio_dev, "Failed to send command to change MTU\n");
+               return -1;
+       }
+
+       if (lio_wait_for_ctrl_cmd(lio_dev, &ctrl_cmd)) {
+               lio_dev_err(lio_dev, "Command to change MTU timed out\n");
+               return -1;
+       }
+
+       if (frame_len > ETHER_MAX_LEN)
+               eth_dev->data->dev_conf.rxmode.jumbo_frame = 1;
+       else
+               eth_dev->data->dev_conf.rxmode.jumbo_frame = 0;
+
+       eth_dev->data->dev_conf.rxmode.max_rx_pkt_len = frame_len;
+       eth_dev->data->mtu = mtu;
+
        return 0;
 }
 
@@ -918,6 +974,7 @@ lio_dev_link_update(struct rte_eth_dev *eth_dev,
        link.link_status = ETH_LINK_DOWN;
        link.link_speed = ETH_SPEED_NUM_NONE;
        link.link_duplex = ETH_LINK_HALF_DUPLEX;
+       link.link_autoneg = ETH_LINK_AUTONEG;
        memset(&old, 0, sizeof(old));
 
        /* Return what we found */
@@ -936,6 +993,9 @@ lio_dev_link_update(struct rte_eth_dev *eth_dev,
        case LIO_LINK_SPEED_10000:
                link.link_speed = ETH_SPEED_NUM_10G;
                break;
+       case LIO_LINK_SPEED_25000:
+               link.link_speed = ETH_SPEED_NUM_25G;
+               break;
        default:
                link.link_speed = ETH_SPEED_NUM_NONE;
                link.link_duplex = ETH_LINK_HALF_DUPLEX;
@@ -1155,14 +1215,13 @@ void
 lio_dev_rx_queue_release(void *rxq)
 {
        struct lio_droq *droq = rxq;
-       struct lio_device *lio_dev = droq->lio_dev;
        int oq_no;
 
-       /* Run time queue deletion not supported */
-       if (lio_dev->port_configured)
-               return;
+       if (droq) {
+               /* Run time queue deletion not supported */
+               if (droq->lio_dev->port_configured)
+                       return;
 
-       if (droq != NULL) {
                oq_no = droq->q_no;
                lio_delete_droq_queue(droq->lio_dev, oq_no);
        }
@@ -1250,14 +1309,14 @@ void
 lio_dev_tx_queue_release(void *txq)
 {
        struct lio_instr_queue *tq = txq;
-       struct lio_device *lio_dev = tq->lio_dev;
        uint32_t fw_mapped_iq_no;
 
-       /* Run time queue deletion not supported */
-       if (lio_dev->port_configured)
-               return;
 
-       if (tq != NULL) {
+       if (tq) {
+               /* Run time queue deletion not supported */
+               if (tq->lio_dev->port_configured)
+                       return;
+
                /* Free sg_list */
                lio_delete_sglist(tq);
 
@@ -1310,6 +1369,11 @@ lio_dev_get_link_status(struct rte_eth_dev *eth_dev)
        lio_swap_8B_data((uint64_t *)ls, sizeof(union octeon_link_status) >> 3);
 
        if (lio_dev->linfo.link.link_status64 != ls->link_status64) {
+               if (ls->s.mtu < eth_dev->data->mtu) {
+                       lio_dev_info(lio_dev, "Lowered VF MTU to %d as PF MTU dropped\n",
+                                    ls->s.mtu);
+                       eth_dev->data->mtu = ls->s.mtu;
+               }
                lio_dev->linfo.link.link_status64 = ls->link_status64;
                lio_dev_link_update(eth_dev, 0);
        }
@@ -1345,7 +1409,8 @@ lio_sync_link_state_check(void *eth_dev)
 static int
 lio_dev_start(struct rte_eth_dev *eth_dev)
 {
-       uint16_t mtu = eth_dev->data->dev_conf.rxmode.max_rx_pkt_len;
+       uint16_t mtu;
+       uint32_t frame_len = eth_dev->data->dev_conf.rxmode.max_rx_pkt_len;
        struct lio_device *lio_dev = LIO_DEV(eth_dev);
        uint16_t timeout = LIO_MAX_CMD_TIMEOUT;
        int ret = 0;
@@ -1380,18 +1445,22 @@ lio_dev_start(struct rte_eth_dev *eth_dev)
 
        if (lio_dev->linfo.link.link_status64 == 0) {
                ret = -1;
-               goto dev_mtu_check_error;
+               goto dev_mtu_set_error;
        }
 
-       if (lio_dev->linfo.link.s.mtu != mtu) {
-               ret = lio_dev_validate_vf_mtu(eth_dev, mtu);
+       mtu = (uint16_t)(frame_len - ETHER_HDR_LEN - ETHER_CRC_LEN);
+       if (mtu < ETHER_MIN_MTU)
+               mtu = ETHER_MIN_MTU;
+
+       if (eth_dev->data->mtu != mtu) {
+               ret = lio_dev_mtu_set(eth_dev, mtu);
                if (ret)
-                       goto dev_mtu_check_error;
+                       goto dev_mtu_set_error;
        }
 
        return 0;
 
-dev_mtu_check_error:
+dev_mtu_set_error:
        rte_eal_alarm_cancel(lio_sync_link_state_check, eth_dev);
 
 dev_lsc_handle_error:
@@ -1518,9 +1587,6 @@ lio_dev_close(struct rte_eth_dev *eth_dev)
                rte_write32(pkt_count, droq->pkts_sent_reg);
        }
 
-       /* Do FLR for the VF */
-       cn23xx_vf_ask_pf_to_do_flr(lio_dev);
-
        /* lio_free_mbox */
        lio_dev->fn_list.free_mbox(lio_dev);
 
@@ -1803,6 +1869,7 @@ static const struct eth_dev_ops liovf_eth_dev_ops = {
        .rss_hash_update        = lio_dev_rss_hash_update,
        .udp_tunnel_port_add    = lio_dev_udp_tunnel_add,
        .udp_tunnel_port_del    = lio_dev_udp_tunnel_del,
+       .mtu_set                = lio_dev_mtu_set,
 };
 
 static void
@@ -1888,11 +1955,6 @@ lio_first_time_init(struct lio_device *lio_dev,
        if (cn23xx_pfvf_handshake(lio_dev))
                goto error;
 
-       /* Initial reset */
-       cn23xx_vf_ask_pf_to_do_flr(lio_dev);
-       /* Wait for FLR for 100ms per SRIOV specification */
-       rte_delay_ms(100);
-
        if (cn23xx_vf_set_io_queues_off(lio_dev)) {
                lio_dev_err(lio_dev, "Setting io queues off failed\n");
                goto error;
@@ -1955,7 +2017,7 @@ lio_eth_dev_uninit(struct rte_eth_dev *eth_dev)
 static int
 lio_eth_dev_init(struct rte_eth_dev *eth_dev)
 {
-       struct rte_pci_device *pdev = RTE_DEV_TO_PCI(eth_dev->device);
+       struct rte_pci_device *pdev = RTE_ETH_DEV_TO_PCI(eth_dev);
        struct lio_device *lio_dev = LIO_DEV(eth_dev);
 
        PMD_INIT_FUNC_TRACE();
@@ -2011,24 +2073,45 @@ lio_eth_dev_init(struct rte_eth_dev *eth_dev)
        return 0;
 }
 
+static int
+lio_eth_dev_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
+                     struct rte_pci_device *pci_dev)
+{
+       struct rte_eth_dev *eth_dev;
+       int ret;
+
+       eth_dev = rte_eth_dev_pci_allocate(pci_dev,
+                                          sizeof(struct lio_device));
+       if (eth_dev == NULL)
+               return -ENOMEM;
+
+       ret = lio_eth_dev_init(eth_dev);
+       if (ret)
+               rte_eth_dev_pci_release(eth_dev);
+
+       return ret;
+}
+
+static int
+lio_eth_dev_pci_remove(struct rte_pci_device *pci_dev)
+{
+       return rte_eth_dev_pci_generic_remove(pci_dev,
+                                             lio_eth_dev_uninit);
+}
+
 /* Set of PCI devices this driver supports */
 static const struct rte_pci_id pci_id_liovf_map[] = {
        { RTE_PCI_DEVICE(PCI_VENDOR_ID_CAVIUM, LIO_CN23XX_VF_VID) },
        { .vendor_id = 0, /* sentinel */ }
 };
 
-static struct eth_driver rte_liovf_pmd = {
-       .pci_drv = {
-               .id_table       = pci_id_liovf_map,
-               .drv_flags      = RTE_PCI_DRV_NEED_MAPPING,
-               .probe          = rte_eth_dev_pci_probe,
-               .remove         = rte_eth_dev_pci_remove,
-       },
-       .eth_dev_init           = lio_eth_dev_init,
-       .eth_dev_uninit         = lio_eth_dev_uninit,
-       .dev_private_size       = sizeof(struct lio_device),
+static struct rte_pci_driver rte_liovf_pmd = {
+       .id_table       = pci_id_liovf_map,
+       .drv_flags      = RTE_PCI_DRV_NEED_MAPPING,
+       .probe          = lio_eth_dev_pci_probe,
+       .remove         = lio_eth_dev_pci_remove,
 };
 
-RTE_PMD_REGISTER_PCI(net_liovf, rte_liovf_pmd.pci_drv);
+RTE_PMD_REGISTER_PCI(net_liovf, rte_liovf_pmd);
 RTE_PMD_REGISTER_PCI_TABLE(net_liovf, pci_id_liovf_map);
-RTE_PMD_REGISTER_KMOD_DEP(net_liovf, "* igb_uio | vfio");
+RTE_PMD_REGISTER_KMOD_DEP(net_liovf, "* igb_uio | vfio-pci");