]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/blobdiff - drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
net: hns3: refactor the hclge_get/set_rss function
[mirror_ubuntu-bionic-kernel.git] / drivers / net / ethernet / hisilicon / hns3 / hns3pf / hclge_main.c
index 113b859b56e42415e4b06a46a2232b1a7534c083..131234dfb05ff565dd545d024a17589fe9c66e54 100644 (file)
@@ -17,6 +17,7 @@
 #include <linux/netdevice.h>
 #include <linux/pci.h>
 #include <linux/platform_device.h>
+#include <linux/if_vlan.h>
 #include <net/rtnetlink.h>
 #include "hclge_cmd.h"
 #include "hclge_dcb.h"
 static int hclge_set_mta_filter_mode(struct hclge_dev *hdev,
                                     enum hclge_mta_dmac_sel_type mta_mac_sel,
                                     bool enable);
+static int hclge_set_mtu(struct hnae3_handle *handle, int new_mtu);
 static int hclge_init_vlan_config(struct hclge_dev *hdev);
 static int hclge_reset_ae_dev(struct hnae3_ae_dev *ae_dev);
+static int hclge_update_led_status(struct hclge_dev *hdev);
 
 static struct hnae3_ae_algo ae_algo;
 
@@ -52,6 +55,8 @@ static const struct pci_device_id ae_algo_pci_tbl[] = {
        {0, }
 };
 
+MODULE_DEVICE_TABLE(pci, ae_algo_pci_tbl);
+
 static const char hns3_nic_test_strs[][ETH_GSTRING_LEN] = {
        "Mac    Loopback test",
        "Serdes Loopback test",
@@ -279,8 +284,8 @@ static const struct hclge_comm_stats_str g_mac_stats_string[] = {
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_broad_pkt_num)},
        {"mac_tx_undersize_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_undersize_pkt_num)},
-       {"mac_tx_overrsize_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_overrsize_pkt_num)},
+       {"mac_tx_oversize_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_oversize_pkt_num)},
        {"mac_tx_64_oct_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_64_oct_pkt_num)},
        {"mac_tx_65_127_oct_pkt_num",
@@ -293,8 +298,24 @@ static const struct hclge_comm_stats_str g_mac_stats_string[] = {
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_512_1023_oct_pkt_num)},
        {"mac_tx_1024_1518_oct_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1024_1518_oct_pkt_num)},
-       {"mac_tx_1519_max_oct_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_max_oct_pkt_num)},
+       {"mac_tx_1519_2047_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_2047_oct_pkt_num)},
+       {"mac_tx_2048_4095_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_2048_4095_oct_pkt_num)},
+       {"mac_tx_4096_8191_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_4096_8191_oct_pkt_num)},
+       {"mac_tx_8192_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_8192_12287_oct_pkt_num)},
+       {"mac_tx_8192_9216_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_8192_9216_oct_pkt_num)},
+       {"mac_tx_9217_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_9217_12287_oct_pkt_num)},
+       {"mac_tx_12288_16383_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_12288_16383_oct_pkt_num)},
+       {"mac_tx_1519_max_good_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_max_good_oct_pkt_num)},
+       {"mac_tx_1519_max_bad_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_1519_max_bad_oct_pkt_num)},
        {"mac_rx_total_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_total_pkt_num)},
        {"mac_rx_total_oct_num",
@@ -315,8 +336,8 @@ static const struct hclge_comm_stats_str g_mac_stats_string[] = {
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_broad_pkt_num)},
        {"mac_rx_undersize_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_undersize_pkt_num)},
-       {"mac_rx_overrsize_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_overrsize_pkt_num)},
+       {"mac_rx_oversize_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_oversize_pkt_num)},
        {"mac_rx_64_oct_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_64_oct_pkt_num)},
        {"mac_rx_65_127_oct_pkt_num",
@@ -329,33 +350,59 @@ static const struct hclge_comm_stats_str g_mac_stats_string[] = {
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_512_1023_oct_pkt_num)},
        {"mac_rx_1024_1518_oct_pkt_num",
                HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1024_1518_oct_pkt_num)},
-       {"mac_rx_1519_max_oct_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_max_oct_pkt_num)},
-
-       {"mac_trans_fragment_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_trans_fragment_pkt_num)},
-       {"mac_trans_undermin_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_trans_undermin_pkt_num)},
-       {"mac_trans_jabber_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_trans_jabber_pkt_num)},
-       {"mac_trans_err_all_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_trans_err_all_pkt_num)},
-       {"mac_trans_from_app_good_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_trans_from_app_good_pkt_num)},
-       {"mac_trans_from_app_bad_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_trans_from_app_bad_pkt_num)},
-       {"mac_rcv_fragment_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rcv_fragment_pkt_num)},
-       {"mac_rcv_undermin_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rcv_undermin_pkt_num)},
-       {"mac_rcv_jabber_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rcv_jabber_pkt_num)},
-       {"mac_rcv_fcs_err_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rcv_fcs_err_pkt_num)},
-       {"mac_rcv_send_app_good_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rcv_send_app_good_pkt_num)},
-       {"mac_rcv_send_app_bad_pkt_num",
-               HCLGE_MAC_STATS_FIELD_OFF(mac_rcv_send_app_bad_pkt_num)}
+       {"mac_rx_1519_2047_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_2047_oct_pkt_num)},
+       {"mac_rx_2048_4095_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_2048_4095_oct_pkt_num)},
+       {"mac_rx_4096_8191_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_4096_8191_oct_pkt_num)},
+       {"mac_rx_8192_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_8192_12287_oct_pkt_num)},
+       {"mac_rx_8192_9216_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_8192_9216_oct_pkt_num)},
+       {"mac_rx_9217_12287_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_9217_12287_oct_pkt_num)},
+       {"mac_rx_12288_16383_oct_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_12288_16383_oct_pkt_num)},
+       {"mac_rx_1519_max_good_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_max_good_oct_pkt_num)},
+       {"mac_rx_1519_max_bad_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_1519_max_bad_oct_pkt_num)},
+
+       {"mac_tx_fragment_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_fragment_pkt_num)},
+       {"mac_tx_undermin_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_undermin_pkt_num)},
+       {"mac_tx_jabber_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_jabber_pkt_num)},
+       {"mac_tx_err_all_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_err_all_pkt_num)},
+       {"mac_tx_from_app_good_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_from_app_good_pkt_num)},
+       {"mac_tx_from_app_bad_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_tx_from_app_bad_pkt_num)},
+       {"mac_rx_fragment_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_fragment_pkt_num)},
+       {"mac_rx_undermin_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_undermin_pkt_num)},
+       {"mac_rx_jabber_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_jabber_pkt_num)},
+       {"mac_rx_fcs_err_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_fcs_err_pkt_num)},
+       {"mac_rx_send_app_good_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_send_app_good_pkt_num)},
+       {"mac_rx_send_app_bad_pkt_num",
+               HCLGE_MAC_STATS_FIELD_OFF(mac_rx_send_app_bad_pkt_num)}
+};
+
+static const struct hclge_mac_mgr_tbl_entry_cmd hclge_mgr_table[] = {
+       {
+               .flags = HCLGE_MAC_MGR_MASK_VLAN_B,
+               .ethter_type = cpu_to_le16(HCLGE_MAC_ETHERTYPE_LLDP),
+               .mac_addr_hi32 = cpu_to_le32(htonl(0x0180C200)),
+               .mac_addr_lo16 = cpu_to_le16(htons(0x000E)),
+               .i_port_bitmap = 0x1,
+       },
 };
 
 static int hclge_64_bit_update_stats(struct hclge_dev *hdev)
@@ -461,9 +508,41 @@ static int hclge_32_bit_update_stats(struct hclge_dev *hdev)
        return 0;
 }
 
+static int hclge_mac_get_traffic_stats(struct hclge_dev *hdev)
+{
+       struct hclge_mac_stats *mac_stats = &hdev->hw_stats.mac_stats;
+       struct hclge_desc desc;
+       __le64 *desc_data;
+       int ret;
+
+       /* for fiber port, need to query the total rx/tx packets statstics,
+        * used for data transferring checking.
+        */
+       if (hdev->hw.mac.media_type != HNAE3_MEDIA_TYPE_FIBER)
+               return 0;
+
+       if (test_bit(HCLGE_STATE_STATISTICS_UPDATING, &hdev->state))
+               return 0;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_STATS_MAC_TRAFFIC, true);
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get MAC total pkt stats fail, ret = %d\n", ret);
+
+               return ret;
+       }
+
+       desc_data = (__le64 *)(&desc.data[0]);
+       mac_stats->mac_tx_total_pkt_num += le64_to_cpu(*desc_data++);
+       mac_stats->mac_rx_total_pkt_num += le64_to_cpu(*desc_data);
+
+       return 0;
+}
+
 static int hclge_mac_update_stats(struct hclge_dev *hdev)
 {
-#define HCLGE_MAC_CMD_NUM 17
+#define HCLGE_MAC_CMD_NUM 21
 #define HCLGE_RTN_DATA_NUM 4
 
        u64 *data = (u64 *)(&hdev->hw_stats.mac_stats);
@@ -525,7 +604,7 @@ static int hclge_tqps_update_stats(struct hnae3_handle *handle)
                        return ret;
                }
                tqp->tqp_stats.rcb_rx_ring_pktnum_rcd +=
-                       le32_to_cpu(desc[0].data[4]);
+                       le32_to_cpu(desc[0].data[1]);
        }
 
        for (i = 0; i < kinfo->num_tqps; i++) {
@@ -545,7 +624,7 @@ static int hclge_tqps_update_stats(struct hnae3_handle *handle)
                        return ret;
                }
                tqp->tqp_stats.rcb_tx_ring_pktnum_rcd +=
-                       le32_to_cpu(desc[0].data[4]);
+                       le32_to_cpu(desc[0].data[1]);
        }
 
        return 0;
@@ -587,7 +666,7 @@ static u8 *hclge_tqps_get_strings(struct hnae3_handle *handle, u8 *data)
        for (i = 0; i < kinfo->num_tqps; i++) {
                struct hclge_tqp *tqp = container_of(handle->kinfo.tqp[i],
                        struct hclge_tqp, q);
-               snprintf(buff, ETH_GSTRING_LEN, "rcb_q%d_tx_pktnum_rcd",
+               snprintf(buff, ETH_GSTRING_LEN, "txq#%d_pktnum_rcd",
                         tqp->index);
                buff = buff + ETH_GSTRING_LEN;
        }
@@ -595,7 +674,7 @@ static u8 *hclge_tqps_get_strings(struct hnae3_handle *handle, u8 *data)
        for (i = 0; i < kinfo->num_tqps; i++) {
                struct hclge_tqp *tqp = container_of(kinfo->tqp[i],
                        struct hclge_tqp, q);
-               snprintf(buff, ETH_GSTRING_LEN, "rcb_q%d_rx_pktnum_rcd",
+               snprintf(buff, ETH_GSTRING_LEN, "rxq#%d_pktnum_rcd",
                         tqp->index);
                buff = buff + ETH_GSTRING_LEN;
        }
@@ -643,23 +722,22 @@ static void hclge_update_netstat(struct hclge_hw_stats *hw_stats,
        net_stats->rx_dropped += hw_stats->all_32_bit_stats.ppp_key_drop_num;
        net_stats->rx_dropped += hw_stats->all_32_bit_stats.ssu_key_drop_num;
 
-       net_stats->rx_errors = hw_stats->mac_stats.mac_rx_overrsize_pkt_num;
+       net_stats->rx_errors = hw_stats->mac_stats.mac_rx_oversize_pkt_num;
        net_stats->rx_errors += hw_stats->mac_stats.mac_rx_undersize_pkt_num;
-       net_stats->rx_errors += hw_stats->all_32_bit_stats.igu_rx_err_pkt;
        net_stats->rx_errors += hw_stats->all_32_bit_stats.igu_rx_no_eof_pkt;
        net_stats->rx_errors += hw_stats->all_32_bit_stats.igu_rx_no_sof_pkt;
-       net_stats->rx_errors += hw_stats->mac_stats.mac_rcv_fcs_err_pkt_num;
+       net_stats->rx_errors += hw_stats->mac_stats.mac_rx_fcs_err_pkt_num;
 
        net_stats->multicast = hw_stats->mac_stats.mac_tx_multi_pkt_num;
        net_stats->multicast += hw_stats->mac_stats.mac_rx_multi_pkt_num;
 
-       net_stats->rx_crc_errors = hw_stats->mac_stats.mac_rcv_fcs_err_pkt_num;
+       net_stats->rx_crc_errors = hw_stats->mac_stats.mac_rx_fcs_err_pkt_num;
        net_stats->rx_length_errors =
                hw_stats->mac_stats.mac_rx_undersize_pkt_num;
        net_stats->rx_length_errors +=
-               hw_stats->mac_stats.mac_rx_overrsize_pkt_num;
+               hw_stats->mac_stats.mac_rx_oversize_pkt_num;
        net_stats->rx_over_errors =
-               hw_stats->mac_stats.mac_rx_overrsize_pkt_num;
+               hw_stats->mac_stats.mac_rx_oversize_pkt_num;
 }
 
 static void hclge_update_stats_for_all(struct hclge_dev *hdev)
@@ -699,6 +777,9 @@ static void hclge_update_stats(struct hnae3_handle *handle,
        struct hclge_hw_stats *hw_stats = &hdev->hw_stats;
        int status;
 
+       if (test_and_set_bit(HCLGE_STATE_STATISTICS_UPDATING, &hdev->state))
+               return;
+
        status = hclge_mac_update_stats(hdev);
        if (status)
                dev_err(&hdev->pdev->dev,
@@ -724,6 +805,8 @@ static void hclge_update_stats(struct hnae3_handle *handle,
                        status);
 
        hclge_update_netstat(hw_stats, net_stats);
+
+       clear_bit(HCLGE_STATE_STATISTICS_UPDATING, &hdev->state);
 }
 
 static int hclge_get_sset_count(struct hnae3_handle *handle, int stringset)
@@ -1100,10 +1183,7 @@ static int hclge_configure(struct hclge_dev *hdev)
        for (i = 0; i < hdev->tm_info.num_tc; i++)
                hnae_set_bit(hdev->hw_tc_map, i, 1);
 
-       if (!hdev->num_vmdq_vport && !hdev->num_req_vfs)
-               hdev->tx_sch_mode = HCLGE_FLAG_TC_BASE_SCH_MODE;
-       else
-               hdev->tx_sch_mode = HCLGE_FLAG_VNET_BASE_SCH_MODE;
+       hdev->tx_sch_mode = HCLGE_FLAG_TC_BASE_SCH_MODE;
 
        return ret;
 }
@@ -2137,28 +2217,6 @@ static int hclge_query_mac_an_speed_dup(struct hclge_dev *hdev, int *speed,
        return 0;
 }
 
-static int hclge_query_autoneg_result(struct hclge_dev *hdev)
-{
-       struct hclge_mac *mac = &hdev->hw.mac;
-       struct hclge_query_an_speed_dup_cmd *req;
-       struct hclge_desc desc;
-       int ret;
-
-       req = (struct hclge_query_an_speed_dup_cmd *)desc.data;
-
-       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_QUERY_AN_RESULT, true);
-       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
-       if (ret) {
-               dev_err(&hdev->pdev->dev,
-                       "autoneg result query cmd failed %d.\n", ret);
-               return ret;
-       }
-
-       mac->autoneg = hnae_get_bit(req->an_syn_dup_speed, HCLGE_QUERY_AN_B);
-
-       return 0;
-}
-
 static int hclge_set_autoneg_en(struct hclge_dev *hdev, bool enable)
 {
        struct hclge_config_auto_neg_cmd *req;
@@ -2194,8 +2252,10 @@ static int hclge_get_autoneg(struct hnae3_handle *handle)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
+       struct phy_device *phydev = hdev->hw.mac.phydev;
 
-       hclge_query_autoneg_result(hdev);
+       if (phydev)
+               return phydev->autoneg;
 
        return hdev->hw.mac.autoneg;
 }
@@ -2226,8 +2286,11 @@ static int hclge_set_default_mac_vlan_mask(struct hclge_dev *hdev,
 
 static int hclge_mac_init(struct hclge_dev *hdev)
 {
+       struct hnae3_handle *handle = &hdev->vport[0].nic;
+       struct net_device *netdev = handle->kinfo.netdev;
        struct hclge_mac *mac = &hdev->hw.mac;
        u8 mac_mask[ETH_ALEN] = {0x00, 0x00, 0x00, 0x00, 0x00, 0x00};
+       int mtu;
        int ret;
 
        ret = hclge_cfg_mac_speed_dup(hdev, hdev->hw.mac.speed, HCLGE_MAC_FULL);
@@ -2261,11 +2324,25 @@ static int hclge_mac_init(struct hclge_dev *hdev)
        }
 
        ret = hclge_set_default_mac_vlan_mask(hdev, true, mac_mask);
-       if (ret)
+       if (ret) {
                dev_err(&hdev->pdev->dev,
                        "set default mac_vlan_mask fail ret=%d\n", ret);
+               return ret;
+       }
 
-       return ret;
+       if (netdev)
+               mtu = netdev->mtu;
+       else
+               mtu = ETH_DATA_LEN;
+
+       ret = hclge_set_mtu(handle, mtu);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "set mtu failed ret=%d\n", ret);
+               return ret;
+       }
+
+       return 0;
 }
 
 static void hclge_mbx_task_schedule(struct hclge_dev *hdev)
@@ -2404,6 +2481,7 @@ static void hclge_service_timer(struct timer_list *t)
        struct hclge_dev *hdev = from_timer(hdev, t, service_timer);
 
        mod_timer(&hdev->service_timer, jiffies + HZ);
+       hdev->hw_stats.stats_timer++;
        hclge_task_schedule(hdev);
 }
 
@@ -2803,9 +2881,20 @@ static void hclge_service_task(struct work_struct *work)
        struct hclge_dev *hdev =
                container_of(work, struct hclge_dev, service_task);
 
+       /* The total rx/tx packets statstics are wanted to be updated
+        * per second. Both hclge_update_stats_for_all() and
+        * hclge_mac_get_traffic_stats() can do it.
+        */
+       if (hdev->hw_stats.stats_timer >= HCLGE_STATS_TIMER_INTERVAL) {
+               hclge_update_stats_for_all(hdev);
+               hdev->hw_stats.stats_timer = 0;
+       } else {
+               hclge_mac_get_traffic_stats(hdev);
+       }
+
        hclge_update_speed_duplex(hdev);
        hclge_update_link_status(hdev);
-       hclge_update_stats_for_all(hdev);
+       hclge_update_led_status(hdev);
        hclge_service_complete(hdev);
 }
 
@@ -2892,31 +2981,6 @@ static u32 hclge_get_rss_indir_size(struct hnae3_handle *handle)
        return HCLGE_RSS_IND_TBL_SIZE;
 }
 
-static int hclge_get_rss_algo(struct hclge_dev *hdev)
-{
-       struct hclge_rss_config_cmd *req;
-       struct hclge_desc desc;
-       int rss_hash_algo;
-       int ret;
-
-       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_RSS_GENERIC_CONFIG, true);
-
-       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
-       if (ret) {
-               dev_err(&hdev->pdev->dev,
-                       "Get link status error, status =%d\n", ret);
-               return ret;
-       }
-
-       req = (struct hclge_rss_config_cmd *)desc.data;
-       rss_hash_algo = (req->hash_config & HCLGE_RSS_HASH_ALGO_MASK);
-
-       if (rss_hash_algo == HCLGE_RSS_HASH_ALGO_TOEPLITZ)
-               return ETH_RSS_HASH_TOP;
-
-       return -EINVAL;
-}
-
 static int hclge_set_rss_algo_key(struct hclge_dev *hdev,
                                  const u8 hfunc, const u8 *key)
 {
@@ -2955,7 +3019,7 @@ static int hclge_set_rss_algo_key(struct hclge_dev *hdev,
        return 0;
 }
 
-static int hclge_set_rss_indir_table(struct hclge_dev *hdev, const u32 *indir)
+static int hclge_set_rss_indir_table(struct hclge_dev *hdev, const u8 *indir)
 {
        struct hclge_rss_indirection_table_cmd *req;
        struct hclge_desc desc;
@@ -3051,12 +3115,11 @@ static int hclge_get_rss(struct hnae3_handle *handle, u32 *indir,
                         u8 *key, u8 *hfunc)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
-       struct hclge_dev *hdev = vport->back;
        int i;
 
        /* Get hash algorithm */
        if (hfunc)
-               *hfunc = hclge_get_rss_algo(hdev);
+               *hfunc = vport->rss_algo;
 
        /* Get the RSS Key required by the user */
        if (key)
@@ -3080,8 +3143,6 @@ static int hclge_set_rss(struct hnae3_handle *handle, const u32 *indir,
 
        /* Set the RSS Hash Key if specififed by the user */
        if (key) {
-               /* Update the shadow RSS key with user specified qids */
-               memcpy(vport->rss_hash_key, key, HCLGE_RSS_KEY_SIZE);
 
                if (hfunc == ETH_RSS_HASH_TOP ||
                    hfunc == ETH_RSS_HASH_NO_CHANGE)
@@ -3091,6 +3152,10 @@ static int hclge_set_rss(struct hnae3_handle *handle, const u32 *indir,
                ret = hclge_set_rss_algo_key(hdev, hash_algo, key);
                if (ret)
                        return ret;
+
+               /* Update the shadow RSS key with user specified qids */
+               memcpy(vport->rss_hash_key, key, HCLGE_RSS_KEY_SIZE);
+               vport->rss_algo = hash_algo;
        }
 
        /* Update the shadow RSS table with user specified qids */
@@ -3098,8 +3163,7 @@ static int hclge_set_rss(struct hnae3_handle *handle, const u32 *indir,
                vport->rss_indirection_tbl[i] = indir[i];
 
        /* Update the hardware */
-       ret = hclge_set_rss_indir_table(hdev, indir);
-       return ret;
+       return hclge_set_rss_indir_table(hdev, vport->rss_indirection_tbl);
 }
 
 static u8 hclge_get_rss_hash_bits(struct ethtool_rxnfc *nfc)
@@ -3372,6 +3436,11 @@ int hclge_bind_ring_with_vector(struct hclge_vport *vport,
                               hnae_get_bit(node->flag, HNAE3_RING_TYPE_B));
                hnae_set_field(tqp_type_and_id, HCLGE_TQP_ID_M,
                               HCLGE_TQP_ID_S, node->tqp_index);
+               hnae_set_field(tqp_type_and_id, HCLGE_INT_GL_IDX_M,
+                              HCLGE_INT_GL_IDX_S,
+                              hnae_get_field(node->int_gl_idx,
+                                             HNAE3_RING_GL_IDX_M,
+                                             HNAE3_RING_GL_IDX_S));
                req->tqp_type_and_id[i] = cpu_to_le16(tqp_type_and_id);
                if (++i >= HCLGE_VECTOR_ELEMENTS_PER_CMD) {
                        req->int_cause_num = HCLGE_VECTOR_ELEMENTS_PER_CMD;
@@ -3625,20 +3694,11 @@ static int hclge_ae_start(struct hnae3_handle *handle)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
-       int i, queue_id, ret;
+       int i, ret;
 
-       for (i = 0; i < vport->alloc_tqps; i++) {
-               /* todo clear interrupt */
-               /* ring enable */
-               queue_id = hclge_get_queue_id(handle->kinfo.tqp[i]);
-               if (queue_id < 0) {
-                       dev_warn(&hdev->pdev->dev,
-                                "Get invalid queue id, ignore it\n");
-                       continue;
-               }
+       for (i = 0; i < vport->alloc_tqps; i++)
+               hclge_tqp_enable(hdev, i, 0, true);
 
-               hclge_tqp_enable(hdev, queue_id, 0, true);
-       }
        /* mac enable */
        hclge_cfg_mac_mode(hdev, true);
        clear_bit(HCLGE_STATE_DOWN, &hdev->state);
@@ -3658,19 +3718,11 @@ static void hclge_ae_stop(struct hnae3_handle *handle)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
-       int i, queue_id;
+       int i;
 
-       for (i = 0; i < vport->alloc_tqps; i++) {
-               /* Ring disable */
-               queue_id = hclge_get_queue_id(handle->kinfo.tqp[i]);
-               if (queue_id < 0) {
-                       dev_warn(&hdev->pdev->dev,
-                                "Get invalid queue id, ignore it\n");
-                       continue;
-               }
+       for (i = 0; i < vport->alloc_tqps; i++)
+               hclge_tqp_enable(hdev, i, 0, false);
 
-               hclge_tqp_enable(hdev, queue_id, 0, false);
-       }
        /* Mac disable */
        hclge_cfg_mac_mode(hdev, false);
 
@@ -4207,6 +4259,91 @@ int hclge_rm_mc_addr_common(struct hclge_vport *vport,
        return status;
 }
 
+static int hclge_get_mac_ethertype_cmd_status(struct hclge_dev *hdev,
+                                             u16 cmdq_resp, u8 resp_code)
+{
+#define HCLGE_ETHERTYPE_SUCCESS_ADD            0
+#define HCLGE_ETHERTYPE_ALREADY_ADD            1
+#define HCLGE_ETHERTYPE_MGR_TBL_OVERFLOW       2
+#define HCLGE_ETHERTYPE_KEY_CONFLICT           3
+
+       int return_status;
+
+       if (cmdq_resp) {
+               dev_err(&hdev->pdev->dev,
+                       "cmdq execute failed for get_mac_ethertype_cmd_status, status=%d.\n",
+                       cmdq_resp);
+               return -EIO;
+       }
+
+       switch (resp_code) {
+       case HCLGE_ETHERTYPE_SUCCESS_ADD:
+       case HCLGE_ETHERTYPE_ALREADY_ADD:
+               return_status = 0;
+               break;
+       case HCLGE_ETHERTYPE_MGR_TBL_OVERFLOW:
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for manager table overflow.\n");
+               return_status = -EIO;
+               break;
+       case HCLGE_ETHERTYPE_KEY_CONFLICT:
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for key conflict.\n");
+               return_status = -EIO;
+               break;
+       default:
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for undefined, code=%d.\n",
+                       resp_code);
+               return_status = -EIO;
+       }
+
+       return return_status;
+}
+
+static int hclge_add_mgr_tbl(struct hclge_dev *hdev,
+                            const struct hclge_mac_mgr_tbl_entry_cmd *req)
+{
+       struct hclge_desc desc;
+       u8 resp_code;
+       u16 retval;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_MAC_ETHTYPE_ADD, false);
+       memcpy(desc.data, req, sizeof(struct hclge_mac_mgr_tbl_entry_cmd));
+
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "add mac ethertype failed for cmd_send, ret =%d.\n",
+                       ret);
+               return ret;
+       }
+
+       resp_code = (le32_to_cpu(desc.data[0]) >> 8) & 0xff;
+       retval = le16_to_cpu(desc.retval);
+
+       return hclge_get_mac_ethertype_cmd_status(hdev, retval, resp_code);
+}
+
+static int init_mgr_tbl(struct hclge_dev *hdev)
+{
+       int ret;
+       int i;
+
+       for (i = 0; i < ARRAY_SIZE(hclge_mgr_table); i++) {
+               ret = hclge_add_mgr_tbl(hdev, &hclge_mgr_table[i]);
+               if (ret) {
+                       dev_err(&hdev->pdev->dev,
+                               "add mac ethertype failed, ret =%d.\n",
+                               ret);
+                       return ret;
+               }
+       }
+
+       return 0;
+}
+
 static void hclge_get_mac_addr(struct hnae3_handle *handle, u8 *p)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
@@ -4220,6 +4357,7 @@ static int hclge_set_mac_addr(struct hnae3_handle *handle, void *p)
        const unsigned char *new_addr = (const unsigned char *)p;
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
+       int ret;
 
        /* mac addr check */
        if (is_zero_ether_addr(new_addr) ||
@@ -4231,14 +4369,39 @@ static int hclge_set_mac_addr(struct hnae3_handle *handle, void *p)
                return -EINVAL;
        }
 
-       hclge_rm_uc_addr(handle, hdev->hw.mac.mac_addr);
+       ret = hclge_rm_uc_addr(handle, hdev->hw.mac.mac_addr);
+       if (ret)
+               dev_warn(&hdev->pdev->dev,
+                        "remove old uc mac address fail, ret =%d.\n",
+                        ret);
 
-       if (!hclge_add_uc_addr(handle, new_addr)) {
-               ether_addr_copy(hdev->hw.mac.mac_addr, new_addr);
-               return 0;
+       ret = hclge_add_uc_addr(handle, new_addr);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "add uc mac address fail, ret =%d.\n",
+                       ret);
+
+               ret = hclge_add_uc_addr(handle, hdev->hw.mac.mac_addr);
+               if (ret) {
+                       dev_err(&hdev->pdev->dev,
+                               "restore uc mac address fail, ret =%d.\n",
+                               ret);
+               }
+
+               return -EIO;
        }
 
-       return -EIO;
+       ret = hclge_mac_pause_addr_cfg(hdev, new_addr);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "configure mac pause address fail, ret =%d.\n",
+                       ret);
+               return -EIO;
+       }
+
+       ether_addr_copy(hdev->hw.mac.mac_addr, new_addr);
+
+       return 0;
 }
 
 static int hclge_set_vlan_filter_ctrl(struct hclge_dev *hdev, u8 vlan_type,
@@ -4264,6 +4427,17 @@ static int hclge_set_vlan_filter_ctrl(struct hclge_dev *hdev, u8 vlan_type,
        return 0;
 }
 
+#define HCLGE_FILTER_TYPE_VF           0
+#define HCLGE_FILTER_TYPE_PORT         1
+
+static void hclge_enable_vlan_filter(struct hnae3_handle *handle, bool enable)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+
+       hclge_set_vlan_filter_ctrl(hdev, HCLGE_FILTER_TYPE_VF, enable);
+}
+
 int hclge_set_vf_vlan_common(struct hclge_dev *hdev, int vfid,
                             bool is_kill, u16 vlan, u8 qos, __be16 proto)
 {
@@ -4492,8 +4666,6 @@ static int hclge_set_vlan_protocol_type(struct hclge_dev *hdev)
 
 static int hclge_init_vlan_config(struct hclge_dev *hdev)
 {
-#define HCLGE_FILTER_TYPE_VF           0
-#define HCLGE_FILTER_TYPE_PORT         1
 #define HCLGE_DEF_VLAN_TYPE            0x8100
 
        struct hnae3_handle *handle;
@@ -4547,22 +4719,39 @@ static int hclge_init_vlan_config(struct hclge_dev *hdev)
        return hclge_set_port_vlan_filter(handle, htons(ETH_P_8021Q), 0, false);
 }
 
+static int hclge_en_hw_strip_rxvtag(struct hnae3_handle *handle, bool enable)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+
+       vport->rxvlan_cfg.strip_tag1_en = false;
+       vport->rxvlan_cfg.strip_tag2_en = enable;
+       vport->rxvlan_cfg.vlan1_vlan_prionly = false;
+       vport->rxvlan_cfg.vlan2_vlan_prionly = false;
+
+       return hclge_set_vlan_rx_offload_cfg(vport);
+}
+
 static int hclge_set_mtu(struct hnae3_handle *handle, int new_mtu)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_config_max_frm_size_cmd *req;
        struct hclge_dev *hdev = vport->back;
        struct hclge_desc desc;
+       int max_frm_size;
        int ret;
 
-       if ((new_mtu < HCLGE_MAC_MIN_MTU) || (new_mtu > HCLGE_MAC_MAX_MTU))
+       max_frm_size = new_mtu + ETH_HLEN + ETH_FCS_LEN + VLAN_HLEN;
+
+       if (max_frm_size < HCLGE_MAC_MIN_FRAME ||
+           max_frm_size > HCLGE_MAC_MAX_FRAME)
                return -EINVAL;
 
-       hdev->mps = new_mtu;
+       max_frm_size = max(max_frm_size, HCLGE_MAC_DEFAULT_FRAME);
+
        hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_CONFIG_MAX_FRM_SIZE, false);
 
        req = (struct hclge_config_max_frm_size_cmd *)desc.data;
-       req->max_frm_size = cpu_to_le16(new_mtu);
+       req->max_frm_size = cpu_to_le16(max_frm_size);
 
        ret = hclge_cmd_send(&hdev->hw, &desc, 1);
        if (ret) {
@@ -4570,6 +4759,8 @@ static int hclge_set_mtu(struct hnae3_handle *handle, int new_mtu)
                return ret;
        }
 
+       hdev->mps = max_frm_size;
+
        return 0;
 }
 
@@ -4617,21 +4808,36 @@ static int hclge_get_reset_status(struct hclge_dev *hdev, u16 queue_id)
        return hnae_get_bit(req->ready_to_reset, HCLGE_TQP_RESET_B);
 }
 
+static u16 hclge_covert_handle_qid_global(struct hnae3_handle *handle,
+                                         u16 queue_id)
+{
+       struct hnae3_queue *queue;
+       struct hclge_tqp *tqp;
+
+       queue = handle->kinfo.tqp[queue_id];
+       tqp = container_of(queue, struct hclge_tqp, q);
+
+       return tqp->index;
+}
+
 void hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
 {
        struct hclge_vport *vport = hclge_get_vport(handle);
        struct hclge_dev *hdev = vport->back;
        int reset_try_times = 0;
        int reset_status;
+       u16 queue_gid;
        int ret;
 
+       queue_gid = hclge_covert_handle_qid_global(handle, queue_id);
+
        ret = hclge_tqp_enable(hdev, queue_id, 0, false);
        if (ret) {
                dev_warn(&hdev->pdev->dev, "Disable tqp fail, ret = %d\n", ret);
                return;
        }
 
-       ret = hclge_send_reset_tqp_cmd(hdev, queue_id, true);
+       ret = hclge_send_reset_tqp_cmd(hdev, queue_gid, true);
        if (ret) {
                dev_warn(&hdev->pdev->dev,
                         "Send reset tqp cmd fail, ret = %d\n", ret);
@@ -4642,7 +4848,7 @@ void hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
        while (reset_try_times++ < HCLGE_TQP_RESET_TRY_TIMES) {
                /* Wait for tqp hw reset */
                msleep(20);
-               reset_status = hclge_get_reset_status(hdev, queue_id);
+               reset_status = hclge_get_reset_status(hdev, queue_gid);
                if (reset_status)
                        break;
        }
@@ -4652,7 +4858,7 @@ void hclge_reset_tqp(struct hnae3_handle *handle, u16 queue_id)
                return;
        }
 
-       ret = hclge_send_reset_tqp_cmd(hdev, queue_id, false);
+       ret = hclge_send_reset_tqp_cmd(hdev, queue_gid, false);
        if (ret) {
                dev_warn(&hdev->pdev->dev,
                         "Deassert the soft reset fail, ret = %d\n", ret);
@@ -4668,6 +4874,100 @@ static u32 hclge_get_fw_version(struct hnae3_handle *handle)
        return hdev->fw_version;
 }
 
+static void hclge_get_flowctrl_adv(struct hnae3_handle *handle,
+                                  u32 *flowctrl_adv)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       struct phy_device *phydev = hdev->hw.mac.phydev;
+
+       if (!phydev)
+               return;
+
+       *flowctrl_adv |= (phydev->advertising & ADVERTISED_Pause) |
+                        (phydev->advertising & ADVERTISED_Asym_Pause);
+}
+
+static void hclge_set_flowctrl_adv(struct hclge_dev *hdev, u32 rx_en, u32 tx_en)
+{
+       struct phy_device *phydev = hdev->hw.mac.phydev;
+
+       if (!phydev)
+               return;
+
+       phydev->advertising &= ~(ADVERTISED_Pause | ADVERTISED_Asym_Pause);
+
+       if (rx_en)
+               phydev->advertising |= ADVERTISED_Pause | ADVERTISED_Asym_Pause;
+
+       if (tx_en)
+               phydev->advertising ^= ADVERTISED_Asym_Pause;
+}
+
+static int hclge_cfg_pauseparam(struct hclge_dev *hdev, u32 rx_en, u32 tx_en)
+{
+       int ret;
+
+       if (rx_en && tx_en)
+               hdev->fc_mode_last_time = HCLGE_FC_FULL;
+       else if (rx_en && !tx_en)
+               hdev->fc_mode_last_time = HCLGE_FC_RX_PAUSE;
+       else if (!rx_en && tx_en)
+               hdev->fc_mode_last_time = HCLGE_FC_TX_PAUSE;
+       else
+               hdev->fc_mode_last_time = HCLGE_FC_NONE;
+
+       if (hdev->tm_info.fc_mode == HCLGE_FC_PFC)
+               return 0;
+
+       ret = hclge_mac_pause_en_cfg(hdev, tx_en, rx_en);
+       if (ret) {
+               dev_err(&hdev->pdev->dev, "configure pauseparam error, ret = %d.\n",
+                       ret);
+               return ret;
+       }
+
+       hdev->tm_info.fc_mode = hdev->fc_mode_last_time;
+
+       return 0;
+}
+
+int hclge_cfg_flowctrl(struct hclge_dev *hdev)
+{
+       struct phy_device *phydev = hdev->hw.mac.phydev;
+       u16 remote_advertising = 0;
+       u16 local_advertising = 0;
+       u32 rx_pause, tx_pause;
+       u8 flowctl;
+
+       if (!phydev->link || !phydev->autoneg)
+               return 0;
+
+       if (phydev->advertising & ADVERTISED_Pause)
+               local_advertising = ADVERTISE_PAUSE_CAP;
+
+       if (phydev->advertising & ADVERTISED_Asym_Pause)
+               local_advertising |= ADVERTISE_PAUSE_ASYM;
+
+       if (phydev->pause)
+               remote_advertising = LPA_PAUSE_CAP;
+
+       if (phydev->asym_pause)
+               remote_advertising |= LPA_PAUSE_ASYM;
+
+       flowctl = mii_resolve_flowctrl_fdx(local_advertising,
+                                          remote_advertising);
+       tx_pause = flowctl & FLOW_CTRL_TX;
+       rx_pause = flowctl & FLOW_CTRL_RX;
+
+       if (phydev->duplex == HCLGE_MAC_HALF) {
+               tx_pause = 0;
+               rx_pause = 0;
+       }
+
+       return hclge_cfg_pauseparam(hdev, rx_pause, tx_pause);
+}
+
 static void hclge_get_pauseparam(struct hnae3_handle *handle, u32 *auto_neg,
                                 u32 *rx_en, u32 *tx_en)
 {
@@ -4697,6 +4997,41 @@ static void hclge_get_pauseparam(struct hnae3_handle *handle, u32 *auto_neg,
        }
 }
 
+static int hclge_set_pauseparam(struct hnae3_handle *handle, u32 auto_neg,
+                               u32 rx_en, u32 tx_en)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       struct phy_device *phydev = hdev->hw.mac.phydev;
+       u32 fc_autoneg;
+
+       /* Only support flow control negotiation for netdev with
+        * phy attached for now.
+        */
+       if (!phydev)
+               return -EOPNOTSUPP;
+
+       fc_autoneg = hclge_get_autoneg(handle);
+       if (auto_neg != fc_autoneg) {
+               dev_info(&hdev->pdev->dev,
+                        "To change autoneg please use: ethtool -s <dev> autoneg <on|off>\n");
+               return -EOPNOTSUPP;
+       }
+
+       if (hdev->tm_info.fc_mode == HCLGE_FC_PFC) {
+               dev_info(&hdev->pdev->dev,
+                        "Priority flow control enabled. Cannot set link flow control.\n");
+               return -EOPNOTSUPP;
+       }
+
+       hclge_set_flowctrl_adv(hdev, rx_en, tx_en);
+
+       if (!fc_autoneg)
+               return hclge_cfg_pauseparam(hdev, rx_en, tx_en);
+
+       return phy_start_aneg(phydev);
+}
+
 static void hclge_get_ksettings_an_result(struct hnae3_handle *handle,
                                          u8 *auto_neg, u32 *speed, u8 *duplex)
 {
@@ -5046,6 +5381,12 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
                return ret;
        }
 
+       ret = init_mgr_tbl(hdev);
+       if (ret) {
+               dev_err(&pdev->dev, "manager table init fail, ret =%d\n", ret);
+               return ret;
+       }
+
        hclge_dcb_ops_set(hdev);
 
        timer_setup(&hdev->service_timer, hclge_service_timer, 0);
@@ -5319,6 +5660,318 @@ static int hclge_set_channels(struct hnae3_handle *handle, u32 new_tqps_num)
        return ret;
 }
 
+static int hclge_get_regs_num(struct hclge_dev *hdev, u32 *regs_num_32_bit,
+                             u32 *regs_num_64_bit)
+{
+       struct hclge_desc desc;
+       u32 total_num;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_QUERY_REG_NUM, true);
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Query register number cmd failed, ret = %d.\n", ret);
+               return ret;
+       }
+
+       *regs_num_32_bit = le32_to_cpu(desc.data[0]);
+       *regs_num_64_bit = le32_to_cpu(desc.data[1]);
+
+       total_num = *regs_num_32_bit + *regs_num_64_bit;
+       if (!total_num)
+               return -EINVAL;
+
+       return 0;
+}
+
+static int hclge_get_32_bit_regs(struct hclge_dev *hdev, u32 regs_num,
+                                void *data)
+{
+#define HCLGE_32_BIT_REG_RTN_DATANUM 8
+
+       struct hclge_desc *desc;
+       u32 *reg_val = data;
+       __le32 *desc_data;
+       int cmd_num;
+       int i, k, n;
+       int ret;
+
+       if (regs_num == 0)
+               return 0;
+
+       cmd_num = DIV_ROUND_UP(regs_num + 2, HCLGE_32_BIT_REG_RTN_DATANUM);
+       desc = kcalloc(cmd_num, sizeof(struct hclge_desc), GFP_KERNEL);
+       if (!desc)
+               return -ENOMEM;
+
+       hclge_cmd_setup_basic_desc(&desc[0], HCLGE_OPC_QUERY_32_BIT_REG, true);
+       ret = hclge_cmd_send(&hdev->hw, desc, cmd_num);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Query 32 bit register cmd failed, ret = %d.\n", ret);
+               kfree(desc);
+               return ret;
+       }
+
+       for (i = 0; i < cmd_num; i++) {
+               if (i == 0) {
+                       desc_data = (__le32 *)(&desc[i].data[0]);
+                       n = HCLGE_32_BIT_REG_RTN_DATANUM - 2;
+               } else {
+                       desc_data = (__le32 *)(&desc[i]);
+                       n = HCLGE_32_BIT_REG_RTN_DATANUM;
+               }
+               for (k = 0; k < n; k++) {
+                       *reg_val++ = le32_to_cpu(*desc_data++);
+
+                       regs_num--;
+                       if (!regs_num)
+                               break;
+               }
+       }
+
+       kfree(desc);
+       return 0;
+}
+
+static int hclge_get_64_bit_regs(struct hclge_dev *hdev, u32 regs_num,
+                                void *data)
+{
+#define HCLGE_64_BIT_REG_RTN_DATANUM 4
+
+       struct hclge_desc *desc;
+       u64 *reg_val = data;
+       __le64 *desc_data;
+       int cmd_num;
+       int i, k, n;
+       int ret;
+
+       if (regs_num == 0)
+               return 0;
+
+       cmd_num = DIV_ROUND_UP(regs_num + 1, HCLGE_64_BIT_REG_RTN_DATANUM);
+       desc = kcalloc(cmd_num, sizeof(struct hclge_desc), GFP_KERNEL);
+       if (!desc)
+               return -ENOMEM;
+
+       hclge_cmd_setup_basic_desc(&desc[0], HCLGE_OPC_QUERY_64_BIT_REG, true);
+       ret = hclge_cmd_send(&hdev->hw, desc, cmd_num);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Query 64 bit register cmd failed, ret = %d.\n", ret);
+               kfree(desc);
+               return ret;
+       }
+
+       for (i = 0; i < cmd_num; i++) {
+               if (i == 0) {
+                       desc_data = (__le64 *)(&desc[i].data[0]);
+                       n = HCLGE_64_BIT_REG_RTN_DATANUM - 1;
+               } else {
+                       desc_data = (__le64 *)(&desc[i]);
+                       n = HCLGE_64_BIT_REG_RTN_DATANUM;
+               }
+               for (k = 0; k < n; k++) {
+                       *reg_val++ = le64_to_cpu(*desc_data++);
+
+                       regs_num--;
+                       if (!regs_num)
+                               break;
+               }
+       }
+
+       kfree(desc);
+       return 0;
+}
+
+static int hclge_get_regs_len(struct hnae3_handle *handle)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       u32 regs_num_32_bit, regs_num_64_bit;
+       int ret;
+
+       ret = hclge_get_regs_num(hdev, &regs_num_32_bit, &regs_num_64_bit);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get register number failed, ret = %d.\n", ret);
+               return -EOPNOTSUPP;
+       }
+
+       return regs_num_32_bit * sizeof(u32) + regs_num_64_bit * sizeof(u64);
+}
+
+static void hclge_get_regs(struct hnae3_handle *handle, u32 *version,
+                          void *data)
+{
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       u32 regs_num_32_bit, regs_num_64_bit;
+       int ret;
+
+       *version = hdev->fw_version;
+
+       ret = hclge_get_regs_num(hdev, &regs_num_32_bit, &regs_num_64_bit);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get register number failed, ret = %d.\n", ret);
+               return;
+       }
+
+       ret = hclge_get_32_bit_regs(hdev, regs_num_32_bit, data);
+       if (ret) {
+               dev_err(&hdev->pdev->dev,
+                       "Get 32 bit register failed, ret = %d.\n", ret);
+               return;
+       }
+
+       data = (u32 *)data + regs_num_32_bit;
+       ret = hclge_get_64_bit_regs(hdev, regs_num_64_bit,
+                                   data);
+       if (ret)
+               dev_err(&hdev->pdev->dev,
+                       "Get 64 bit register failed, ret = %d.\n", ret);
+}
+
+static int hclge_set_led_status_sfp(struct hclge_dev *hdev, u8 speed_led_status,
+                                   u8 act_led_status, u8 link_led_status,
+                                   u8 locate_led_status)
+{
+       struct hclge_set_led_state_cmd *req;
+       struct hclge_desc desc;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_LED_STATUS_CFG, false);
+
+       req = (struct hclge_set_led_state_cmd *)desc.data;
+       hnae_set_field(req->port_speed_led_config, HCLGE_LED_PORT_SPEED_STATE_M,
+                      HCLGE_LED_PORT_SPEED_STATE_S, speed_led_status);
+       hnae_set_field(req->link_led_config, HCLGE_LED_ACTIVITY_STATE_M,
+                      HCLGE_LED_ACTIVITY_STATE_S, act_led_status);
+       hnae_set_field(req->activity_led_config, HCLGE_LED_LINK_STATE_M,
+                      HCLGE_LED_LINK_STATE_S, link_led_status);
+       hnae_set_field(req->locate_led_config, HCLGE_LED_LOCATE_STATE_M,
+                      HCLGE_LED_LOCATE_STATE_S, locate_led_status);
+
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret)
+               dev_err(&hdev->pdev->dev,
+                       "Send set led state cmd error, ret =%d\n", ret);
+
+       return ret;
+}
+
+enum hclge_led_status {
+       HCLGE_LED_OFF,
+       HCLGE_LED_ON,
+       HCLGE_LED_NO_CHANGE = 0xFF,
+};
+
+static int hclge_set_led_id(struct hnae3_handle *handle,
+                           enum ethtool_phys_id_state status)
+{
+#define BLINK_FREQUENCY                2
+       struct hclge_vport *vport = hclge_get_vport(handle);
+       struct hclge_dev *hdev = vport->back;
+       struct phy_device *phydev = hdev->hw.mac.phydev;
+       int ret = 0;
+
+       if (phydev || hdev->hw.mac.media_type != HNAE3_MEDIA_TYPE_FIBER)
+               return -EOPNOTSUPP;
+
+       switch (status) {
+       case ETHTOOL_ID_ACTIVE:
+               ret = hclge_set_led_status_sfp(hdev,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_ON);
+               break;
+       case ETHTOOL_ID_INACTIVE:
+               ret = hclge_set_led_status_sfp(hdev,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_NO_CHANGE,
+                                              HCLGE_LED_OFF);
+               break;
+       default:
+               ret = -EINVAL;
+               break;
+       }
+
+       return ret;
+}
+
+enum hclge_led_port_speed {
+       HCLGE_SPEED_LED_FOR_1G,
+       HCLGE_SPEED_LED_FOR_10G,
+       HCLGE_SPEED_LED_FOR_25G,
+       HCLGE_SPEED_LED_FOR_40G,
+       HCLGE_SPEED_LED_FOR_50G,
+       HCLGE_SPEED_LED_FOR_100G,
+};
+
+static u8 hclge_led_get_speed_status(u32 speed)
+{
+       u8 speed_led;
+
+       switch (speed) {
+       case HCLGE_MAC_SPEED_1G:
+               speed_led = HCLGE_SPEED_LED_FOR_1G;
+               break;
+       case HCLGE_MAC_SPEED_10G:
+               speed_led = HCLGE_SPEED_LED_FOR_10G;
+               break;
+       case HCLGE_MAC_SPEED_25G:
+               speed_led = HCLGE_SPEED_LED_FOR_25G;
+               break;
+       case HCLGE_MAC_SPEED_40G:
+               speed_led = HCLGE_SPEED_LED_FOR_40G;
+               break;
+       case HCLGE_MAC_SPEED_50G:
+               speed_led = HCLGE_SPEED_LED_FOR_50G;
+               break;
+       case HCLGE_MAC_SPEED_100G:
+               speed_led = HCLGE_SPEED_LED_FOR_100G;
+               break;
+       default:
+               speed_led = HCLGE_LED_NO_CHANGE;
+       }
+
+       return speed_led;
+}
+
+static int hclge_update_led_status(struct hclge_dev *hdev)
+{
+       u8 port_speed_status, link_status, activity_status;
+       u64 rx_pkts, tx_pkts;
+
+       if (hdev->hw.mac.media_type != HNAE3_MEDIA_TYPE_FIBER)
+               return 0;
+
+       port_speed_status = hclge_led_get_speed_status(hdev->hw.mac.speed);
+
+       rx_pkts = hdev->hw_stats.mac_stats.mac_rx_total_pkt_num;
+       tx_pkts = hdev->hw_stats.mac_stats.mac_tx_total_pkt_num;
+       if (rx_pkts != hdev->rx_pkts_for_led ||
+           tx_pkts != hdev->tx_pkts_for_led)
+               activity_status = HCLGE_LED_ON;
+       else
+               activity_status = HCLGE_LED_OFF;
+       hdev->rx_pkts_for_led = rx_pkts;
+       hdev->tx_pkts_for_led = tx_pkts;
+
+       if (hdev->hw.mac.link)
+               link_status = HCLGE_LED_ON;
+       else
+               link_status = HCLGE_LED_OFF;
+
+       return hclge_set_led_status_sfp(hdev, port_speed_status,
+                                       activity_status, link_status,
+                                       HCLGE_LED_NO_CHANGE);
+}
+
 static const struct hnae3_ae_ops hclge_ops = {
        .init_ae_dev = hclge_init_ae_dev,
        .uninit_ae_dev = hclge_uninit_ae_dev,
@@ -5352,6 +6005,7 @@ static const struct hnae3_ae_ops hclge_ops = {
        .set_autoneg = hclge_set_autoneg,
        .get_autoneg = hclge_get_autoneg,
        .get_pauseparam = hclge_get_pauseparam,
+       .set_pauseparam = hclge_set_pauseparam,
        .set_mtu = hclge_set_mtu,
        .reset_queue = hclge_reset_tqp,
        .get_stats = hclge_get_stats,
@@ -5360,12 +6014,18 @@ static const struct hnae3_ae_ops hclge_ops = {
        .get_sset_count = hclge_get_sset_count,
        .get_fw_version = hclge_get_fw_version,
        .get_mdix_mode = hclge_get_mdix_mode,
+       .enable_vlan_filter = hclge_enable_vlan_filter,
        .set_vlan_filter = hclge_set_port_vlan_filter,
        .set_vf_vlan_filter = hclge_set_vf_vlan_filter,
+       .enable_hw_strip_rxvtag = hclge_en_hw_strip_rxvtag,
        .reset_event = hclge_reset_event,
        .get_tqps_and_rss_info = hclge_get_tqps_and_rss_info,
        .set_channels = hclge_set_channels,
        .get_channels = hclge_get_channels,
+       .get_flowctrl_adv = hclge_get_flowctrl_adv,
+       .get_regs_len = hclge_get_regs_len,
+       .get_regs = hclge_get_regs,
+       .set_led_id = hclge_set_led_id,
 };
 
 static struct hnae3_ae_algo ae_algo = {