net/hns3: add runtime config for mailbox limit time
[dpdk.git] / drivers / net / hns3 / hns3_ethdev_vf.c
index 71f3f95..8e5df05 100644 (file)
@@ -784,8 +784,6 @@ hns3vf_dev_configure(struct rte_eth_dev *dev)
        uint16_t nb_rx_q = dev->data->nb_rx_queues;
        uint16_t nb_tx_q = dev->data->nb_tx_queues;
        struct rte_eth_rss_conf rss_conf;
-       uint32_t max_rx_pkt_len;
-       uint16_t mtu;
        bool gro_en;
        int ret;
 
@@ -801,13 +799,11 @@ hns3vf_dev_configure(struct rte_eth_dev *dev)
         * work as usual. But these fake queues are imperceptible, and can not
         * be used by upper applications.
         */
-       if (!hns3_dev_indep_txrx_supported(hw)) {
-               ret = hns3_set_fake_rx_or_tx_queues(dev, nb_rx_q, nb_tx_q);
-               if (ret) {
-                       hns3_err(hw, "fail to set Rx/Tx fake queues, ret = %d.",
-                                ret);
-                       return ret;
-               }
+       ret = hns3_set_fake_rx_or_tx_queues(dev, nb_rx_q, nb_tx_q);
+       if (ret) {
+               hns3_err(hw, "fail to set Rx/Tx fake queues, ret = %d.", ret);
+               hw->cfg_max_queues = 0;
+               return ret;
        }
 
        hw->adapter_state = HNS3_NIC_CONFIGURING;
@@ -827,28 +823,9 @@ hns3vf_dev_configure(struct rte_eth_dev *dev)
                        goto cfg_err;
        }
 
-       /*
-        * If jumbo frames are enabled, MTU needs to be refreshed
-        * according to the maximum RX packet length.
-        */
-       if (conf->rxmode.offloads & DEV_RX_OFFLOAD_JUMBO_FRAME) {
-               max_rx_pkt_len = conf->rxmode.max_rx_pkt_len;
-               if (max_rx_pkt_len > HNS3_MAX_FRAME_LEN ||
-                   max_rx_pkt_len <= HNS3_DEFAULT_FRAME_LEN) {
-                       hns3_err(hw, "maximum Rx packet length must be greater "
-                                "than %u and less than %u when jumbo frame enabled.",
-                                (uint16_t)HNS3_DEFAULT_FRAME_LEN,
-                                (uint16_t)HNS3_MAX_FRAME_LEN);
-                       ret = -EINVAL;
-                       goto cfg_err;
-               }
-
-               mtu = (uint16_t)HNS3_PKTLEN_TO_MTU(max_rx_pkt_len);
-               ret = hns3vf_dev_mtu_set(dev, mtu);
-               if (ret)
-                       goto cfg_err;
-               dev->data->mtu = mtu;
-       }
+       ret = hns3vf_dev_mtu_set(dev, conf->rxmode.mtu);
+       if (ret != 0)
+               goto cfg_err;
 
        ret = hns3vf_dev_configure_vlan(dev);
        if (ret)
@@ -866,6 +843,7 @@ hns3vf_dev_configure(struct rte_eth_dev *dev)
        return 0;
 
 cfg_err:
+       hw->cfg_max_queues = 0;
        (void)hns3_set_fake_rx_or_tx_queues(dev, 0, 0);
        hw->adapter_state = HNS3_NIC_INITIALIZED;
 
@@ -930,13 +908,6 @@ hns3vf_dev_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)
                rte_spinlock_unlock(&hw->lock);
                return ret;
        }
-       if (mtu > RTE_ETHER_MTU)
-               dev->data->dev_conf.rxmode.offloads |=
-                                               DEV_RX_OFFLOAD_JUMBO_FRAME;
-       else
-               dev->data->dev_conf.rxmode.offloads &=
-                                               ~DEV_RX_OFFLOAD_JUMBO_FRAME;
-       dev->data->dev_conf.rxmode.max_rx_pkt_len = frame_size;
        rte_spinlock_unlock(&hw->lock);
 
        return 0;
@@ -973,7 +944,6 @@ hns3vf_dev_infos_get(struct rte_eth_dev *eth_dev, struct rte_eth_dev_info *info)
                                 DEV_RX_OFFLOAD_SCATTER |
                                 DEV_RX_OFFLOAD_VLAN_STRIP |
                                 DEV_RX_OFFLOAD_VLAN_FILTER |
-                                DEV_RX_OFFLOAD_JUMBO_FRAME |
                                 DEV_RX_OFFLOAD_RSS_HASH |
                                 DEV_RX_OFFLOAD_TCP_LRO);
        info->tx_offload_capa = (DEV_TX_OFFLOAD_OUTER_IPV4_CKSUM |
@@ -989,10 +959,10 @@ hns3vf_dev_infos_get(struct rte_eth_dev *eth_dev, struct rte_eth_dev_info *info)
                                 DEV_TX_OFFLOAD_MBUF_FAST_FREE |
                                 hns3_txvlan_cap_get(hw));
 
-       if (hns3_dev_outer_udp_cksum_supported(hw))
+       if (hns3_dev_get_support(hw, OUTER_UDP_CKSUM))
                info->tx_offload_capa |= DEV_TX_OFFLOAD_OUTER_UDP_CKSUM;
 
-       if (hns3_dev_indep_txrx_supported(hw))
+       if (hns3_dev_get_support(hw, INDEP_TXRX))
                info->dev_capa = RTE_ETH_DEV_CAPA_RUNTIME_RX_QUEUE_SETUP |
                                 RTE_ETH_DEV_CAPA_RUNTIME_TX_QUEUE_SETUP;
 
@@ -1117,6 +1087,8 @@ hns3vf_interrupt_handler(void *param)
 
        /* Read out interrupt causes */
        event_cause = hns3vf_check_event_cause(hns, &clearval);
+       /* Clear interrupt causes */
+       hns3vf_clear_event_cause(hw, clearval);
 
        switch (event_cause) {
        case HNS3VF_VECTOR0_EVENT_RST:
@@ -1129,9 +1101,6 @@ hns3vf_interrupt_handler(void *param)
                break;
        }
 
-       /* Clear interrupt causes */
-       hns3vf_clear_event_cause(hw, clearval);
-
        /* Enable interrupt */
        hns3vf_enable_irq0(hw);
 }
@@ -1409,28 +1378,48 @@ hns3vf_get_queue_depth(struct hns3_hw *hw)
        return 0;
 }
 
+static void
+hns3vf_update_caps(struct hns3_hw *hw, uint32_t caps)
+{
+       if (hns3_get_bit(caps, HNS3VF_CAPS_VLAN_FLT_MOD_B))
+               hns3_set_bit(hw->capability,
+                               HNS3_DEV_SUPPORT_VF_VLAN_FLT_MOD_B, 1);
+}
+
 static int
-hns3vf_get_tc_info(struct hns3_hw *hw)
+hns3vf_get_num_tc(struct hns3_hw *hw)
 {
-       uint8_t resp_msg;
-       int ret;
+       uint8_t num_tc = 0;
        uint32_t i;
 
-       ret = hns3_send_mbx_msg(hw, HNS3_MBX_GET_TCINFO, 0, NULL, 0,
-                               true, &resp_msg, sizeof(resp_msg));
-       if (ret) {
-               hns3_err(hw, "VF request to get TC info from PF failed %d",
-                        ret);
-               return ret;
+       for (i = 0; i < HNS3_MAX_TC_NUM; i++) {
+               if (hw->hw_tc_map & BIT(i))
+                       num_tc++;
        }
+       return num_tc;
+}
 
-       hw->hw_tc_map = resp_msg;
+static int
+hns3vf_get_basic_info(struct hns3_hw *hw)
+{
+       uint8_t resp_msg[HNS3_MBX_MAX_RESP_DATA_SIZE];
+       struct hns3_basic_info *basic_info;
+       int ret;
 
-       for (i = 0; i < HNS3_MAX_TC_NUM; i++) {
-               if (hw->hw_tc_map & BIT(i))
-                       hw->num_tc++;
+       ret = hns3_send_mbx_msg(hw, HNS3_MBX_GET_BASIC_INFO, 0, NULL, 0,
+                               true, resp_msg, sizeof(resp_msg));
+       if (ret) {
+               hns3_err(hw, "failed to get basic info from PF, ret = %d.",
+                               ret);
+               return ret;
        }
 
+       basic_info = (struct hns3_basic_info *)resp_msg;
+       hw->hw_tc_map = basic_info->hw_tc_map;
+       hw->num_tc = hns3vf_get_num_tc(hw);
+       hw->pf_vf_if_version = basic_info->pf_vf_if_version;
+       hns3vf_update_caps(hw, basic_info->caps);
+
        return 0;
 }
 
@@ -1469,6 +1458,11 @@ hns3vf_get_configuration(struct hns3_hw *hw)
 
        hns3vf_get_push_lsc_cap(hw);
 
+       /* Get basic info from PF */
+       ret = hns3vf_get_basic_info(hw);
+       if (ret)
+               return ret;
+
        /* Get queue configuration from PF */
        ret = hns3vf_get_queue_info(hw);
        if (ret)
@@ -1484,12 +1478,7 @@ hns3vf_get_configuration(struct hns3_hw *hw)
        if (ret)
                return ret;
 
-       ret = hns3vf_get_port_base_vlan_filter_state(hw);
-       if (ret)
-               return ret;
-
-       /* Get tc configuration from PF */
-       return hns3vf_get_tc_info(hw);
+       return hns3vf_get_port_base_vlan_filter_state(hw);
 }
 
 static int
@@ -1498,18 +1487,6 @@ hns3vf_set_tc_queue_mapping(struct hns3_adapter *hns, uint16_t nb_rx_q,
 {
        struct hns3_hw *hw = &hns->hw;
 
-       if (nb_rx_q < hw->num_tc) {
-               hns3_err(hw, "number of Rx queues(%u) is less than tcs(%u).",
-                        nb_rx_q, hw->num_tc);
-               return -EINVAL;
-       }
-
-       if (nb_tx_q < hw->num_tc) {
-               hns3_err(hw, "number of Tx queues(%u) is less than tcs(%u).",
-                        nb_tx_q, hw->num_tc);
-               return -EINVAL;
-       }
-
        return hns3_queue_to_tc_mapping(hw, nb_rx_q, nb_tx_q);
 }
 
@@ -1611,6 +1588,26 @@ hns3vf_vlan_filter_set(struct rte_eth_dev *dev, uint16_t vlan_id, int on)
        return ret;
 }
 
+static int
+hns3vf_en_vlan_filter(struct hns3_hw *hw, bool enable)
+{
+       uint8_t msg_data;
+       int ret;
+
+       if (!hns3_dev_get_support(hw, VF_VLAN_FLT_MOD))
+               return 0;
+
+       msg_data = enable ? 1 : 0;
+       ret = hns3_send_mbx_msg(hw, HNS3_MBX_SET_VLAN,
+                       HNS3_MBX_ENABLE_VLAN_FILTER, &msg_data,
+                       sizeof(msg_data), true, NULL, 0);
+       if (ret)
+               hns3_err(hw, "%s vlan filter failed, ret = %d.",
+                               enable ? "enable" : "disable", ret);
+
+       return ret;
+}
+
 static int
 hns3vf_en_hw_strip_rxvtag(struct hns3_hw *hw, bool enable)
 {
@@ -1621,7 +1618,8 @@ hns3vf_en_hw_strip_rxvtag(struct hns3_hw *hw, bool enable)
        ret = hns3_send_mbx_msg(hw, HNS3_MBX_SET_VLAN, HNS3_MBX_VLAN_RX_OFF_CFG,
                                &msg_data, sizeof(msg_data), false, NULL, 0);
        if (ret)
-               hns3_err(hw, "vf enable strip failed, ret =%d", ret);
+               hns3_err(hw, "vf %s strip failed, ret = %d.",
+                               enable ? "enable" : "disable", ret);
 
        return ret;
 }
@@ -1641,6 +1639,19 @@ hns3vf_vlan_offload_set(struct rte_eth_dev *dev, int mask)
        }
 
        tmp_mask = (unsigned int)mask;
+
+       if (tmp_mask & ETH_VLAN_FILTER_MASK) {
+               rte_spinlock_lock(&hw->lock);
+               /* Enable or disable VLAN filter */
+               if (dev_conf->rxmode.offloads & DEV_RX_OFFLOAD_VLAN_FILTER)
+                       ret = hns3vf_en_vlan_filter(hw, true);
+               else
+                       ret = hns3vf_en_vlan_filter(hw, false);
+               rte_spinlock_unlock(&hw->lock);
+               if (ret)
+                       return ret;
+       }
+
        /* Vlan stripping setting */
        if (tmp_mask & ETH_VLAN_STRIP_MASK) {
                rte_spinlock_lock(&hw->lock);
@@ -1738,9 +1749,10 @@ hns3vf_dev_configure_vlan(struct rte_eth_dev *dev)
        }
 
        /* Apply vlan offload setting */
-       ret = hns3vf_vlan_offload_set(dev, ETH_VLAN_STRIP_MASK);
+       ret = hns3vf_vlan_offload_set(dev, ETH_VLAN_STRIP_MASK |
+                                       ETH_VLAN_FILTER_MASK);
        if (ret)
-               hns3_err(hw, "dev config vlan offload failed, ret =%d", ret);
+               hns3_err(hw, "dev config vlan offload failed, ret = %d.", ret);
 
        return ret;
 }
@@ -1891,12 +1903,6 @@ hns3vf_init_hardware(struct hns3_adapter *hns)
                goto err_init_hardware;
        }
 
-       ret = hns3vf_set_alive(hw, true);
-       if (ret) {
-               PMD_INIT_LOG(ERR, "Failed to VF send alive to PF: %d", ret);
-               goto err_init_hardware;
-       }
-
        return 0;
 
 err_init_hardware:
@@ -1939,6 +1945,8 @@ hns3vf_init_vf(struct rte_eth_dev *eth_dev)
                goto err_cmd_init;
        }
 
+       hns3_tx_push_init(eth_dev);
+
        /* Get VF resource */
        ret = hns3_query_vf_resource(hw);
        if (ret)
@@ -1995,6 +2003,12 @@ hns3vf_init_vf(struct rte_eth_dev *eth_dev)
 
        hns3_rss_set_default_args(hw);
 
+       ret = hns3vf_set_alive(hw, true);
+       if (ret) {
+               PMD_INIT_LOG(ERR, "Failed to VF send alive to PF: %d", ret);
+               goto err_set_tc_queue;
+       }
+
        return 0;
 
 err_set_tc_queue:
@@ -2028,6 +2042,7 @@ hns3vf_uninit_vf(struct rte_eth_dev *eth_dev)
        (void)hns3_config_gro(hw, false);
        (void)hns3vf_set_alive(hw, false);
        (void)hns3vf_set_promisc_mode(hw, false, false, false);
+       hns3_flow_uninit(eth_dev);
        hns3_tqp_stats_uninit(hw);
        hns3vf_disable_irq0(hw);
        rte_intr_disable(&pci_dev->intr_handle);
@@ -2119,7 +2134,7 @@ hns3vf_dev_stop(struct rte_eth_dev *dev)
        /* Disable datapath on secondary process. */
        hns3_mp_req_stop_rxtx(dev);
        /* Prevent crashes when queues are still in use. */
-       rte_delay_ms(hw->tqps_num);
+       rte_delay_ms(hw->cfg_max_queues);
 
        rte_spinlock_lock(&hw->lock);
        if (__atomic_load_n(&hw->reset.resetting, __ATOMIC_RELAXED) == 0) {
@@ -2143,11 +2158,8 @@ hns3vf_dev_close(struct rte_eth_dev *eth_dev)
        struct hns3_hw *hw = &hns->hw;
        int ret = 0;
 
-       if (rte_eal_process_type() != RTE_PROC_PRIMARY) {
-               rte_free(eth_dev->process_private);
-               eth_dev->process_private = NULL;
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
                return 0;
-       }
 
        if (hw->adapter_state == HNS3_NIC_STARTED)
                ret = hns3vf_dev_stop(eth_dev);
@@ -2161,8 +2173,6 @@ hns3vf_dev_close(struct rte_eth_dev *eth_dev)
        hns3vf_uninit_vf(eth_dev);
        hns3_free_all_queues(eth_dev);
        rte_free(hw->reset.wait_data);
-       rte_free(eth_dev->process_private);
-       eth_dev->process_private = NULL;
        hns3_mp_uninit_primary();
        hns3_warn(hw, "Close port %u finished", hw->data->port_id);
 
@@ -2217,16 +2227,18 @@ hns3vf_dev_link_update(struct rte_eth_dev *eth_dev,
        case ETH_SPEED_NUM_50G:
        case ETH_SPEED_NUM_100G:
        case ETH_SPEED_NUM_200G:
-               new_link.link_speed = mac->link_speed;
+               if (mac->link_status)
+                       new_link.link_speed = mac->link_speed;
                break;
        default:
                if (mac->link_status)
                        new_link.link_speed = ETH_SPEED_NUM_UNKNOWN;
-               else
-                       new_link.link_speed = ETH_SPEED_NUM_NONE;
                break;
        }
 
+       if (!mac->link_status)
+               new_link.link_speed = ETH_SPEED_NUM_NONE;
+
        new_link.link_duplex = mac->link_duplex;
        new_link.link_status = mac->link_status ? ETH_LINK_UP : ETH_LINK_DOWN;
        new_link.link_autoneg =
@@ -2317,7 +2329,7 @@ hns3vf_map_rx_interrupt(struct rte_eth_dev *dev)
        return 0;
 
 vf_bind_vector_error:
-       free(intr_handle->intr_vec);
+       rte_free(intr_handle->intr_vec);
        intr_handle->intr_vec = NULL;
 vf_alloc_intr_vec_error:
        rte_intr_efd_disable(intr_handle);
@@ -2568,7 +2580,7 @@ hns3vf_stop_service(struct hns3_adapter *hns)
        rte_wmb();
        /* Disable datapath on secondary process. */
        hns3_mp_req_stop_rxtx(eth_dev);
-       rte_delay_ms(hw->tqps_num);
+       rte_delay_ms(hw->cfg_max_queues);
 
        rte_spinlock_lock(&hw->lock);
        if (hw->adapter_state == HNS3_NIC_STARTED ||
@@ -2706,6 +2718,13 @@ hns3vf_restore_conf(struct hns3_adapter *hns)
                hns3_info(hw, "hns3vf dev restart successful!");
        } else if (hw->adapter_state == HNS3_NIC_STOPPING)
                hw->adapter_state = HNS3_NIC_CONFIGURED;
+
+       ret = hns3vf_set_alive(hw, true);
+       if (ret) {
+               hns3_err(hw, "failed to VF send alive to PF: %d", ret);
+               goto err_vlan_table;
+       }
+
        return 0;
 
 err_vlan_table:
@@ -2910,15 +2929,6 @@ hns3vf_dev_init(struct rte_eth_dev *eth_dev)
 
        PMD_INIT_FUNC_TRACE();
 
-       eth_dev->process_private = (struct hns3_process_private *)
-           rte_zmalloc_socket("hns3_filter_list",
-                              sizeof(struct hns3_process_private),
-                              RTE_CACHE_LINE_SIZE, eth_dev->device->numa_node);
-       if (eth_dev->process_private == NULL) {
-               PMD_INIT_LOG(ERR, "Failed to alloc memory for process private");
-               return -ENOMEM;
-       }
-
        hns3_flow_init(eth_dev);
 
        hns3_set_rxtx_function(eth_dev);
@@ -2931,8 +2941,8 @@ hns3vf_dev_init(struct rte_eth_dev *eth_dev)
                                          "process, ret = %d", ret);
                        goto err_mp_init_secondary;
                }
-
                hw->secondary_cnt++;
+               hns3_tx_push_init(eth_dev);
                return 0;
        }
 
@@ -3020,8 +3030,6 @@ err_mp_init_secondary:
        eth_dev->tx_pkt_burst = NULL;
        eth_dev->tx_pkt_prepare = NULL;
        eth_dev->tx_descriptor_status = NULL;
-       rte_free(eth_dev->process_private);
-       eth_dev->process_private = NULL;
 
        return ret;
 }
@@ -3034,11 +3042,8 @@ hns3vf_dev_uninit(struct rte_eth_dev *eth_dev)
 
        PMD_INIT_FUNC_TRACE();
 
-       if (rte_eal_process_type() != RTE_PROC_PRIMARY) {
-               rte_free(eth_dev->process_private);
-               eth_dev->process_private = NULL;
+       if (rte_eal_process_type() != RTE_PROC_PRIMARY)
                return 0;
-       }
 
        if (hw->adapter_state < HNS3_NIC_CLOSING)
                hns3vf_dev_close(eth_dev);
@@ -3081,4 +3086,5 @@ RTE_PMD_REGISTER_KMOD_DEP(net_hns3_vf, "* igb_uio | vfio-pci");
 RTE_PMD_REGISTER_PARAM_STRING(net_hns3_vf,
                HNS3_DEVARG_RX_FUNC_HINT "=vec|sve|simple|common "
                HNS3_DEVARG_TX_FUNC_HINT "=vec|sve|simple|common "
-               HNS3_DEVARG_DEV_CAPS_MASK "=<1-65535> ");
+               HNS3_DEVARG_DEV_CAPS_MASK "=<1-65535> "
+               HNS3_DEVARG_MBX_TIME_LIMIT_MS "=<uint16_t> ");